keith-turner opened a new pull request, #5631: URL: https://github.com/apache/accumulo/pull/5631
In #5621 a bug was introduced where if an iterator seeks with range and columns families that are disjoint it will throw an exception. This was caused by Range.bound() that had this behavior. To fix this copied the code in Range.bound() into RangeImpl and made a very slight modification. This was done to preserve the existing behavior of the public API method Range.bound() and reuse 99% of its code and test for a slightly different behavior. Also documented the behavior of Range.bound(), considered changing its behavior but this subtle change in runtime behavior seemed dicey w/ hard to predict outcomes (like it would change the runtime behavior of Scanner and BatchScanner which currently use Range.bound()). Now when the range and column families are disjoint ColumnFamilySkippingIterator will seek its underlying iterator with a new empty range. This bug was found by BigRootTabletIT. After this change that test passes. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
