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

Update job jar location #15

Merged
merged 2 commits into from
Sep 6, 2024
Merged

Conversation

robobario
Copy link
Contributor

@robobario robobario commented Sep 6, 2024

  • Update to new job jar location
  • Update value.avro-confluent.schema-registry.url -> value.avro-confluent.url

streamshub/flink-sql#20 changed the job jar location to /opt/streamshub/

The job was also complaining about deprecated value.avro-confluent.schema-registry.url so I went and changed it.

The jar was moved in the runner image

Signed-off-by: Robert Young <[email protected]>
Flink was complaining it was deprecated, it looks like the
property naming changed in flink 1.14

Signed-off-by: Robert Young <[email protected]>
@showuon
Copy link
Contributor

showuon commented Sep 6, 2024

Could you explain more about this?

The job was also complaining about deprecated value.avro-confluent.schema-registry.url so I went and changed it.

The change doesn't have any problem. Just be interested in the change motivation. Thanks.

@robobario
Copy link
Contributor Author

When running the job prior to the change you see this in the logs multiple times:

2024-09-06 03:23:46,254 WARN org.apache.flink.configuration.Configuration [] - Config uses deprecated configuration key 'value.avro-confluent.schema-registry.url' instead of proper key 'value.avro-confluent.url'

Copy link
Contributor

@showuon showuon left a comment

Choose a reason for hiding this comment

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

Thanks for the explanation. LGTM!

@robobario robobario merged commit 1b14c11 into streamshub:main Sep 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants