You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "dlmarion (via GitHub)" <gi...@apache.org> on 2023/11/01 20:58:33 UTC
[PR] Removed TableInfo.majors field [accumulo]
dlmarion opened a new pull request, #3921:
URL: https://github.com/apache/accumulo/pull/3921
Fixes #3609
--
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
Re: [PR] Removed TableInfo.majors field [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on PR #3921:
URL: https://github.com/apache/accumulo/pull/3921#issuecomment-1789678304
Before this change, the Monitor would likely show `0(0)` for major compactions on the tserver page. I'm not sure what it will show now, it could even be broken. However, we might need to re-imagine how the Monitor is going to work anyway.
--
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
Re: [PR] Removed TableInfo.majors field [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on PR #3921:
URL: https://github.com/apache/accumulo/pull/3921#issuecomment-1790996383
> Thinking before this and its issue is closed out, need to open a new issue to decide what do with the column in the monitor.
Do you think this is covered by #3608?
--
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
Re: [PR] Removed TableInfo.majors field [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion merged PR #3921:
URL: https://github.com/apache/accumulo/pull/3921
--
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
Re: [PR] Removed TableInfo.majors field [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on PR #3921:
URL: https://github.com/apache/accumulo/pull/3921#issuecomment-1791015454
> Do you think this is covered by https://github.com/apache/accumulo/issues/3608?
I added a comment to the issue about what still needs to be done.
--
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
Re: [PR] Removed TableInfo.majors field [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on PR #3921:
URL: https://github.com/apache/accumulo/pull/3921#issuecomment-1790930637
> Before this change, the Monitor would likely show 0(0) for major compactions on the tserver page. I'm not sure what it will show now, it could even be broken. However, we might need to re-imagine how the Monitor is going to work anyway.
Interesting, we do need to decide what we want to do with that column in monitor. We could still gather per table information on queued and running compactions, but it would be done in a very different way that in used to. May need a follow on issue for that.
--
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