FasterXML / FasterXML/jackson-dataformats-binary

[Avro] Rename `VisitorFormatWrapperImpl` to `AvroFormatVisitorWrapper` in 2.14 or 3.0

Aberta
#294 0 comentários 0 reações 0 responsáveis Ver no GitHub
3.2 avro
Linguagem predominante
Java
Estrelas
347
Forks
156
Merge médio
3d 3h
PRs com merge (30d)
22

Descrição

`VisitorFormatWrapperImpl` class name is unfortunate. It makes sense to rename it into something more descriptive, for example `AvroFormatVisitorWrapper`.

Implementation in 2.13:

public class VisitorFormatWrapperImpl
implements JsonFormatVisitorWrapper

Citation from #292 comment:
> Maybe it would make sense to also file a follow-up issue about actual renaming (of `VisitorFormatWrapperImpl` ) like you suggested first (to `AvroFormatVisitorWrapper`).
> I'd have to think if it actually could go in 2.14, after all, or at least in 3.0.

Question is, in which release?

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.