[I] Bad usage of ample filter [accumulo]

["DomGarguilo (via GitHub)" <gi...@apache.org>]

DomGarguilo opened a new issue, #4138:
URL: https://github.com/apache/accumulo/issues/4138

   **Describe the bug**
   Filtering was added to ample in #3968. There is logic to prevent `TabletsMetadata.filter()` and `TabletsMetadata.checkConsistency()` from being used in the same call. There is a spot in the code where both are used which will cause an exception to be thrown:
   
   https://github.com/apache/accumulo/blob/7e8f9d3b4a1a2a53d1d880623e01de6a16aac63b/server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectWriteAheadLogs.java#L80-L93
   
   And the check:
   
   https://github.com/apache/accumulo/blob/7e8f9d3b4a1a2a53d1d880623e01de6a16aac63b/core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletsMetadata.java#L125-L134
   
   **Versions (OS, Maven, Java, and others, as appropriate):**
    - Affected version(s) of this project: elasticity
   
   We need to remove either `filter` or `checkConsistency` in this instance.


-- 
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, e-mail: notifications-unsubscribe@accumulo.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Bad usage of ample filter [accumulo]

["DomGarguilo (via GitHub)" <gi...@apache.org>]

DomGarguilo closed issue #4138: Bad usage of ample filter
URL: https://github.com/apache/accumulo/issues/4138


-- 
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, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Bad usage of ample filter [accumulo]

["DomGarguilo (via GitHub)" <gi...@apache.org>]

DomGarguilo commented on issue #4138:
URL: https://github.com/apache/accumulo/issues/4138#issuecomment-1883817840

   Fixed via #4140


-- 
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, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org