keith-turner commented on issue #3340:
URL: https://github.com/apache/accumulo/issues/3340#issuecomment-1591605275
@jmark99 those test look good to me. I think the test covers the case of
splitting a table where all of the goals are the same. Could add a test to
cover the case of splits in a table with mixed goals. Something like the
following.
1. create table
2. add split `m`
3. set goal ALWAYS on last tablet
4. add splits `d` and `s`
5. verify two tablets have ONDEMAND and two have ALWAYS goal
Or modify an existing test for this case. I don't think this has to be done
before closing this issue, I think the coverage is good in the test you already
added. The above was just something I thought of when looking at the test.
--
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]
