Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
2.7.0
-
Reviewed
Description
Remove duplicate close for LogWriter in AppLogAggregatorImpl#uploadLogsForContainers
writer.close() was called twice in uploadLogsForContainers. One is called at the middle of the function and the other one is called at finally block. It will be better to just close writer once. Also LogWriter didn't implement java.io.Closeable interface. So we can't suppose LogWriter.close is idempotent.