-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
FLINK-36479 remove deprecated method Table getSchema as #25540
base: master
Are you sure you want to change the base?
FLINK-36479 remove deprecated method Table getSchema as #25540
Conversation
/** | ||
* Renames the fields of the expression result. Use this to disambiguate fields before joining | ||
* to operations. | ||
* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see a test that uses this function in Scala in OverWindowValidationTest is has .
Over.partitionBy($"c").orderBy($"rowtime").preceding(2).following($"xx").as($"w"))
Hi, @jbalchan can you take a look at the ci failure? |
1355727
to
e0aa873
Compare
@flinkbot run azure |
@flinkbot run azure |
Hi, @jbalchan any update? If you're busy, I'll take charge of this task and add you as co-author since the code freeze date for flink 2.0.0 is approaching... |
Hi @xuyangzhong, I'm struggling to fix the scala test cases for this. I'm stuck on it and might need more time. |
No worries, I'll continue handling it :) |
What is the purpose of the change
[FLINK-36479] (https://issues.apache.org/jira/browse/FLINK-36479)
Component: Table SQL / API
Remove deprecated methods Table getSchema and as
Brief change log
Remove deprecated methods Table getSchema and as
Verifying this change
This change is already covered by existing tests
Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: noDocumentation