You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "keith-turner (via GitHub)" <gi...@apache.org> on 2023/07/11 22:48:37 UTC
[GitHub] [accumulo] keith-turner opened a new issue, #3594: Remove thrift field TabletStats.majors
keith-turner opened a new issue, #3594:
URL: https://github.com/apache/accumulo/issues/3594
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.
--
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: notifications-unsubscribe@accumulo.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] keith-turner commented on issue #3594: Remove thrift field TabletStats.majors
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on issue #3594:
URL: https://github.com/apache/accumulo/issues/3594#issuecomment-1636568144
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.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] keith-turner commented on issue #3594: Remove thrift field TabletStats.majors
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on issue #3594:
URL: https://github.com/apache/accumulo/issues/3594#issuecomment-1635062046
The field is no longer populated after #3604, but the field is still there and need to be remove from thrift and monitor code.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org