search

apache / accumulo

Apache Accumulo
https://accumulo.apache.org
Apache License 2.0
1.07k stars 445 forks source link

Remove thrift field TabletStats.majors #3594

Open keith-turner opened 1 year ago

keith-turner commented 1 year ago

While working on #3592 noticed this thrift field will not longer be used as its was set in the tserver based on compaction activity in the tserver. The field seems to be used in the monitor.

keith-turner commented 1 year ago

The field is no longer populated after #3604, but the field is still there and need to be remove from thrift and monitor code.

keith-turner commented 1 year ago

The following comments are based on #3618

The TabletStats also has a split related field that is no longer used. This could be removed.

Also there is a class in the tablet server module named TabletStatsKeeper. This class does not do much anymore, need to evaluate if its still needed.