[GitHub] [accumulo] dlmarion commented on pull request #3393: Set hosting goals to ALWAYS in getDiskUsage test

["dlmarion (via GitHub)" <gi...@apache.org>]

dlmarion commented on PR #3393:
URL: https://github.com/apache/accumulo/pull/3393#issuecomment-1544516619

   We did not fix the ITs when merging the OnDemand changes because we thought that it made sense that they continue to fail until we moved the tablet management code from the TabletServer to the Manager. For getting disk usage, `TableDiskUsage.getDiskUsage` scans either the `Root` or `Metadata` tables, both of which should always be hosted. While this fix makes the test pass, I'm not sure that it's the right one. Do you know where in the codepath it's waiting for the tables to be hosted? I think that's what we want to fix.


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