Ben-Zvi opened a new pull request #1859: DRILL-7170: Ignore uninitialized vector containers for OOM error messages URL: https://github.com/apache/drill/pull/1859 A Hash-Table OOM exception may happen while a vector container was allocated but not yet initialized (with _setRecordCount()_ ). This exception is caught and a detailed error message is generated, which involves calling _getRecordCount()_ on existing vector containers, hence could trigger an **IllegalStateException** on such uninitialized container. The fix: Ignore such uninitialized vector container (i.e., like record-count == 0) in the code that is used for generating that error message (currently _getActualSize()_ is only used for this purpose). Another code change: While testing on the Mac, the spilling code for regular file systems failed to delete leftover spilled directories (which were not empty). The code was changed to delete all the spill files first in a spill directory, and last the directory itself.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
