[GitHub] [accumulo] cradal commented on issue #1618: Investigate TODO in exception message in new Bulk Import

[GitBox <gi...@apache.org>]

cradal commented on issue #1618:
URL: https://github.com/apache/accumulo/issues/1618#issuecomment-644151827


   @millerruntime I traced through the logic for mergeOverlappinh() in BulkImport. It appears that the only scenario in which a merge will occur is if the overlapping KeyExtent (oke) fits lexicographically within the the previousEndRow and endRow of the KeyExtent (ke). The function will throw the exception if either end of oke extends beyond ke. It does not seem like anyone intended the function to do anything else. Perhaps we should change the exception message to read something like this:
   "Unable to execute merge. Key extent " + oke.toString() +  " extends beyond the range of Key extent " + ke.toString() + ". Out of range data would be lost."


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

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