apache / apache/parquet-java

Deprecate Log and move to SLF4J Logger

Ouverte
#1,460 8 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Component: Java Component: Parquet Priority: Major Type: bug
Langage dominant
Java
Étoiles
3.1k
Forks
1.6k
Merge moyen
3 j 12 h
PR mergées (30 j)
33

Description

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.*

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par la classe org.apache.parquet.Log et recherchez ses utilisations actuelles dans l’ensemble du projet. Lisez la documentation de migration liée avant de modifier l’approche de logging. Le travail est terminé lorsque la classe Log est dépréciée et que les utilisations actuelles ont été remplacées directement par SLF4J.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
backend
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.