[GitHub] [geode] kirklund commented on issue #2915: GEODE-6117: Makes modules out of geode-core and geode-cq

["kirklund (GitHub)" <gi...@apache.org>]

I forgot about geode-cq as a module. It still seems like cq is part of query(ing) though. We can't use org.apache.geode.query.cq inside geode-cq? Or is that just convention? Would we ever create geode-query to contain org.apache.geode.query and then geode-cq adds cq onto that?

I'm thinking that geode-core will eventually contain org.apache.geode.logging to contain our LogService and SPI for the LogWriter and Alert appenders (or even move that to geode-logging). Then geode-log4j would contain our default impls of that SPI which are the LogWriter and Alert appenders that are written with log4j-core. Then geode-core uses geode-logging and geode-log4j is an optional provider to the SPI in geode-logging. Or is that too many sub-modules (I'm new to modules!)?

So, my input on cq is based mainly on cq being something that builds on query. If org.apache.geode.query.cq is ugly because of geode-cq or if we would never split out query to geode-query then my suggestion probably isn't very relevant. Either way I'll mark my request as resolved and let you decide which way to go.

[ Full content available at: https://github.com/apache/geode/pull/2915 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org