Jackie-Jiang commented on code in PR #10837:
URL: https://github.com/apache/pinot/pull/10837#discussion_r1214915685
##########
pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java:
##########
@@ -691,9 +691,10 @@ File getSegmentDataDir(String segmentName, @Nullable
String segmentTier, TableCo
File tierTableDataDir = new File(tierDataDir, _tableNameWithType);
return new File(tierTableDataDir, segmentName);
} catch (Exception e) {
- LOGGER.warn("Failed to get dataDir for segment: {} of table: {} on tier:
{} due to error: {}", segmentName,
- _tableNameWithType, segmentTier, e.getMessage());
- return getSegmentDataDir(segmentName);
+ File defaultDir = getSegmentDataDir(segmentName);
+ LOGGER.debug("Failed to get dataDir for segment: {} of table: {} on
tier: {} due to error: {}. Use default: {}",
Review Comment:
Is this a common exception? If this is unexpected, we shouldn't log in debug
level
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
