jmark99 commented on issue #2371: URL: https://github.com/apache/accumulo/issues/2371#issuecomment-982984779
I was looking at the code today. The value of 'r' will always be <= 1 so the while-loop should never execute more than one time per for-loop iteration. Switching the while-loop to an if-statement would not affect results while perhaps clearing up the code a small amount. Either way, the code accomplishes its intended purpose at the moment. As for this particular ticket, three options can be considered: * leaving correctly running code as is * updating the while-loop to an if-statement to remove a small amount of confusion * look into other methods of evenly parsing data as referenced above. Thoughts on this ticket in particular? -- 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]
