[GitHub] [pinot] Jackie-Jiang commented on a diff in pull request #9145: Cleanup: Modify log message to indicate failure reasons when creating data directory on the server.

[GitBox]

Jackie-Jiang commented on code in PR #9145:
URL: https://github.com/apache/pinot/pull/9145#discussion_r935142585


##########
pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java:
##########
@@ -122,15 +122,18 @@ 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. "
+              + "Please 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."

Review Comment:
   ```suggestion
             "Unable to create temp resources directory at %s. "
   ```



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