You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/03/30 14:24:01 UTC

[GitHub] [accumulo] milleruntime opened a new pull request #2600: Create IteratorBuilder to expose potential bugs

milleruntime opened a new pull request #2600:
URL: https://github.com/apache/accumulo/pull/2600


   * Create builder classes for iterator stack creation
   * Replace IterLoad class with new IteratorBuilder and refactor classes
   that use it to determine whether or not there is a bug in iterator code


-- 
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



[GitHub] [accumulo] milleruntime commented on pull request #2600: Create IteratorBuilder to expose potential bugs

Posted by GitBox <gi...@apache.org>.
milleruntime commented on pull request #2600:
URL: https://github.com/apache/accumulo/pull/2600#issuecomment-1083654416


   > > Replace IterLoad class with new IteratorBuilder and refactor classes that use it to determine whether or not there is a bug in iterator code
   > 
   > @milleruntime how do these changes help find a bug?
   
   I was working on these changes when I thought there was a bug (see #2601). But it turns out that it is not a bug so I can removed the validation check I added to the `IteratorBuilder`. This check was throwing an error is some of the tests.


-- 
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



[GitHub] [accumulo] milleruntime commented on pull request #2600: Create IteratorBuilder to expose potential bugs

Posted by GitBox <gi...@apache.org>.
milleruntime commented on pull request #2600:
URL: https://github.com/apache/accumulo/pull/2600#issuecomment-1083421410


   These changes will depend on the outcome of https://github.com/apache/accumulo/pull/2601


-- 
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



[GitHub] [accumulo] keith-turner commented on pull request #2600: Create IteratorBuilder to expose potential bugs

Posted by GitBox <gi...@apache.org>.
keith-turner commented on pull request #2600:
URL: https://github.com/apache/accumulo/pull/2600#issuecomment-1083587271


   > Replace IterLoad class with new IteratorBuilder and refactor classes that use it to determine whether or not there is a bug in iterator code
   
   @milleruntime  how do these changes help find a bug?


-- 
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