aballano
03/03/2020, 4:04 PMJannis
03/03/2020, 4:06 PMarrow-*-test
libraries as well! Not all of them are published atmaballano
03/03/2020, 4:08 PMJannis
03/03/2020, 4:08 PMarrow-core-test
is right now. Also they have lots of duplicated code atm, so not sure if we want to clean that first, but that may take a while because it fails weirdly. I have a pr in fx that does that which fails in weird ways. Basically doesn't find references to stuff from arrow-core-test
despite it being included and intellij not showing anything as wrong...Jannis
03/03/2020, 4:09 PMarrow-fx-test
is not published at all for example. Everyone who was testing laws that are now in this module can't build/test with latest snapshotaballano
03/03/2020, 4:11 PMJannis
03/03/2020, 4:12 PMarrow-core-test
and previously arrow-test
Rachel
03/03/2020, 4:19 PMsimon.vergauwen
03/03/2020, 4:19 PMCoroutineContext
receiver functions in Concurrent
and add functions that add CoroutineContext
as the first param.
I.e. parMapN(ctx: CoroutineContext, …)
EDIT: add parTupledN
as wellsimon.vergauwen
03/03/2020, 4:20 PMRachel
03/03/2020, 4:20 PMarrow-XXX-test
modules aren't in OSS repository, just arrow-core-test
Rachel
03/03/2020, 4:21 PMarrow-XXX-test
modules have the same package and it could generate "problems"aballano
03/03/2020, 4:22 PMRachel
03/03/2020, 4:23 PMJannis
03/03/2020, 4:28 PMI think the refactor can be done now or afterwards👍 Don't think it creates any problems. Once they are all published I can give it a test to see if there are any conflicts because of it.
Rachel
03/03/2020, 4:28 PMJorge Castillo
03/04/2020, 11:30 AMaballano
03/04/2020, 11:32 AMJannis
03/04/2020, 11:46 AMaballano
03/04/2020, 11:52 AM