Re: Review Request 58853: GEODE-2847: Get correct version tags for retried bulk operations

2017-05-01 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58853/#review173555 --- Fix it, then Ship it! fix and ship it. geode-core/src/main/ja

Requesting Commenting Access to Geode Confluence wiki for Binary Protocol Proposal

2017-05-01 Thread Matt Kalinowski
Hello, I was invited to review the proposal for the new binary client-server protocol ( https://cwiki.apache.org/confluence/display/GEODE/New+Client+Server+Protocol) and would like to add comments to the documents/pages as part of the review. It appears I do not have commenting access. My username

Re: Review Request 58891: GEODE-2837: Start server creates specified working dir if necessary

2017-05-01 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58891/#review173551 --- Ship it! Ship It! - Jinmei Liao On May 1, 2017, 10:16 p.m.,

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-01 Thread Hitesh Khamesra
>>>The message header currently is specified to have things like correlation id, isPartial message, and also metdatadata about whether the key or the value is JSON. IsPartialMessage: This flag gives us ability to send partial message without serializing the whole key-value(request). lets say I exe

Re: Review Request 58853: GEODE-2847: Get correct version tags for retried bulk operations

2017-05-01 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58853/#review173472 --- geode-core/src/main/java/org/apache/geode/internal/cache/EventTra

Re: Review Request 58853: GEODE-2847: Get correct version tags for retried bulk operations

2017-05-01 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58853/ --- (Updated May 2, 2017, 1 a.m.) Review request for geode, anilkumar gingade, Darr

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-01 Thread Dan Smith
I think the current proposal seems to be gloming some things together that probably belong in different layers. The message header currently is specified to have things like correlation id, isPartial message, and also metdatadata about whether the key or the value is JSON. Fragmenting messages (is

[jira] [Assigned] (GEODE-1996) There are confusing references to "multicast" for locators ports

2017-05-01 Thread Dave Barnes (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1996?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dave Barnes reassigned GEODE-1996: -- Assignee: Dave Barnes > There are confusing references to "multicast" for locators ports >

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173543 --- geode-core/src/main/java/org/apache/geode/cache/query/internal/in

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-01 Thread Dan Smith
I think any new client driver or server we develop might want to incorporate function execution at lower level than region operations like get and put, etc. We could then easily build operations like GET, PUT, PUTALL, etc. on top of that by making them functions. The original client protocol isn't

Re: Remove keys from a PartitionedRegion

2017-05-01 Thread Barry Oglesby
Are you passing the keys as a filter or an argument? What does optimizeForWrite in your function return? Thanks, Barry Oglesby On Mon, May 1, 2017 at 4:41 PM, Goutam Tadi wrote: > Hi Dan, > > Thanks for the reply. > No, we are neither executing the function nor passing keys from a client. > We

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Kirk Lund
> On May 1, 2017, 11:34 p.m., Kirk Lund wrote: > > geode-core/src/test/java/org/apache/geode/cache/query/CacheUtils.java > > Line 77 (original), 80 (patched) > > > > > > These AssertionError changes may cause unexpect

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/ --- (Updated May 2, 2017, 12:06 a.m.) Review request for geode, Darrel Schneider, J

[jira] [Assigned] (GEODE-2830) Required permission for executing a function should be DATA:WRITE

2017-05-01 Thread Dave Barnes (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2830?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dave Barnes reassigned GEODE-2830: -- Assignee: Dave Barnes > Required permission for executing a function should be DATA:WRITE > ---

[jira] [Commented] (GEODE-2830) Required permission for executing a function should be DATA:WRITE

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991814#comment-15991814 ] ASF GitHub Bot commented on GEODE-2830: --- GitHub user davebarnes97 opened a pull requ

[GitHub] geode pull request #486: GEODE-2830 Required permission for executing a func...

2017-05-01 Thread davebarnes97
GitHub user davebarnes97 opened a pull request: https://github.com/apache/geode/pull/486 GEODE-2830 Required permission for executing a function should be DATA:WRITE Simple typo correction You can merge this pull request into a Git repository by running: $ git pull https:/

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Kirk Lund
> On May 1, 2017, 9:14 p.m., Darrel Schneider wrote: > > For all the RegionEntry subclasses do the following: > > * Do not modify this class. It was generated. Instead modify > > LeafRegionEntry.cpp and then run > > * ./dev-tools/generateRegionEntryClasses.sh (it must be run from the top > >

[jira] [Updated] (GEODE-2841) unable to create off-heap region from cache.xml

2017-05-01 Thread Darrel Schneider (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2841?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Darrel Schneider updated GEODE-2841: Component/s: (was: offheap) management > unable to create off-heap regi

[jira] [Commented] (GEODE-2841) unable to create off-heap region from cache.xml

2017-05-01 Thread Darrel Schneider (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2841?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991802#comment-15991802 ] Darrel Schneider commented on GEODE-2841: - This is an xml parser issue. If you cha

Re: Remove keys from a PartitionedRegion

2017-05-01 Thread Goutam Tadi
Hi Dan, Thanks for the reply. No, we are neither executing the function nor passing keys from a client. We are trying to remove a significant portion of the keys from a region (most, but not all) at once. Thanks. On Mon, May 1, 2017 at 2:40 PM Dan Smith wrote: > That seems like it should do th

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Kirk Lund
> On May 1, 2017, 10:28 p.m., Ken Howe wrote: > > geode-core/src/main/java/org/apache/geode/cache/AttributesFactory.java > > Line 237 (original), 232 (patched) > > > > > > {@code byteThreshold should be > > {@co

[jira] [Commented] (GEODE-2843) User Guide - example should specify

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2843?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991796#comment-15991796 ] ASF GitHub Bot commented on GEODE-2843: --- GitHub user davebarnes97 opened a pull requ

[GitHub] geode pull request #485: GEODE-2843 User Guide - example should specify

2017-05-01 Thread davebarnes97
GitHub user davebarnes97 opened a pull request: https://github.com/apache/geode/pull/485 GEODE-2843 User Guide - example should specify Fixed multiple occurrences in the reference. You can merge this pull request into a Git repository by running: $ git pull https://github.com

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173511 --- geode-core/src/main/java/org/apache/geode/internal/cache/ProxyReg

[jira] [Assigned] (GEODE-2843) User Guide - example should specify

2017-05-01 Thread Dave Barnes (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2843?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dave Barnes reassigned GEODE-2843: -- Assignee: Dave Barnes > User Guide - example should specify >

Re: Review Request 58682: GEODE-2662: Missing keys cause columns to shift in gfsh table display.

2017-05-01 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58682/#review173528 --- Ship it! Ship It! - Ken Howe On April 27, 2017, 6:30 p.m., P

[jira] [Commented] (GEODE-2795) Nightly build failure: CreateAlterDestroyRegionCommandsDUnitTest

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2795?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991726#comment-15991726 ] ASF subversion and git services commented on GEODE-2795: Commit 95

[jira] [Commented] (GEODE-2857) Add getProcess to ProcessWrapper

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2857?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991730#comment-15991730 ] ASF subversion and git services commented on GEODE-2857: Commit c2

[jira] [Commented] (GEODE-2723) remove "localCacheEnabled" dead code from PartitionedRegion

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991725#comment-15991725 ] ASF subversion and git services commented on GEODE-2723: Commit a5

[jira] [Commented] (GEODE-1597) gfsh parameters are not validated

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1597?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991722#comment-15991722 ] ASF subversion and git services commented on GEODE-1597: Commit 1f

[jira] [Commented] (GEODE-2823) The LuceneEventListener caues deserialized values to be stored in the entry when the region contains DataSerializable or Serializable values

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991728#comment-15991728 ] ASF subversion and git services commented on GEODE-2823: Commit 8d

[jira] [Commented] (GEODE-2840) add a dunit test to test concurrent deploy scenario

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991727#comment-15991727 ] ASF subversion and git services commented on GEODE-2840: Commit 82

[jira] [Commented] (GEODE-2632) Integrated Security performance improvements

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991729#comment-15991729 ] ASF subversion and git services commented on GEODE-2632: Commit 73

[jira] [Commented] (GEODE-2632) Integrated Security performance improvements

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991724#comment-15991724 ] ASF subversion and git services commented on GEODE-2632: Commit f1

[jira] [Commented] (GEODE-2632) Integrated Security performance improvements

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991723#comment-15991723 ] ASF subversion and git services commented on GEODE-2632: Commit a4

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

2017-05-01 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #541 was successful. --- Scheduled 1845 tests in total. https://build.spring.io/browse/SGF-NAG-541/ -- This

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173500 --- Posting my review in chunks. So far I've gotten through the first

Re: Review Request 58891: GEODE-2837: Start server creates specified working dir if necessary

2017-05-01 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58891/ --- (Updated May 1, 2017, 10:16 p.m.) Review request for geode, Jinmei Liao, Ken Ho

[jira] [Commented] (GEODE-2632) Integrated Security performance improvements

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991643#comment-15991643 ] ASF subversion and git services commented on GEODE-2632: Commit 73

[jira] [Commented] (GEODE-2795) Nightly build failure: CreateAlterDestroyRegionCommandsDUnitTest

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2795?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991640#comment-15991640 ] ASF subversion and git services commented on GEODE-2795: Commit 95

[jira] [Commented] (GEODE-2857) Add getProcess to ProcessWrapper

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2857?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991644#comment-15991644 ] ASF subversion and git services commented on GEODE-2857: Commit c2

[jira] [Commented] (GEODE-2823) The LuceneEventListener caues deserialized values to be stored in the entry when the region contains DataSerializable or Serializable values

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991642#comment-15991642 ] ASF subversion and git services commented on GEODE-2823: Commit 8d

[jira] [Commented] (GEODE-2840) add a dunit test to test concurrent deploy scenario

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991641#comment-15991641 ] ASF subversion and git services commented on GEODE-2840: Commit 82

[jira] [Commented] (GEODE-2723) remove "localCacheEnabled" dead code from PartitionedRegion

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991639#comment-15991639 ] ASF subversion and git services commented on GEODE-2723: Commit a5

[jira] [Updated] (GEODE-2829) VMRegionVersionVector allows/stores DiskStoreId as its member id

2017-05-01 Thread Fred Krone (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2829?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fred Krone updated GEODE-2829: -- Labels: storage_2 (was: ) > VMRegionVersionVector allows/stores DiskStoreId as its member id >

[jira] [Updated] (GEODE-2827) ClientCacheFactory might ignore a provided PoolFactory

2017-05-01 Thread Fred Krone (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2827?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fred Krone updated GEODE-2827: -- Component/s: (was: core) > ClientCacheFactory might ignore a provided PoolFactory >

[jira] [Updated] (GEODE-2847) Retried bulk operation from client could not get applied version tag from the server and cause data inconsistency

2017-05-01 Thread Fred Krone (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fred Krone updated GEODE-2847: -- Labels: storage_2 (was: ) > Retried bulk operation from client could not get applied version tag from t

Re: Remove keys from a PartitionedRegion

2017-05-01 Thread Dan Smith
That seems like it should do things fairly quickly. Are you executing the function from a client? Did you find that using a function was actually faster than just calling removeAll from the client? I think removeAll from the client should send your keys in a single message, similar to your function

[jira] [Updated] (GEODE-2854) GEODE REST API Docs Missing Authentication

2017-05-01 Thread Michael Martell (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2854?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Martell updated GEODE-2854: --- Description: The REST API docs don't document how to do authentication. Looks like HEAD param

[jira] [Commented] (GEODE-2848) While destroying a LuceneIndex, the AsyncEventQueue region is destroyed in remote members before stopping the AsyncEventQueue

2017-05-01 Thread xiaojian zhou (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991583#comment-15991583 ] xiaojian zhou commented on GEODE-2848: -- I think it does not worth to introduce comple

Re: Review Request 58891: GEODE-2837: Start server creates specified working dir if necessary

2017-05-01 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58891/#review173513 --- According to the documentation on this --dir: Specify the director

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173512 --- For all the RegionEntry subclasses do the following: * Do not mod

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173501 --- geode-core/src/main/java/org/apache/geode/cache/query/internal/in

Re: Review Request 58682: GEODE-2662: Missing keys cause columns to shift in gfsh table display.

2017-05-01 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58682/#review173509 --- Ship it! Ship It! - Jared Stewart On April 27, 2017, 6:30 p.

Review Request 58891: GEODE-2837: Start server creates specified working dir if necessary

2017-05-01 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58891/ --- Review request for geode, Jinmei Liao, Ken Howe, Kirk Lund, and Patrick Rhomberg

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991479#comment-15991479 ] ASF GitHub Bot commented on GEODE-2828: --- Github user jhuynh1 commented on a diff in

[GitHub] geode pull request #481: GEODE-2828: AEQ being created before the user regio...

2017-05-01 Thread jhuynh1
Github user jhuynh1 commented on a diff in the pull request: https://github.com/apache/geode/pull/481#discussion_r114195735 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/AbstractPartitionedRepositoryManager.java --- @@ -111,6 +121,10 @@ protected Inde

Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/ --- Review request for geode, Darrel Schneider, Jinmei Liao, Jared Stewart, Ken Howe

Remove keys from a PartitionedRegion

2017-05-01 Thread Goutam Tadi
Hi Team, With +Bradford D Boyle We are trying to remove a set of keys from a `PartitionedRegion`. Currently, we execute a function with `onRegion()`. Inside the function, we call `PartitionRegionHelper.getLocalPrimaryData()` and use the returned region to execute `region.removeAll(keys)`. The p

Re: Build failed in Jenkins: Geode-nightly #822

2017-05-01 Thread Udo Kohlmeyer
Agree, multiple repos does introduce a higher amount of management for a release. So... this is a discussion for the future then. --Udo On 5/1/17 11:33, Dan Smith wrote: I've usually seen this happen when one test logs a bunch of stuff, and jenkins runs out memory trying to process that log.

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991292#comment-15991292 ] ASF GitHub Bot commented on GEODE-2828: --- Github user nabarunnag commented on a diff

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991294#comment-15991294 ] ASF GitHub Bot commented on GEODE-2828: --- Github user nabarunnag commented on a diff

[GitHub] geode pull request #481: GEODE-2828: AEQ being created before the user regio...

2017-05-01 Thread nabarunnag
Github user nabarunnag commented on a diff in the pull request: https://github.com/apache/geode/pull/481#discussion_r114175681 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegion.java --- @@ -78,17 +87,9 @@ protected Reposito

[GitHub] geode pull request #481: GEODE-2828: AEQ being created before the user regio...

2017-05-01 Thread nabarunnag
Github user nabarunnag commented on a diff in the pull request: https://github.com/apache/geode/pull/481#discussion_r114175765 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneIndexImpl.java --- @@ -131,30 +131,41 @@ protected void initialize() {

Re: Build failed in Jenkins: Geode-nightly #822

2017-05-01 Thread Dan Smith
I've usually seen this happen when one test logs a bunch of stuff, and jenkins runs out memory trying to process that log. I don't think we should reorganize repos because of jenkins issues - even if the # of tests were an issue we could just create multiple jobs. I'm not quite sure I see the valu

[jira] [Commented] (GEODE-2857) Add getProcess to ProcessWrapper

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2857?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991251#comment-15991251 ] ASF subversion and git services commented on GEODE-2857: Commit c2

[jira] [Created] (GEODE-2857) Add getProcess to ProcessWrapper

2017-05-01 Thread Kirk Lund (JIRA)
Kirk Lund created GEODE-2857: Summary: Add getProcess to ProcessWrapper Key: GEODE-2857 URL: https://issues.apache.org/jira/browse/GEODE-2857 Project: Geode Issue Type: Wish Components:

[jira] [Assigned] (GEODE-2857) Add getProcess to ProcessWrapper

2017-05-01 Thread Kirk Lund (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund reassigned GEODE-2857: Assignee: Kirk Lund > Add getProcess to ProcessWrapper > > >

[jira] [Resolved] (GEODE-2856) CacheServerLauncherDUnitTest.testRebalance intermittently fails assertion for number of buckets

2017-05-01 Thread Kirk Lund (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2856?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund resolved GEODE-2856. -- Resolution: Won't Fix Test does not exist > CacheServerLauncherDUnitTest.testRebalance intermittently f

[jira] [Closed] (GEODE-2856) CacheServerLauncherDUnitTest.testRebalance intermittently fails assertion for number of buckets

2017-05-01 Thread Kirk Lund (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2856?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund closed GEODE-2856. > CacheServerLauncherDUnitTest.testRebalance intermittently fails assertion for > number of buckets > -

[jira] [Commented] (GEODE-2554) Geode incubator docs are still up

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2554?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991240#comment-15991240 ] ASF GitHub Bot commented on GEODE-2554: --- Github user sboorlagadda commented on the i

[GitHub] geode-examples issue #6: Fix broken links to geode docs.

2017-05-01 Thread sboorlagadda
Github user sboorlagadda commented on the issue: https://github.com/apache/geode-examples/pull/6 [GEODE-2554](https://issues.apache.org/jira/browse/GEODE-2554) addresses to fix the doc references issue and we would not need this fix. --- If your project is set up for it, you can repl

[GitHub] geode-examples pull request #6: Fix broken links to geode docs.

2017-05-01 Thread sboorlagadda
Github user sboorlagadda closed the pull request at: https://github.com/apache/geode-examples/pull/6 --- 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 fea

[jira] [Created] (GEODE-2856) CacheServerLauncherDUnitTest.testRebalance intermittently fails assertion for number of buckets

2017-05-01 Thread Kirk Lund (JIRA)
Kirk Lund created GEODE-2856: Summary: CacheServerLauncherDUnitTest.testRebalance intermittently fails assertion for number of buckets Key: GEODE-2856 URL: https://issues.apache.org/jira/browse/GEODE-2856

[jira] [Reopened] (GEODE-1988) CI failure: RegisterInterestKeysPRDUnitTest fails intermittently

2017-05-01 Thread xiaojian zhou (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1988?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] xiaojian zhou reopened GEODE-1988: -- It is reproduced in CI FlakeyTest #569 and #564 org.apache.geode.internal.cache.wan.parallel.Parall

Re: Build failed in Jenkins: Geode-nightly #822

2017-05-01 Thread Udo Kohlmeyer
Am I reading this correctly... Do we need to provide more memory to gradle to complete the steps to publish the JUnit results? Is this a problem that we would constantly face, as we add more and more "modules" to the Geode code base? Should we maybe see this a chance to move some of the code

Build failed in Jenkins: Geode-nightly #822

2017-05-01 Thread Apache Jenkins Server
See -- [...truncated 112.80 KB...] :geode-lucene:compileTestJavaNote: Some input files use or override a deprecated API. Note: Recompile with -Xlint:deprecation for details. Note: Some input

Re: Review Request 58849: GEODE-2840: add a DUnit test to test concurrent deploy

2017-05-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58849/#review173482 --- Fix it, then Ship it! Fix it and ship it! geode-core/src/test

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991185#comment-15991185 ] ASF GitHub Bot commented on GEODE-2828: --- Github user nabarunnag commented on a diff

Re: Review Request 58682: GEODE-2662: Missing keys cause columns to shift in gfsh table display.

2017-05-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58682/#review173481 --- Ship it! Ship It! - Kirk Lund On April 27, 2017, 6:30 p.m.,

[GitHub] geode pull request #481: GEODE-2828: AEQ being created before the user regio...

2017-05-01 Thread nabarunnag
Github user nabarunnag commented on a diff in the pull request: https://github.com/apache/geode/pull/481#discussion_r114164780 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/AbstractPartitionedRepositoryManager.java --- @@ -111,6 +121,10 @@ protected I

Re: Review Request 58853: GEODE-2847: Get correct version tags for retried bulk operations

2017-05-01 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58853/#review173479 --- geode-core/src/main/java/org/apache/geode/internal/cache/LocalReg

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991155#comment-15991155 ] ASF GitHub Bot commented on GEODE-2828: --- Github user jhuynh1 commented on a diff in

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991154#comment-15991154 ] ASF GitHub Bot commented on GEODE-2828: --- Github user jhuynh1 commented on a diff in

[GitHub] geode pull request #481: GEODE-2828: AEQ being created before the user regio...

2017-05-01 Thread jhuynh1
Github user jhuynh1 commented on a diff in the pull request: https://github.com/apache/geode/pull/481#discussion_r114161620 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/AbstractPartitionedRepositoryManager.java --- @@ -111,6 +121,10 @@ protected Inde

[GitHub] geode pull request #481: GEODE-2828: AEQ being created before the user regio...

2017-05-01 Thread jhuynh1
Github user jhuynh1 commented on a diff in the pull request: https://github.com/apache/geode/pull/481#discussion_r114160633 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/AbstractPartitionedRepositoryManager.java --- @@ -47,18 +48,22 @@ new

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991140#comment-15991140 ] ASF GitHub Bot commented on GEODE-2828: --- Github user upthewaterspout commented on a

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991141#comment-15991141 ] ASF GitHub Bot commented on GEODE-2828: --- Github user upthewaterspout commented on a

[GitHub] geode pull request #481: GEODE-2828: AEQ being created before the user regio...

2017-05-01 Thread upthewaterspout
Github user upthewaterspout commented on a diff in the pull request: https://github.com/apache/geode/pull/481#discussion_r114158872 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegion.java --- @@ -78,17 +87,9 @@ protected Rep

[GitHub] geode pull request #481: GEODE-2828: AEQ being created before the user regio...

2017-05-01 Thread upthewaterspout
Github user upthewaterspout commented on a diff in the pull request: https://github.com/apache/geode/pull/481#discussion_r114158965 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneIndexImpl.java --- @@ -131,30 +131,41 @@ protected void initialize(

[jira] [Commented] (GEODE-2778) gfsh help now uses log4j log levels: update user guide

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991131#comment-15991131 ] ASF GitHub Bot commented on GEODE-2778: --- GitHub user davebarnes97 opened a pull requ

[GitHub] geode pull request #484: GEODE-2778: gfsh help now uses log4j log levels: up...

2017-05-01 Thread davebarnes97
GitHub user davebarnes97 opened a pull request: https://github.com/apache/geode/pull/484 GEODE-2778: gfsh help now uses log4j log levels: update user guide Confined changes to gfsh commands, only. Further changes may be needed for xml-based configs. You can merge this pull reque

[jira] [Commented] (GEODE-2632) Integrated Security performance improvements

2017-05-01 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991128#comment-15991128 ] ASF subversion and git services commented on GEODE-2632: Commit 73

[jira] [Commented] (GEODE-2778) gfsh help now uses log4j log levels: update user guide

2017-05-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15991112#comment-15991112 ] ASF GitHub Bot commented on GEODE-2778: --- Github user davebarnes97 closed the pull re

[GitHub] geode pull request #482: gfsh help now uses log4j log levels: update user gu...

2017-05-01 Thread davebarnes97
Github user davebarnes97 closed the pull request at: https://github.com/apache/geode/pull/482 --- 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

[GitHub] geode issue #482: gfsh help now uses log4j log levels: update user guide

2017-05-01 Thread davebarnes97
Github user davebarnes97 commented on the issue: https://github.com/apache/geode/pull/482 PR associated itself with a secondary ticket. Will re-post with an improved comment. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[jira] [Resolved] (GEODE-2381) Make enums not get so mangled by Spotless

2017-05-01 Thread Galen O'Sullivan (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Galen O'Sullivan resolved GEODE-2381. - Resolution: Fixed Assignee: Galen O'Sullivan Fix Version/s: 1.2.0 > Make e

[jira] [Created] (GEODE-2855) Document Rename Execution.withArgs to Execution.setArguments

2017-05-01 Thread Joey McAllister (JIRA)
Joey McAllister created GEODE-2855: -- Summary: Document Rename Execution.withArgs to Execution.setArguments Key: GEODE-2855 URL: https://issues.apache.org/jira/browse/GEODE-2855 Project: Geode

[jira] [Created] (GEODE-2854) GEODE REST API Docs Missing Authentication

2017-05-01 Thread Michael Martell (JIRA)
Michael Martell created GEODE-2854: -- Summary: GEODE REST API Docs Missing Authentication Key: GEODE-2854 URL: https://issues.apache.org/jira/browse/GEODE-2854 Project: Geode Issue Type: Task

[jira] [Resolved] (GEODE-958) AgentUtilJUnitTest.testRESTApiExists failed with java.lang.AssertionError: GemFire REST API WAR File was not found

2017-05-01 Thread Jared Stewart (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-958?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jared Stewart resolved GEODE-958. - Resolution: Fixed > AgentUtilJUnitTest.testRESTApiExists failed with java.lang.AssertionError: > G

  1   2   >