Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Task]: [Avro] Use "extensions/avro" instead of avro from"core" in Java SDK modules #24748

Closed
1 of 15 tasks
Tracked by #24292
aromanenko-dev opened this issue Dec 21, 2022 · 2 comments · Fixed by #24992
Closed
1 of 15 tasks
Tracked by #24292

Comments

@aromanenko-dev
Copy link
Contributor

What needs to happen?

Once Avro support is extracted into a dedicated module extensions/avro, then all Beam Java SDK modules have to switch to use it instead of avro-related classes from "core".

Affected packages:

  • org.apache.beam.examples
  • org.apache.beam.runners.dataflow
  • org.apache.beam.sdk.extensions.sql
  • org.apache.beam.sdk.io.gcp
  • org.apache.beam.sdk.io.kafka
  • org.apache.beam.sdk.io.parquet
  • org.apache.beam.sdk.nexmark

Issue Priority

Priority: 2 (default / most normal work should be filed as P2)

Issue Components

  • Component: Python SDK
  • Component: Java SDK
  • Component: Go SDK
  • Component: Typescript SDK
  • Component: IO connector
  • Component: Beam examples
  • Component: Beam playground
  • Component: Beam katas
  • Component: Website
  • Component: Spark Runner
  • Component: Flink Runner
  • Component: Samza Runner
  • Component: Twister2 Runner
  • Component: Hazelcast Jet Runner
  • Component: Google Cloud Dataflow Runner
@aromanenko-dev aromanenko-dev changed the title [Task]: Use "extensions/avro" instead of avro from"core" in Java SDK modules [Task]: [Avro] Use "extensions/avro" instead of avro from"core" in Java SDK modules Dec 28, 2022
@Abacn
Copy link
Contributor

Abacn commented Dec 29, 2022

Note that some changes related to avro in core happened after #24293 was in such as #24756. Need to synchronize the core avro and extension avro as part of this task.

@aromanenko-dev
Copy link
Contributor Author

@Abacn Thanks! I applied these changes on extensions/avro at #24854

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants