[PARQUET-395] System.out is used as logger in org.apache.parquet.Log

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.8.0
• Fix Version/s: 1.9.0

Environment

parquet-common-1.8.1.jar

• Assignee: Ryan Blue
• Reporter: Henrik Baastrup
• Created: 25/Nov/15 17:22
• Updated: 09/Dec/15 01:00
• Resolved: 09/Dec/15 01:00

Description

The use of System.out in the StreamHandler at line 62 in the org.apache.parquet.Log provoke that the java.util.logging.LogManager will close System.out in case the reset method is called on the LogManager.
This is special problematic when Shutdown Hooks are used in a project, as the LogManager set-up one, there call the reset method, and for this reason a race condition exist on System.out, if used in the project Shutdown Hook.
Other scenarios might also exist where a program call the LogManager reset method.

An eventual solution is to NOT use java.util.logging.Logger at all in the Parquet environment but use either log4j or slf4j as used in almost every other part in the Hadoop environment, this would also allow the user to control the logging much better than today.

Issue Links

• requires: PARQUET-305 Logger instantiated for package org.apache.parquet may be GC-ed