Re: [PR] checks the entire map, solves #3522 [accumulo]

dlmarion closed pull request #4175: checks the entire map, solves #3522 URL: https://github.com/apache/accumulo/pull/4175 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] checks the entire map, solves #3522 [accumulo]

dlmarion commented on PR #4175: URL: https://github.com/apache/accumulo/pull/4175#issuecomment-2104603308 Superceded by #4486 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] checks the entire map, solves #3522 [accumulo]

keith-turner commented on PR #4175: URL: https://github.com/apache/accumulo/pull/4175#issuecomment-1946415329 Now that #4227 is merged, it would be good to merge elasticity into this branch and then add unit test for this new code. -- This is an automated message from the Apache Git

Re: [PR] checks the entire map, solves #3522 [accumulo]

keith-turner commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1474726694 ## server/base/src/main/java/org/apache/accumulo/server/metadata/iterators/SetEqualityIterator.java: ## @@ -79,9 +83,24 @@ public void seek(Range range,

Re: [PR] checks the entire map, solves #3522 [accumulo]

keith-turner commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1474672362 ## server/base/src/main/java/org/apache/accumulo/server/metadata/iterators/SetEqualityIterator.java: ## @@ -83,14 +83,22 @@ public void seek(Range range,

Re: [PR] checks the entire map, solves #3522 [accumulo]

rsingh433 commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1474662748 ## server/base/src/main/java/org/apache/accumulo/server/metadata/iterators/SetEqualityIterator.java: ## @@ -82,6 +86,13 @@ public void seek(Range range, Collection

Re: [PR] checks the entire map, solves #3522 [accumulo]

rsingh433 commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1474656868 ## core/src/main/java/org/apache/accumulo/core/metadata/StoredTabletFile.java: ## @@ -55,6 +55,10 @@ public class StoredTabletFile extends AbstractTabletFile {

Re: [PR] checks the entire map, solves #3522 [accumulo]

keith-turner commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1474591110 ## core/src/main/java/org/apache/accumulo/core/metadata/StoredTabletFile.java: ## @@ -55,6 +55,10 @@ public class StoredTabletFile extends AbstractTabletFile {

Re: [PR] checks the entire map, solves #3522 [accumulo]

keith-turner commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1474560039 ## server/base/src/main/java/org/apache/accumulo/server/metadata/iterators/SetEqualityIterator.java: ## @@ -82,6 +86,13 @@ public void seek(Range range,

Re: [PR] checks the entire map, solves #3522 [accumulo]

keith-turner commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1469962970 ## server/base/src/main/java/org/apache/accumulo/server/metadata/iterators/SetEqualityIterator.java: ## @@ -186,4 +187,49 @@ public static Condition

Re: [PR] checks the entire map, solves #3522 [accumulo]

rsingh433 commented on PR #4175: URL: https://github.com/apache/accumulo/pull/4175#issuecomment-1909149589 > I don't think this is the correct interpretation of #3522. I think that issue is saying to modify the code

Re: [PR] checks the entire map, solves #3522 [accumulo]

rsingh433 commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1464117381 ## server/base/src/main/java/org/apache/accumulo/server/metadata/ConditionalTabletMutatorImpl.java: ## @@ -158,6 +161,14 @@ public Ample.ConditionalTabletMutator

Re: [PR] checks the entire map, solves #3522 [accumulo]

EdColeman commented on code in PR #4175: URL: https://github.com/apache/accumulo/pull/4175#discussion_r1463858048 ## server/base/src/main/java/org/apache/accumulo/server/metadata/ConditionalTabletMutatorImpl.java: ## @@ -158,6 +161,14 @@ public Ample.ConditionalTabletMutator

Re: [PR] checks the entire map, solves #3522 [accumulo]

dlmarion commented on PR #4175: URL: https://github.com/apache/accumulo/pull/4175#issuecomment-1904711259 I don't think this is the correct interpretation of #3522. I think that issue is saying to modify the code

[PR] checks the entire map, solves #3522 [accumulo]

rsingh433 opened a new pull request, #4175: URL: https://github.com/apache/accumulo/pull/4175 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,