Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2706#discussion_r216884374
--- Diff:
core/src/main/java/org/apache/carbondata/core/memory/UnsafeMemoryManager.java
---
@@ -200,7 +200,7 @@ public static MemoryBlock allocateMemoryWithRetry(long
taskId, long size)
}
if (baseBlock == null) {
INSTANCE.printCurrentMemoryUsage();
- throw new MemoryException("Not enough memory");
+ throw new MemoryException("Not enough memory, increase
carbon.unsafe.working.memory.in.mb");
--- End diff --
I think you can optimize the error message to
`Not enough unsafe working memory (total: , available: , request: )`
---
