Re: [DISCUSS] and the NEW Apache Geode 1.7.0 release branch has been created

2018-09-13 Thread Sai Boorlagadda
+1 to revert and fix on develop On Wed, Sep 12, 2018, 4:43 PM Nabarun Nag wrote: > Reverting them on release/1.7.0 will bring it to the previous status quo, > how all previous releases were done. I don't think anyone will build > release/1.7.0 repeatedly, hence there is no advantage of making bu

Re: [DISCUSS] and the NEW Apache Geode 1.7.0 release branch has been created

2018-09-13 Thread Patrick Rhomberg
Sounds like we have a plan! I'll take it upon myself to do the revert. On Thu, Sep 13, 2018 at 7:24 AM, Sai Boorlagadda wrote: > +1 to revert and fix on develop > > On Wed, Sep 12, 2018, 4:43 PM Nabarun Nag wrote: > > > Reverting them on release/1.7.0 will bring it to the previous status quo,

Re: [DISCUSS] and the NEW Apache Geode 1.7.0 release branch has been created

2018-09-13 Thread Nabarun Nag
Thank you Patrick. Please do send a notification email once this is reverted in the release/1.7.0 branch. Thank you Jinmei for putting the fix for GEODE-5727 into release/1.7.0. However the GEODE ticket is still in open state. Can it be closed? Regards Nabarun Nag On Thu, Sep 13, 2018 at 9:21 AM

Re: [DISCUSS] and the NEW Apache Geode 1.7.0 release branch has been created

2018-09-13 Thread Nabarun Nag
GEODE-5727 is closed and merged to release/1.7.0 Thank you Jinmei On Thu, Sep 13, 2018 at 9:47 AM Nabarun Nag wrote: > Thank you Patrick. Please do send a notification email once this is > reverted in the release/1.7.0 branch. > Thank you Jinmei for putting the fix for GEODE-5727 into release/1.

[PROPOSAL] Change Geode Log4J2 appenders to avoid programmatic add/remove

2018-09-13 Thread Kirk Lund
Geode currently manipulates Log4J2 at runtime to add/remove Appenders based on the Cache lifecycle. While working on GEODE-5637, Sai and I tried to add ListAppender [1] from log4j-core test-jar programmatically at runtime using the same code that Geode uses for AlertAppender, LogWriterAppender and

Re: [DISCUSS] and the NEW Apache Geode 1.7.0 release branch has been created

2018-09-13 Thread Patrick Rhomberg
Revert of 5600 commits pushed to release/1.7.0. Built clean locally and `$> gfsh version --full` is behaving as expected. On Thu, Sep 13, 2018 at 9:49 AM, Nabarun Nag wrote: > GEODE-5727 is closed and merged to release/1.7.0 > Thank you Jinmei > > On Thu, Sep 13, 2018 at 9:47 AM Nabarun Nag wr

Re: [DISCUSS] and the NEW Apache Geode 1.7.0 release branch has been created

2018-09-13 Thread Nabarun Nag
Thank you Patrick. Starting with creation of the first release candidate Regards Nabarun Nag On Thu, Sep 13, 2018 at 11:39 AM Patrick Rhomberg wrote: > Revert of 5600 commits pushed to release/1.7.0. Built clean locally and > `$> gfsh version --full` is behaving as expected. > > On Thu, Sep 13

Re: [PROPOSAL] Change Geode Log4J2 appenders to avoid programmatic add/remove

2018-09-13 Thread Jens Deppe
+1 on this approach! I'd like to work with you on this to get more familiar with our logging setup. --Jens On Thu, Sep 13, 2018 at 10:01 AM Kirk Lund wrote: > Geode currently manipulates Log4J2 at runtime to add/remove Appenders based > on the Cache lifecycle. While working on GEODE-5637, Sai

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #1039 was SUCCESSFUL (with 2456 tests)

2018-09-13 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #1039 was successful. --- Scheduled 2458 tests in total. https://build.spring.io/browse/SGF-NAG-1039/ -- Thi