keith-turner commented on pull request #1614: URL: https://github.com/apache/accumulo/pull/1614#issuecomment-637796852
> Ah I see. If that is the case, I should probably add the check for max tablets on the server side as well. I could see how 1 tablet could easily get expanded beyond the limit. If its possible, then I think the best place to do it on the servers side would be `PrepBulkImport.checkForMerge()` because this is before any files have been loaded to any tablets. I am not sure if its possible though, because at that point I think the data is grouped by range like `Map<Range, List<File>>`. To easily do the check we need `Map<File, List<Range>>`. Another option if that does not work is when Range.TABLE is seen AND maxTablets > 0, then we actually try to resolve the range to tablets. However this kinda defeats the purpose of Range.TABLE as it is for the case when user has the most knowledge and Accumulo can do the least work in the fastest manner. Not sure what is best, just trying to think through the options. Need to look at the code some more too. ---------------------------------------------------------------- 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: [email protected]
