ctubbsii commented on issue #708: Simplify API definition by moving impl packages from under API packages URL: https://github.com/apache/accumulo/issues/708#issuecomment-436026721 Could also go with `ximpl`, but that hurts readability. `x.impl → impl.x` is problematic because: 1. it creates potential for collision across jars (see the minicluster example above, which would create the "top-level" `org.apache.accumulo.impl` package in the more narrowly scoped minicluster jar), and 2. we lose the whole attempt to scope the implementation "near" the API to which it corresponds.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
