-
Notifications
You must be signed in to change notification settings - Fork 13.8k
Atlas changes #18415
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
Atlas changes #18415
Conversation
|
Thanks a lot for your contribution to the Apache Flink project. I'm the @flinkbot. I help the community Automated ChecksLast check on commit b6b9832 (Thu Jan 20 07:42:58 UTC 2022) Warnings:
Mention the bot in a comment to re-run the automated checks. Review Progress
Please see the Pull Request Review Guide for a full explanation of the review process. DetailsThe Bot is tracking the review progress through labels. Labels are applied according to the order of the review items. For consensus, approval by a Flink committer of PMC member is required Bot commandsThe @flinkbot bot supports the following commands:
|
|
Is this feature currently supported? How can I use it? Is there a Flink Atlas plugin available |
|
This PR is being marked as stale since it has not had any activity in the last 180 days. If you are having difficulty finding a reviewer, please reach out to the [community](https://flink.apache.org/what-is-flink/community/). If this PR is no longer valid or desired, please feel free to close it. If no activity occurs in the next 90 days, it will be automatically closed. |
|
This PR has been closed since it has not had any activity in 120 days. |
If Flink 1.10 is used, this JobListener is not used after it is created, so the Hook does not take effect. It can take effect after adding the following code.
JobListener listener = factory.createFromConfig(configuration, classLoader);
registerJobListener(listener);
If you use Flink 1.12, the option of execution.job-listeners has been officially supported, and the listeners have been added to env, so this PR is not required