keith-turner commented on issue #4089:
URL: https://github.com/apache/accumulo/issues/4089#issuecomment-1864880535

   Discussed this with @ctubbsii and we think the following is a good way 
forward.
   
    1. Drop the property in 3.x
    2. Update the documentation in 2.x to explain that the property is only 
used by deprecated code. Could also deprecate the property in 2.x.  The 
property should have been deprecated for 2.1.0 since its only used by 
deprecated code, it was just missed. Document that the property was very 
inefficient and that compaction ratio is the best way to handle the number of 
files on a per table basis and mention it was used by merging minor compactions.
    3. Modify the DefaultCompactionStrategy to log a warning when it performs a 
sub optimal compaction based on table.files.max
    4. Make the new compaction planning code log a warning when no compactions 
are scheduled for a tablet that has more than tserver.scan.files.open.max files.


-- 
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]

Reply via email to