apache / apache/parquet-java

Deprecate Log and move to SLF4J Logger

Offen
#1,460 8 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Component: Java Component: Parquet Priority: Major Type: bug
Vorherrschende Sprache
Java
Sterne
3.1k
Forks
1.6k
Ø Merge
3 T. 12 Std.
Gemergte PRs (30 T.)
33

Beschreibung

The current Log class is intended to allow swapping out logger back-ends, but SLF4J already does this. It also doesn't expose as nice of an API as SLF4J, which can handle formatting to avoid the cost of building log messages that won't be used. I think we should deprecate the org.apache.parquet.Log class and move to using SLF4J directly, instead of wrapping SLF4J (PARQUET-305).

This will require deprecating the current Log class and replacing the current uses of it with SLF4J.

**Reporter**: [Ryan Blue](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=rdblue) / @rdblue
#### Related issues:
- [Fix flush() for RecordConsumer and implementations](https://github.com/apache/parquet-java/issues/1941) (is blocked by)
- [Avoid evoking job.toString() in ParquetLoader](https://github.com/apache/parquet-java/issues/1942) (is blocked by)
- [Shading SLF4J prevents SLF4J locating org.slf4j.impl.StaticLoggerBinder](https://github.com/apache/parquet-format/issues/302) (is related to)
- [Logger instantiated for package org.apache.parquet may be GC-ed](https://github.com/apache/parquet-java/issues/1830) (is related to)
- [Shutdown hook in parquet-avro library corrupts data and disables logging](https://github.com/apache/parquet-java/issues/1916) (is required by)

**Note**: *This issue was originally created as [PARQUET-401](https://issues.apache.org/jira/browse/PARQUET-401). Please see the [migration documentation](https://issues.apache.org/jira/browse/PARQUET-2502) for further details.*

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne mit der Klasse org.apache.parquet.Log und ermittle ihre derzeitige Verwendung im gesamten Projekt. Lies die verlinkte Migrationsdokumentation, bevor du den Logging-Ansatz änderst. Als erledigt gilt die Aufgabe, wenn die Klasse Log als veraltet gekennzeichnet ist und die derzeitigen Verwendungen direkt durch SLF4J ersetzt wurden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
backend
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.