You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "kirklund (GitHub)" <gi...@apache.org> on 2019/09/18 23:24:49 UTC

[GitHub] [geode] kirklund commented on issue #4003: GEODE-6964: Move geode log4j core classes to geode-log4j

@upthewaterspout I'm committed to properly exposing SPI classes as non-internal. I also intend to finish up my geode-logback repo which implements a logback provider with a corresponding blog article. I've discussed this (having non-internal SPIs) with other Geode committers and most developers approve of using non-internal SPIs. My opinion is that we should not have ANY internal APIs that are exported for other modules or for SPI usage. Please note that I feel very strongly about this and do not want to be involved in crafting internal SPIs or exporting internal APIs.

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