ctubbsii commented on issue #3348: URL: https://github.com/apache/accumulo/issues/3348#issuecomment-1530315463
One thing to keep in mind, that may make this easier to address: SplitUtils is not public API and is not intended for direct consumption. It is used internally to help us approximate relative split sizes when calculating InputSplits. So, it doesn't matter if its method returns a negative number, as long as the places where it's used check its sign to ensure that they handle that situation appropriately. We could also try to come up with better approximation methods for split sizes, but I think addressing the negative in the places where it's used is the quickest and easiest way to fix this, without completely rewriting the approximation algorithm. -- 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]
