Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/749
@ladyVader @jhuynh1 @upthewaterspout @boglesby
-- Potential reviewers.
---
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
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/749
GEODE-3502: Added awaitility clauses.
* Added awaitility clauses while waiting for the stat comparison checks
* Replaced the deprecated Wait Criterion with Awaitility clause
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/743
GEODE-3131: Increasing the time
* Increasing the time elapsed before the dispatchers are started.
* All validations must be done before the dispatcher start dispatching.
Thank
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/732
GEODE-3276: Managing race conditions while the senders are stopped
* When a connection is initialized, a readAckThread may be alive from a
previous incarnation.
* This AckThread
Github user nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/659
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/681
Potential reviewers
@ladyVader @jhuynh1 @boglesby @upthewaterspout @gesterzhou
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/681
Lucene refactoring and clean up
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/678
GEODE-3384: Javadoc change for PageableLuceneQueryResults
* Javadocs now mention that PageableLuceneQueryResults are serializable
and can be sent to other members
Thank you
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/674#discussion_r130654524
--- Diff:
geode-cq/src/test/java/org/apache/geode/internal/cache/PutAllCSDUnitTest.java
---
@@ -500,6 +502,81 @@ public void run2() throws CacheException
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/659#discussion_r129679647
--- Diff:
geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneSearchWithRollingUpgradeDUnit.java
---
@@ -0,0 +1,1044
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/659
* These tests were ported from RollingUpgradeDUnitTest and
RollingUpgrade2DUnitTest
* Instead of put and get verification, it is now lucent query result size
verification
Potential
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/659
GEODE-3308: Lucene rolling upgrade and backwards compatibility tests â¦
â¦added
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/628
Thank you @ladyVader .
Fixed the typo
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/628
Potential Reviewers:
@ladyVader @boglesby @jhuynh1 @upthewaterspout @gesterzhou @DivineEnder
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/628
GEODE-3131: Replaced Thread.sleep
* Replaced sleeps with an Awaitility clause which waits for the region
sizes to be correct before starting validation.
Thank you for submitting
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/625
Potential reviewers
@ladyVader @nabarunnag @boglesby @jhuynh1 @upthewaterspout @gesterzhou
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/622
Merging into develop
---
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
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/588
@jhuynh1 Changed the validation checks to verify that the size of the
indexes are correct.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/588
GEODE-2820: Added awaitlity clause to wait for the region size to be â¦
â¦correct
* Added awaitility clause to wait for all the regions to have all the
entries
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/574
@jhuynh1 , Last when we experimented with this function in WANTestBase
tests, the listener is destroyed once the vm is closed, which I believe that
happens at the end of each test.
---
If your
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/574
GEODE-3066: Test tweaks to stabilize it.
* Regions created before senders and receivers are started.
* Added a listener to slow down the puts so that when the remote region
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/562
@ladyVader @upthewaterspout @boglesby @gesterzhou
-- Adding more potential reviewers.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/562
GEODE-3025: GET operation before Lucene query
* To allow Lucene queries in single hop with transaction enabled mode,
the client must get the metadata about the regions.
* A Get
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/547
I will be merging this.
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/543
I will be merging this.
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/541
Working on merging this.
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/523
@DivineEnder
---
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 nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/523
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/526
Working on merging this change
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/523
Potential reviewers
@jhuynh1 @ladyVader @boglesby @gesterzhou @upthewaterspout
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/523
GEODE-2967: ResultCollection instead of StructCollection
* If we have one runtime iterator which is in case of self joins,
ResultSet or ResultBags are created
* Otherwise StructBag
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/517
Yes, we are opening a new ticket for eliminating the repeated code.
Analyzing the testcode for ORDER BY queries.
---
If your project is set up for it, you can reply to this email and have your
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/517
@ladyVader I am assuming that there should be tests already present testing
the functional aspect. One of the tests did fail and caught an issue in my fix.
@jhuynh1 what do you think
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/517
Potential Reviewers
@ladyVader @jhuynh1 @boglesby @upthewaterspout @gesterzhou
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/517
GEODE-2587: Refactored the OrderByComparator's compare method
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
Github user nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/516
---
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 nabarunnag opened a pull request:
https://github.com/apache/geode/pull/516
GEODE-2587: Refactored the OrderByComparator's compare method
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/514
GEODE-2905: Added extra logging to determine the cause of failure.
* Replaced assertTrue with assertEquals so that the commandResult is
displayed in case of an error.
Thank you
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/512
Potential reviewers
@ladyVader @upthewaterspout @jhuynh1 @boglesby @gesterzhou
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/512
GEODE-2637: Refactored LuceneQueryFactory.setResultLimit()
* Renamed LuceneQueryFactory.getResult to setLimit()
* This is done to match the cwiki documents
Thank you
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/503#discussion_r115629507
--- Diff:
geode-lucene/src/main/java/org/apache/geode/cache/lucene/LuceneResultStruct.java
---
@@ -39,6 +41,8 @@
* Return score the score
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/503#discussion_r115629448
--- Diff:
geode-lucene/src/main/java/org/apache/geode/cache/lucene/LuceneIndex.java ---
@@ -19,39 +19,62 @@
import
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/503#discussion_r115629459
--- Diff:
geode-lucene/src/main/java/org/apache/geode/cache/lucene/LuceneQuery.java ---
@@ -37,21 +42,38 @@
* Results are returned in order
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/503
GEODE-2907: Removed @Experimental tag from the Lucene module
* Removed the @experimental tag fromt the files in Lucene module.
* Improve on the javadocs for the interfaces present
Github user nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/492
---
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 nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/498
---
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 nabarunnag opened a pull request:
https://github.com/apache/geode/pull/499
GEODE-2879: Shutdown() called from close() in LonerDistributionManager
* LonerDistributionManager shutdown was not being called from close()
method call.
* This resulted in the thread
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/498
GEODE-2881: Wait for waitForFlushBeforeExecuteTextSearch to complete
* Test now waits for waitForFlushBeforeExecuteTextSearch initiated by
the test hook.
* The test hook gets called
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/492
GEODE-2754: Changed the name of unknown host
* Assert failures occured if the network contained machines named
"unknown" as the test function also named the unknown host a
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
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
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
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/481
GEODE-2828: AEQ being created before the user region
Potential Reviewers
@gesterzhou @jhuynh1 @upthewaterspout @ladyVader @boglesby
You can merge this pull request into a Git repository
Github user nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/479
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/479
Sending in a fresh pull request with all modification
---
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
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/479#discussion_r113630780
--- Diff:
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneEventListener.java
---
@@ -53,6 +54,8 @@
Logger logger
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/479#discussion_r113630711
--- Diff:
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneServiceImpl.java
---
@@ -166,28 +166,28 @@ public void createIndex
Github user nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/476
---
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 nabarunnag commented on the issue:
https://github.com/apache/geode/pull/479
Pushed to develop
Failure in Travis fixed in GEODE-2632
---
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
GitHub user nabarunnag reopened a pull request:
https://github.com/apache/geode/pull/479
GEM-1331: AEQ created before the user region
Potential reviewers
@gesterzhou @upthewaterspout @jhuynh1 @ladyVader
You can merge this pull request into a Git repository by running
Github user nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/479
---
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 nabarunnag opened a pull request:
https://github.com/apache/geode/pull/479
GEM-1331: AEQ created before the user region
Potential reviewers
@gesterzhou @upthewaterspout @jhuynh1 @ladyVader
You can merge this pull request into a Git repository by running:
$ git
Github user nabarunnag closed the pull request at:
https://github.com/apache/geode/pull/442
---
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 nabarunnag opened a pull request:
https://github.com/apache/geode/pull/476
GEODE-2816: Redundancy recovery inititiated even if redundancy set to 0
Potential reviewers
@upthewaterspout @jhuynh1 @gesterzhou @boglesby @ladyVader
You can merge this pull request
Github user nabarunnag commented on the issue:
https://github.com/apache/geode/pull/449
@jhuynh1
I have the changes you mentioned and also added a dunit test to confirm the
behaviour
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/451#discussion_r111272945
--- Diff:
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/distributed/LuceneQueryFunction.java
---
@@ -132,6 +133,25 @@ private
Github user nabarunnag commented on a diff in the pull request:
https://github.com/apache/geode/pull/449#discussion_r111037135
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/CreateIndexFunction.java
---
@@ -93,6 +91,31 @@ public void execute
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/449
GEODE-2764: Added checks on the region name
* Substring the region name till the last occurance of '.'
* Check if this substring is a valid region name
* Continue this till we
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/442
AEQ being created before region
@jhuynh1 Please do review , Is there any window where the redundancy
failure may occur?
You can merge this pull request into a Git repository by running
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/424
GEODE-2655: Added DUnit tests for lucene indexes on mixed objects.
* Lucene must be able to index on the common field name of mixed objects
* Lucene indexes must also work when
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/422
GEODE-2639: Added DUnit tests for expiration on Lucene
* Tests added to validate the effect of the expiration of Lucene Index
entries.
* Once the entries expire, the lucene index
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/420
GEODE-2635: Creating DUnit tests to test eviction in lucene
* DUnit tests for eviction with local destroy and overflow
* Refactored the integration tests for eviction
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/418
GEODE-2620: Rate stats removed from LuceneIndexMetrics
* Commit Rate, update rate and query rate removed from
LuceneIndexMetrics
* They can be added back again when the getRate method
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/416
GEODE-2618: PrimaryBucketExceptions handled in LuceneQueryFunction.exâ¦
â¦ecute
* PrimaryBucketException while executing a LuceneQuery will now be
caught and wrapped
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/415
GEODE-2604: Added javadoc comments to LuceneIndexMetrics.java
@upthewaterspout @jhuynh1 @gesterzhou @boglesby @ladyVader
You can merge this pull request into a Git repository by running
GitHub user nabarunnag opened a pull request:
https://github.com/apache/geode/pull/414
GEODE-2596: Lucene metrics moved to public API
* LuceneIndexMetrics and LuceneServiceMXBean were moved to
org.apache.geode.cache.lucene.management
-- Lucene precheckin
76 matches
Mail list logo
