renechoi commented on PR #5000:
URL: https://github.com/apache/zeppelin/pull/5000#issuecomment-3148572645
@Reamer Thank you for the review! I've updated the implementation based on
your suggestions:
Changes made:
- Removed the need for temporary files entirely by capturing the process
error stream directly using `IOUtils.toString(process.getErrorStream(),
StandardCharsets.UTF_8)`
- This eliminates the need for `File.createTempFile`, `FileInputStream`,
and `Files.deleteIfExists`
- Updated the unit tests to reflect the new implementation without temp
file handling
The solution is now simpler, more efficient, and completely eliminates the
resource leak risk by not using temporary files at all.
--
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: reviews-unsubscr...@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
