This is an automated email from the ASF dual-hosted git repository.

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new cc8362fe60 Cleanup: Modify log message to indicate failure reasons 
when creating data directory on the server. (#9145)
cc8362fe60 is described below

commit cc8362fe6033b11b5b83fa1c509ea556f6ee1c25
Author: Mayank Shrivastava <maya...@apache.org>
AuthorDate: Mon Aug 1 22:58:32 2022 -0700

    Cleanup: Modify log message to indicate failure reasons when creating data 
directory on the server. (#9145)
---
 .../org/apache/pinot/core/data/manager/BaseTableDataManager.java   | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git 
a/pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java
 
b/pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java
index 03c01083f8..bcb21234c4 100644
--- 
a/pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java
+++ 
b/pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java
@@ -122,7 +122,9 @@ public abstract class BaseTableDataManager implements 
TableDataManager {
     _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
@@ -130,7 +132,8 @@ public abstract class BaseTableDataManager implements 
TableDataManager {
     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. "
+              + "Please check for available space and write-permissions for 
this directory.",
           _resourceTmpDir);
     }
     _errorCache = errorCache;


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to