Skip to content

[FLINK-34250][docs] add new option for avro format docs#24649

Merged
PatrickRen merged 1 commit intoapache:masterfrom
HuangZhenQiu:FLINK-34250-avro-docs
Apr 11, 2024
Merged

[FLINK-34250][docs] add new option for avro format docs#24649
PatrickRen merged 1 commit intoapache:masterfrom
HuangZhenQiu:FLINK-34250-avro-docs

Conversation

@HuangZhenQiu
Copy link
Copy Markdown
Contributor

What is the purpose of the change

Add the missing new Avro option in corresponding doc.

Brief change log

  • Add new option in avro.md

Verifying this change

This change is a trivial doc change without any test coverage.

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
  • The serializers: (no)
  • The runtime per-record code paths (performance sensitive): (no)
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
  • The S3 file system connector: (no)

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (docs)

@flinkbot
Copy link
Copy Markdown
Collaborator

flinkbot commented Apr 10, 2024

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

Copy link
Copy Markdown
Contributor

@PatrickRen PatrickRen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@HuangZhenQiu Thanks for the patch! LGTM.

Could you also add this option to the Chinese doc at docs/content.zh/docs/connectors/table/formats/avro.md?

@HuangZhenQiu HuangZhenQiu force-pushed the FLINK-34250-avro-docs branch from 2e53fff to 2328332 Compare April 10, 2024 15:00
@HuangZhenQiu
Copy link
Copy Markdown
Contributor Author

@HuangZhenQiu Thanks for the patch! LGTM.

Could you also add this option to the Chinese doc at docs/content.zh/docs/connectors/table/formats/avro.md?

Thanks for pointing it out. Updated accordingly.

Copy link
Copy Markdown
Contributor

@PatrickRen PatrickRen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@HuangZhenQiu Thanks for the update! LGTM

@PatrickRen PatrickRen merged commit 722fb9b into apache:master Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants