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.

Sign up for GitHub

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

Jump to bottom

Bump Apache Beam SDK version #42

Closed
pradithya opened this issue Jan 7, 2019 · 3 comments
Closed

Bump Apache Beam SDK version #42

pradithya opened this issue Jan 7, 2019 · 3 comments

Comments

@pradithya
Copy link
Collaborator

pradithya commented Jan 7, 2019 •
edited
Loading

Currently ingestion uses Apache Beam V2.5.0 which is compatible with Flink runner 1.4.x.
This version of flink doesn't have REST API to access metrics, e.g: /jobs/<jobid>/metrics which is added in Flink 1.5.x.
In order to use it we have to upgrade ingestion's BEAM SDK to either 2.8.0, 2.7.0 or 2.6.0
@zhilingc
Copy link
Collaborator

zhilingc commented Jan 7, 2019

+1

@tims
Copy link
Contributor

tims commented Jan 7, 2019

+1, I think we're up to 2.9.0 now. I actually started looking at this already, and there's some weird proto version compatibility issues that I think are easily addressed. will look again.

@pradithya pradithya mentioned this issue Jan 7, 2019
Merged
@tims
Copy link
Contributor

tims commented Jan 7, 2019

oops just saw you made a pr for this :)

Yanson pushed a commit to Yanson/feast that referenced this issue Jul 29, 2020
@PaulDFoster
Merge pull request feast-dev#42 from farfetch-external/enpolat/KE-754
0e92b49 
Long Term Automatic Test Scenarios
dmartinol pushed a commit to dmartinol/feast that referenced this issue Jul 12, 2024
@tmihalac
Merge pull request feast-dev#42 from tmihalac/add-client-auth-grpc
57ca900 
Client module-grpc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tims @pradithya @zhilingc