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

Posted to notifications@geode.apache.org by "mcmellawatt (GitHub)" <gi...@apache.org> on 2019/09/18 00:04:26 UTC

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

Thanks for the picture @aaronlindsey!  I think we are on the same page regarding the end result.  Currently in the GEODE-7177 PR we have moved select classes which depend on log4j into geode-logging, but now that seems a bit odd since those should really be in geode-log4j (as they are in this PR).  I think we should get this PR merged in, then take another pass at getting the membership logging dependencies into geode-logging without moving any log4j related stuff.

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