mayankshriv commented on code in PR #9145:
URL: https://github.com/apache/pinot/pull/9145#discussion_r935042746
##########
pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java:
##########
@@ -122,15 +122,16 @@ public void init(TableDataManagerConfig
tableDataManagerConfig, String instanceI
_indexDir = new File(_tableDataDir);
if (!_indexDir.exists()) {
Preconditions.checkState(_indexDir.mkdirs(),
- "Unable to create index directory at %s. Check that the user has
permissions on this directory.", _indexDir);
+ "Unable to create index directory at %s. Check for available space
and write-permissions for this directory.",
+ _indexDir);
}
_resourceTmpDir = new File(_indexDir, "tmp");
// This is meant to cleanup temp resources from TableDataManager. But
other code using this same
// directory will have those deleted as well.
FileUtils.deleteQuietly(_resourceTmpDir);
if (!_resourceTmpDir.exists()) {
Preconditions.checkState(_resourceTmpDir.mkdirs(),
- "Unable to create temp resources directory at %s. Check that the
user has permissions on this directory.",
+ "Unable to create temp resources directory at %s. Check for
available space and write-permissions for this directory.",
Review Comment:
I formatted it using IDE (and Pinot formatting style). It applied the 120
char limit to the first change, but not the second. Seems it is because it
doesn't want to split the String.
--
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
