[jira] [Commented] (GEODE-165) Add build support for generating antlr classes from grammar

[ https://issues.apache.org/jira/browse/GEODE-165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15769133#comment-15769133 ] ASF subversion and git services commented on GEODE-165: --- Commit

[jira] [Commented] (GEODE-2109) calling submit on ExecutionService can cause exceptions to be lost

[ https://issues.apache.org/jira/browse/GEODE-2109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15769057#comment-15769057 ] ASF GitHub Bot commented on GEODE-2109: --- Github user deepakddixit commented on the issue:

[jira] [Commented] (GEODE-2109) calling submit on ExecutionService can cause exceptions to be lost

[ https://issues.apache.org/jira/browse/GEODE-2109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15769028#comment-15769028 ] Deepak Dixit commented on GEODE-2109: - I think this is for [~deepakddixit] not for [~deepak.dixit] :)

Review Request 54964: GEODE-2243: reference the formatter relative to geode-core and not the root project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54964/ --- Review request for geode, Anthony Baker, Mark Bretl, and Dan Smith.

Re: GEODE-2243: Adjust path of spotless formatter

Hi Scott, The issue is that this line only works if your geode checkout is in a folder called open: eclipseFormatFile "${rootProject.projectDir}/../open/etc/eclipse-java-google-style.xml" That's not going to work for many people; the default name of the folder is geode if you do git clone

Re: New proposal for type definitons

I'm assuming the type ids here are a different set than the type ids used with regular PDX serialization so they won't conflict if the pdx registry assigns 1 to some class and a user puts @typeId: 1 in their json? I'm concerned that this won't really address the type explosion issue. Users that

Re: GEODE-2243: Adjust path of spotless formatter

Hi Dan, Since the spotless stuff lives on the open side, you just need to make sure to pull a fresh copy of the open side in order for the closed side build to work. I suppose we could have left the redundant xml file in the closed branch for a while so that stale open branches would still allow

Re: GEODE-2243: Adjust path of spotless formatter

I'm seeing the same thing. I went ahead and reverted the change so our build isn't broken. -Dan On Wed, Dec 21, 2016 at 4:49 PM, Mark Bretl wrote: > Jens, > > Please revert this commit. I feel like the JIRA issue, GEODE-2243, was done > with separate specific environment in

[jira] [Commented] (GEODE-2243) Modify path to spotless formatter

[ https://issues.apache.org/jira/browse/GEODE-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768727#comment-15768727 ] ASF subversion and git services commented on GEODE-2243: Commit

Fwd: geode git commit: GEODE-2243: Adjust path of spotless formatter

Jens, Please revert this commit. I feel like the JIRA issue, GEODE-2243, was done with separate specific environment in mind and not done for the community. This does not work for standard Geode development environment... Unable to apply step Eclipse Formatter to src/main/java/org/apache/geode

[jira] [Commented] (GEODE-2238) Member may fail to receive cluster configuration from locator

[ https://issues.apache.org/jira/browse/GEODE-2238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768569#comment-15768569 ] Kirk Lund commented on GEODE-2238: -- There are two request/response messages involving cluster config: 1)

[jira] [Commented] (GEODE-2238) Member may fail to receive cluster configuration from locator

[ https://issues.apache.org/jira/browse/GEODE-2238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768547#comment-15768547 ] Kirk Lund commented on GEODE-2238: -- The LocatorStarterRule currently has a workaround related to the

[jira] [Commented] (GEODE-2232) Rename GemFireConfigException

[ https://issues.apache.org/jira/browse/GEODE-2232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768519#comment-15768519 ] Anthony Baker commented on GEODE-2232: -- Probably this discussion can be split into three parts: -

Re: Review Request 54924: GEODE-2238: Fix how peers discover locators with cluster config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54924/#review159878 --- Ship it! Ship It! - Kirk Lund On Dec. 21, 2016, 11:40 p.m.,

Re: Review Request 54924: GEODE-2238: Fix how peers discover locators with cluster config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54924/ --- (Updated Dec. 21, 2016, 11:40 p.m.) Review request for geode, Bruce

[jira] [Commented] (GEODE-2238) Member may fail to receive cluster configuration from locator

[ https://issues.apache.org/jira/browse/GEODE-2238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768480#comment-15768480 ] Kirk Lund commented on GEODE-2238: -- I closed GEODE-2244 as a duplicate of GEODE-2238. Below is where

[jira] [Resolved] (GEODE-2244) Cluster config start is async and may not be ready when requests start coming in

[ https://issues.apache.org/jira/browse/GEODE-2244?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund resolved GEODE-2244. -- Resolution: Duplicate > Cluster config start is async and may not be ready when requests start coming

[jira] [Closed] (GEODE-2244) Cluster config start is async and may not be ready when requests start coming in

[ https://issues.apache.org/jira/browse/GEODE-2244?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund closed GEODE-2244. > Cluster config start is async and may not be ready when requests start coming > in >

[jira] [Assigned] (GEODE-2244) Cluster config start is async and may not be ready when requests start coming in

[ https://issues.apache.org/jira/browse/GEODE-2244?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund reassigned GEODE-2244: Assignee: Kirk Lund > Cluster config start is async and may not be ready when requests start

[jira] [Updated] (GEODE-2244) Cluster config start is async and may not be ready when requests start coming in

[ https://issues.apache.org/jira/browse/GEODE-2244?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund updated GEODE-2244: - Affects Version/s: 1.0.0-incubating > Cluster config start is async and may not be ready when requests

[jira] [Updated] (GEODE-2244) Cluster config start is async and may not be ready when requests start coming in

[ https://issues.apache.org/jira/browse/GEODE-2244?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund updated GEODE-2244: - Component/s: configuration > Cluster config start is async and may not be ready when requests start

[jira] [Commented] (GEODE-2238) Member may fail to receive cluster configuration from locator

[ https://issues.apache.org/jira/browse/GEODE-2238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768466#comment-15768466 ] Dan Smith commented on GEODE-2238: -- Even with my fix to the changes, I saw GEODE-1976 happen in my

[jira] [Commented] (GEODE-2243) Modify path to spotless formatter

[ https://issues.apache.org/jira/browse/GEODE-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768407#comment-15768407 ] ASF subversion and git services commented on GEODE-2243: Commit

[jira] [Resolved] (GEODE-1943) CI failure: SessionReplicationIntegrationJUnitTest.testInvalidateSession9

[ https://issues.apache.org/jira/browse/GEODE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh resolved GEODE-1943. Resolution: Fixed Fix Version/s: 1.1.0 > CI failure:

[jira] [Resolved] (GEODE-2205) Race condition in startup of ConcurrentSerialGatewaySenderProcessor

[ https://issues.apache.org/jira/browse/GEODE-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh resolved GEODE-2205. Resolution: Fixed Fix Version/s: 1.1.0 Was introduced and fixed before 1.1 has released >

[jira] [Assigned] (GEODE-1943) CI failure: SessionReplicationIntegrationJUnitTest.testInvalidateSession9

[ https://issues.apache.org/jira/browse/GEODE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh reassigned GEODE-1943: -- Assignee: Jason Huynh > CI failure:

[jira] [Created] (GEODE-2243) Modify path to spotless formatter

Jens Deppe created GEODE-2243: - Summary: Modify path to spotless formatter Key: GEODE-2243 URL: https://issues.apache.org/jira/browse/GEODE-2243 Project: Geode Issue Type: Improvement

[jira] [Assigned] (GEODE-2243) Modify path to spotless formatter

[ https://issues.apache.org/jira/browse/GEODE-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jens Deppe reassigned GEODE-2243: - Assignee: Jens Deppe > Modify path to spotless formatter > - > >

[jira] [Commented] (GEODE-2205) Race condition in startup of ConcurrentSerialGatewaySenderProcessor

[ https://issues.apache.org/jira/browse/GEODE-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768402#comment-15768402 ] Jason Huynh commented on GEODE-2205: Commit as 164f04fbd85f20de6c7f9edef267d3f48463a954 Was committed

[jira] [Commented] (GEODE-2215) NPE in ViewCreator thread setting public keys into a NetView

[ https://issues.apache.org/jira/browse/GEODE-2215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768399#comment-15768399 ] Jason Huynh commented on GEODE-2215: Sorry, I had the wrong jira number with this commit. Please

Re: Review Request 54943: GEODE-2197: refactor cluster config

> On Dec. 21, 2016, 8:52 p.m., Bruce Schuchardt wrote: > > geode-core/src/main/java/org/apache/geode/distributed/internal/SharedConfiguration.java, > > line 743 > > > > > > I can't tell from the diff if this method

[jira] [Commented] (GEODE-2232) Rename GemFireConfigException

[ https://issues.apache.org/jira/browse/GEODE-2232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768352#comment-15768352 ] Mark Bretl commented on GEODE-2232: --- I would vote for removing product names from class names as well. I

[jira] [Updated] (GEODE-165) Add build support for generating antlr classes from grammar

[ https://issues.apache.org/jira/browse/GEODE-165?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Bretl updated GEODE-165: - Assignee: Avinash Dongre (was: Mark Bretl) > Add build support for generating antlr classes from grammar

[jira] [Commented] (GEODE-165) Add build support for generating antlr classes from grammar

[ https://issues.apache.org/jira/browse/GEODE-165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768336#comment-15768336 ] ASF GitHub Bot commented on GEODE-165: -- Github user scmbuildguy commented on the issue:

[GitHub] geode issue #322: [GEODE-165] Fix for Add build support for generating antlr...

Github user scmbuildguy commented on the issue: https://github.com/apache/geode/pull/322 +1 - I agree with generating these files should be done at build time and not checked into source. --- If your project is set up for it, you can reply to this email and have your reply appear

[jira] [Commented] (GEODE-2232) Rename GemFireConfigException

[ https://issues.apache.org/jira/browse/GEODE-2232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768328#comment-15768328 ] Darrel Schneider commented on GEODE-2232: - In addition to the exceptions listed we also have the

[jira] [Commented] (GEODE-2232) Rename GemFireConfigException

[ https://issues.apache.org/jira/browse/GEODE-2232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768317#comment-15768317 ] Darrel Schneider commented on GEODE-2232: - I think this would causes issues with being backwards

[jira] [Updated] (GEODE-2232) Rename GemFireConfigException

[ https://issues.apache.org/jira/browse/GEODE-2232?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Darrel Schneider updated GEODE-2232: Assignee: Bruce Schuchardt > Rename GemFireConfigException > -

[jira] [Updated] (GEODE-2232) Rename GemFireConfigException

[ https://issues.apache.org/jira/browse/GEODE-2232?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Darrel Schneider updated GEODE-2232: Component/s: (was: core) > Rename GemFireConfigException >

Review Request 54948: fix unsafe concurrent mods on expiredTombstones ArrayList

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54948/ --- Review request for geode and anilkumar gingade. Bugs: GEODE-2240

[jira] [Commented] (GEODE-165) Add build support for generating antlr classes from grammar

[ https://issues.apache.org/jira/browse/GEODE-165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768252#comment-15768252 ] ASF GitHub Bot commented on GEODE-165: -- Github user upthewaterspout commented on the issue:

[GitHub] geode issue #322: [GEODE-165] Fix for Add build support for generating antlr...

Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/322 @agingade - Yeah, they'll get generated every time you do a clean followed a by a build. Presumably if you don't do a clean the antlr plugin will detect if the files are up to date.

Re: Review Request 54943: GEODE-2197: refactor cluster config

If it's used at startup you could have this happen: Locator A running 1.0.0 Locator B running 1.0.0 Server R running 1.0.0 Server S running 1.0.0 In a rolling upgrade Locators are rolled to the new version first, one by one. Locator B running 1.0.0 Server R running 1.0.0

[jira] [Created] (GEODE-2242) Destroy operations on PRELOADED regions are not applied in the receiving WAN site

Barry Oglesby created GEODE-2242: Summary: Destroy operations on PRELOADED regions are not applied in the receiving WAN site Key: GEODE-2242 URL: https://issues.apache.org/jira/browse/GEODE-2242

[jira] [Assigned] (GEODE-2242) Destroy operations on PRELOADED regions are not applied in the receiving WAN site

[ https://issues.apache.org/jira/browse/GEODE-2242?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Barry Oglesby reassigned GEODE-2242: Assignee: Barry Oglesby > Destroy operations on PRELOADED regions are not applied in the

Re: Review Request 54943: GEODE-2197: refactor cluster config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54943/#review159856 --- I don't know much about shared-configuration code but I looked

[jira] [Commented] (GEODE-165) Add build support for generating antlr classes from grammar

[ https://issues.apache.org/jira/browse/GEODE-165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768129#comment-15768129 ] ASF GitHub Bot commented on GEODE-165: -- Github user agingade commented on the issue:

[jira] [Commented] (GEODE-2109) calling submit on ExecutionService can cause exceptions to be lost

[ https://issues.apache.org/jira/browse/GEODE-2109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768124#comment-15768124 ] ASF GitHub Bot commented on GEODE-2109: --- Github user asfgit closed the pull request at:

[GitHub] geode pull request #296: GEODE-2109 : Calling submit on ExecutionService can...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/296 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[jira] [Commented] (GEODE-2109) calling submit on ExecutionService can cause exceptions to be lost

[ https://issues.apache.org/jira/browse/GEODE-2109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768122#comment-15768122 ] ASF subversion and git services commented on GEODE-2109: Commit

[jira] [Commented] (GEODE-2215) NPE in ViewCreator thread setting public keys into a NetView

[ https://issues.apache.org/jira/browse/GEODE-2215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768102#comment-15768102 ] ASF subversion and git services commented on GEODE-2215: Commit

[jira] [Commented] (GEODE-1943) CI failure: SessionReplicationIntegrationJUnitTest.testInvalidateSession9

[ https://issues.apache.org/jira/browse/GEODE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15768096#comment-15768096 ] ASF subversion and git services commented on GEODE-1943: Commit

[GitHub] geode issue #296: GEODE-2109 : Calling submit on ExecutionService can cause ...

Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/296 @deepakddixit - I think using LoggingThreadGroup is definitely a better fix. The changes look good to me now, I'll merge them. Thanks! --- If your project is set up for it, you can reply to

Re: Review Request 54209: GEODE-1931: CI Failure: LocatorUDPSecurityDUnitTest.testStartTwoLocators

> On Nov. 30, 2016, 6:49 p.m., Bruce Schuchardt wrote: > > I don't see anything wrong here - lots of improvements. Was there anything > > that you corrected in the test other than getting rid of pauses and > > WaitCriteria? How about running the test a few hundred times before > > checking

[jira] [Created] (GEODE-2241) gfsh lucene will generate duplicate results

xiaojian zhou created GEODE-2241: Summary: gfsh lucene will generate duplicate results Key: GEODE-2241 URL: https://issues.apache.org/jira/browse/GEODE-2241 Project: Geode Issue Type: Bug

Review Request 54945: GEODE-419: Added test to test fail over to javax properties if ssl enabled but relevant properties are not set

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54945/ --- Review request for geode, Bruce Schuchardt and Kirk Lund. Repository: geode

[jira] [Commented] (GEODE-2240) unexpected NullPointerException from Tombstone service

[ https://issues.apache.org/jira/browse/GEODE-2240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15767845#comment-15767845 ] Darrel Schneider commented on GEODE-2240: - I think this bug might be caused by the code that adds

[jira] [Commented] (GEODE-165) Add build support for generating antlr classes from grammar

[ https://issues.apache.org/jira/browse/GEODE-165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15767804#comment-15767804 ] ASF GitHub Bot commented on GEODE-165: -- Github user upthewaterspout commented on the issue:

[jira] [Created] (GEODE-2240) unexpected NullPointerException from Tombstone service

Darrel Schneider created GEODE-2240: --- Summary: unexpected NullPointerException from Tombstone service Key: GEODE-2240 URL: https://issues.apache.org/jira/browse/GEODE-2240 Project: Geode

Re: snappy-java dependency version

We switched to org.iq80.snappy because that's a pure java implementation, as opposed to just a wrapper around the C++ implementation. See GEODE-1573 for some details. I'm kinda surprised it doesn't work on Solaris. We switched to the new version

Re: Review Request 54943: GEODE-2197: refactor cluster config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54943/ --- (Updated Dec. 21, 2016, 6:16 p.m.) Review request for geode, Bruce Schuchardt,

Review Request 54943: GEODE-2197: refactor cluster config

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54943/ --- Review request for geode, Bruce Schuchardt, Jared Stewart, John Blum, Kevin

[jira] [Updated] (GEODE-2239) CI failure: org.apache.geode.pdx.JSONFormatterJUnitTest.testJSONFormatterAPIs

[ https://issues.apache.org/jira/browse/GEODE-2239?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bruce Schuchardt updated GEODE-2239: Labels: ci flaky (was: ) > CI failure:

[jira] [Resolved] (GEODE-1893) remove ability to create a locator without all location services

[ https://issues.apache.org/jira/browse/GEODE-1893?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bruce Schuchardt resolved GEODE-1893. - Resolution: Fixed Fix Version/s: 1.1.0 > remove ability to create a locator

[jira] [Commented] (GEODE-77) Replace JGroups 2.2.9

[ https://issues.apache.org/jira/browse/GEODE-77?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15767359#comment-15767359 ] ASF subversion and git services commented on GEODE-77: -- Commit

[jira] [Commented] (GEODE-1893) remove ability to create a locator without all location services

[ https://issues.apache.org/jira/browse/GEODE-1893?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15767358#comment-15767358 ] ASF subversion and git services commented on GEODE-1893: Commit