[PARQUET-305] Logger instantiated for package org.apache.parquet may be GC-ed

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.7.0
• Fix Version/s: 1.9.0, 1.8.2
• Component/s: parquet-mr
• Assignee: Ryan Blue
• Reporter: Cheng Lian
• Created: 09/Jun/15 06:53
• Updated: 23/Jun/24 03:27
• Resolved: 08/Dec/15 18:40

Description

This ticket is derived from SPARK-8122.

According to Javadoc of java.util.Logger:

It is important to note that the Logger returned by one of the getLogger factory methods may be garbage collected at any time if a strong reference to the Logger is not kept.

However, the only reference to the Logger created for package org.apache.parquet goes out of scope outside the static initialization block, and thus is possible to be garbage collected.

More details can be found in this comment.

Issue Links

• blocks: PARQUET-392 Release Parquet-mr 1.9.0
• is required by: PARQUET-395 System.out is used as logger in org.apache.parquet.Log
• relates to: PARQUET-401 Deprecate Log and move to SLF4J Logger
• relates to: PARQUET-412 Format: Do not shade slf4j-api
• relates to: SPARK-39156 Remove ParquetLogRedirector usage from ParquetFileFormat
• links to: PR #290