Hello Jens, Thanks for your detailed email. That said, I'm aware of the fact that the main purpose of 1.9.2 release is to provide fixes that are required for the Spring Data Geode project, that's why I want to include GEODE-7079 :-). Long story short: GEODE-7079 can be hit by *spring-data-geode* users that restart a member configured with a persistent asynchronous event queue (with conflation enabled) without pausing the event processor. The ability to pause the event processor is what we're mainly adding in 1.9.2, that's why I believe this fix should also be included. Best regards.
On Fri, Oct 4, 2019 at 5:58 PM Jens Deppe <jde...@pivotal.io> wrote: > Hi Juan, > > The purpose of this release is to provide fixes that are required for the > Spring Data Geode project. Since the current release of SDG (2.2.0.RELEASE) > is based against Geode 1.9.1, it cannot be bumped to any newer (major or > even minor) version that may contain fixes it requires. Thus, for the > current release of SDG, any necessary Geode fixes will need to be made > against 1.9.x. > > With that in mind, unless GEODE-7079 is affecting SDG (or SBDG), I do not > wish to add it. > > Thanks for your understanding > --Jens > > On Fri, Oct 4, 2019 at 7:09 AM Juan José Ramos <jra...@pivotal.io> wrote: > > > Hello Jens, > > > > Can we add *GEODE-7079 [1]* to this release branch as well?. > > The fix basically prevents a NPE during restart of members that have > > configured asynchronous event distribution (either WAN replication of > async > > event listener) & conflation. > > Best regards. > > > > [1]: https://issues.apache.org/jira/browse/GEODE-7079 > > > > On Fri, Oct 4, 2019 at 2:49 PM Robert Houghton <rhough...@pivotal.io> > > wrote: > > > > > I'm about to merge the war pr into develop. > > > > > > On Fri, Oct 4, 2019, 06:47 Jens Deppe <jde...@pivotal.io> wrote: > > > > > > > Hello Geode Dev Community, > > > > > > > > We have created a new release branch for Apache Geode 1.9.2 - > > > > "release/1.9.2" > > > > > > > > > > > > This branch currently includes the following commits: > > > > > > > > > > > > GEODE-7124: Fixing tests for older version > > > > GEODE-7204: Add changes to AEQ documentation for > > > > 'pause-event-processing' functionality (#4053) > > > > GEODE-7179: alter async queue command to change state of event > > > > processor during creation (#4057) > > > > GEODE-7128: APIs and GFSH commands to resume AEQ processing. (#4056) > > > > GEODE-7149: Changes to support AsyncEventQueue's dispatcher status > > > > with AsyncEventQueue beans (#4029) > > > > GEODE-7127: Fixing the list AEQ command. > > > > GEODE-7127: Added gfsh arguments for starting AEQ with paused event > > > > processing. > > > > GEODE-7126: Refactoring API names to be more consistent. > > > > GEODE-7129: Adding XML config for creating AEQ with paused event > > > > processing. > > > > GEODE-7126: Added new API to resume AEQ event processing > > > > GEODE-7124: Added new API to create AEQ with paused event processing > > > > GEODE-7261: Include spring-core in the geode-web war artifact > > > > > > > > Please do review and raise any concern with the release branch. > > > > If no concerns are raised, we will start with the voting for the > > > > release candidate soon. > > > > > > > > Regards > > > > --Jens > > > > > > > > > > > > > -- > > Juan José Ramos Cassella > > Senior Software Engineer > > Email: jra...@pivotal.io > > > -- Juan José Ramos Cassella Senior Software Engineer Email: jra...@pivotal.io
