These are chat archives for atomix/atomix

4th
Jun 2018
Johno Crawford
@johnou
Jun 04 2018 14:17
@kuujo tests pass locally now in the classpath-scanning branch
Jordan Halterman
@kuujo
Jun 04 2018 20:07
Cool I just got to the office and I’ll check it out
Johno Crawford
@johnou
Jun 04 2018 21:23
BTW wrt scan spec, might be nice to have that as an advanced option
We have multiple services start at once and they all contend on bootstrap
Iirc scanning full class path took thirty seconds
Using scan spec dropped to four or something
Doesn't help that it was a low end aws box with non ssd storage (staging) but still
Jordan Halterman
@kuujo
Jun 04 2018 21:55
use OSGi and it hardly has anything to scan :-)
nothing wrong with limiting the classpath that needs to be scanned, it probably just can’t be a requirement for now
Johno Crawford
@johnou
Jun 04 2018 22:57
Well requirement certainly not, I meant optional, would need to see if it's even needed first though
Jordan Halterman
@kuujo
Jun 04 2018 23:35
I was thinking just add the builder pattern to AtomixRegistry and let an instance be passed into the Atomix builder. Users should probably be able to override that class pretty easily anyways. Could even preserve the configuration file based implementation if we wanted
This thing’s pretty much done. Just going to finish updating all the primitive implementations to use the proxy API tonight. Will do the RC some time this week.