GitHub user ameybarve15 opened a pull request:
https://github.com/apache/geode/pull/751
GEODE-2719 : Deprecated setTotalMaxMemory and getTotalMaxMemory and updated
their javadocs.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review
Github user ameybarve15 commented on a diff in the pull request:
https://github.com/apache/geode/pull/609#discussion_r132678804
--- Diff:
geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneQueriesIntegrationTest.java
---
@@ -331,6 +331,29 @@ public void
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode/pull/609
@upthewaterspout , @jhuynh1
I have updated PR, please review.
---
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
Github user ameybarve15 commented on a diff in the pull request:
https://github.com/apache/geode/pull/609#discussion_r131056489
--- Diff:
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/distributed/WaitUntilFlushedFunction.java
---
@@ -85,8 +72,10 @@ public void
Github user ameybarve15 commented on a diff in the pull request:
https://github.com/apache/geode/pull/609#discussion_r129480518
--- Diff:
geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneQueriesIntegrationTest.java
---
@@ -290,6 +295,24 @@ public void
GitHub user ameybarve15 opened a pull request:
https://github.com/apache/geode/pull/609
GEODE-2886 : sent IllegalStateException instead of throwing
IllegalArgumentException inside WaitUntilFlushedFunction
1. sent IllegalStateException instead of throwing IllegalArgumentException
GitHub user ameybarve15 opened a pull request:
https://github.com/apache/geode/pull/595
GEODE-3104 : Modified code to use --include-locators instead of existing
--shutdown-locators option in shutdown command.
GEODE-3104 : Modified code to use --include-locators instead of existing
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode/pull/568
LGTM
---
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
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode/pull/515
+1 LGTM
---
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
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode/pull/507
+1 Looks good , only fix conflicts.
---
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
GitHub user ameybarve15 opened a pull request:
https://github.com/apache/geode/pull/505
[GEODE-2890]: Corrected debug log location in
AbstractGatewaySenderEventProcessor.processQueue().
Corrected debug log location in
AbstractGatewaySenderEventProcessor.processQueue().
You can
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode-native/pull/90
Ok, but for this branch I have already merged develop, Let me know If you
can merge this request in this state or else I will create a new branch again
and follow the entire procedure again
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode-native/pull/90
Hi @echobravopapa , and @PivotalSarge Thanks for your review.
I am trying to merge my pull request but get this error.
>~/source/open/t/geode-native (develop) $ git fe
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode-native/pull/90
@echobravopapa I am following
https://cwiki.apache.org/confluence/display/GEODE/Code+contributions
I can squash commits post pull request is approved.
---
If your project is set up
GitHub user ameybarve15 opened a pull request:
https://github.com/apache/geode-native/pull/90
GEODE-2763: Remove of nonSingleHopsCount stat in client
GEODE-2763: Remove of nonSingleHopsCount stat in client from
1. source
2. cpp and csharp unit tests and
3
GitHub user ameybarve15 opened a pull request:
https://github.com/apache/geode/pull/332
GEODE-2049: Replaced unit of measure for all of the tcpfinalCheck
statistics from currently "nanoseconds" to be "messages".
You can merge this pull request into a Git r
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode/pull/323
These changes have been merged in commit id:
c33ce7db8148fe0b300066581f26582e3e044e96
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user ameybarve15 closed the pull request at:
https://github.com/apache/geode/pull/323
---
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
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode/pull/323
These changes have been merged in commit id:
c33ce7db8148fe0b300066581f26582e3e044e96
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user ameybarve15 closed the pull request at:
https://github.com/apache/geode/pull/314
---
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
Github user ameybarve15 commented on the issue:
https://github.com/apache/geode/pull/304
Is this PR ready for commit?
---
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
GitHub user ameybarve15 opened a pull request:
https://github.com/apache/incubator-geode/pull/292
[GEODE-1509] Fixed : Reduce the memory usage of GatewayEventCallbackArgument
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
22 matches
Mail list logo