FasterXML / FasterXML/jackson-dataformats-binary

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

Open
#294 0 comments 0 reactions 0 assignees View on GitHub
3.2 avro
Dominant language
Java
Stars
347
Forks
156
Avg merge
3d 3h
Merged PRs (30d)
22

Description

`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?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.