keith-turner commented on code in PR #5668:
URL: https://github.com/apache/accumulo/pull/5668#discussion_r2183256061
##########
server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java:
##########
@@ -488,9 +488,17 @@ private TableMgmtStats
manageTablets(Iterator<TabletManagement> iter,
if (mtiError != null) {
// An error happened on the TabletServer in the
TabletManagementIterator
// when trying to process this extent.
+ KeyExtent ke = null;
+ try {
+ ke = tm.getExtent();
+ } catch (IllegalStateException e) {
+ // thrown when no prev endrow in tablet metadata.
+ // suppress this exception and leave ke set to null
+ }
+ String errorLocation = ke == null ? " table: " + tm.getTableId() : "
extent: " + ke;
Review Comment:
On the server side there is now code in TabletManagementIter that handles
when TabletMetadata.convertRow throws an exception. For that case what happens
here on the client side in the manger, will tm be null? Wondering about the
call to `tm.getTableId()` for that case.
--
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]
