ddanielr commented on PR #4148: URL: https://github.com/apache/accumulo/pull/4148#issuecomment-1886259202
Overall the changes look good. The build failures seem to be caused by an issue with the mock `extent` object in CompactorTest. https://github.com/apache/accumulo/pull/4148/files#diff-34232683deeb79b4d648e68a75995a1f462256013c2d8b4c2a1f0e7a92e666ffR563-R564 The CompactionITs build fine. I checked out your branch and replaced the mock object with an actual tKeyExtent and the code passed, but threw me a warning. ``` WARNING: An illegal reflective access operation has occurred WARNING: Illegal reflective access by org.powermock.reflect.internal.WhiteboxImpl ``` -- 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]
