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