Skip to content

[hotfix][build] Update Python CI Java version to fix CI #199

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hlteoh37
Copy link
Contributor

Purpose of the change

Set up the Java version for Python CI to >= JDK 11.

Verifying this change

This change is already covered by existing tests (Existing CI)

Significant changes

(Please check any boxes [x] if the answer is "yes". You can first publish the PR and check them afterwards, for convenience.)

  • Dependencies have been added or upgraded
  • Public API has been changed (Public API is any class annotated with @Public(Evolving))
  • Serializers have been changed
  • New feature has been introduced
    • If yes, how is this documented? (not applicable / docs / JavaDocs / not documented)

Copy link
Contributor

@leekeiabstraction leekeiabstraction left a comment

Choose a reason for hiding this comment

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

Thank you for the PR!

@leekeiabstraction
Copy link
Contributor

Looks like we're past the bytecode version error now. Are the new errors due to test setup or stale test cases or actual regression?

FAILED pyflink/datastream/connectors/tests/test_kinesis.py::FlinkKinesisTest::test_kinesis_firehose_sink - py4j.protocol.Py4JError: org.apache.flink.connector.firehose.sink.KinesisFirehoseSink does not exist in the JVM
FAILED pyflink/datastream/connectors/tests/test_kinesis.py::FlinkKinesisTest::test_kinesis_source - py4j.protocol.Py4JError: org.apache.flink.streaming.connectors.kinesis.FlinkKinesisConsumer does not exist in the JVM
FAILED pyflink/datastream/connectors/tests/test_kinesis.py::FlinkKinesisTest::test_kinesis_streams_sink - py4j.protocol.Py4JError: org.apache.flink.connector.kinesis.sink.KinesisStreamsSink does not exist in the JVM

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.

2 participants