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

Add Flink application mode support #106

Open
regadas opened this issue Sep 16, 2021 · 6 comments
Open

Add Flink application mode support #106

regadas opened this issue Sep 16, 2021 · 6 comments

Comments

@regadas
Copy link
Contributor

regadas commented Sep 16, 2021

No description provided.

@hjwalt
Copy link
Contributor

hjwalt commented Sep 24, 2021

@regadas for clarification, is this an issue for supporting flink native kubernetes deployment? As per this documentation.

@regadas
Copy link
Contributor Author

regadas commented Sep 24, 2021

Right so the idea is to support https://nightlies.apache.org/flink/flink-docs-release-1.13/docs/deployment/overview/#application-mode as alternative way to the current way of submitting jobs: Per-job and session.

Adding Flink Kubernetes native is smth we can also have a look at.

@elanv
Copy link
Contributor

elanv commented Sep 24, 2021

It would be good to discuss ways to improve the API design when considering new features.

It would probably be nice to add a new CRD to support Flink-native k8s cluster. I think also implementing the application mode with new CRD will also help lower the complexity.

API improvements, for example:

  • Rename current FlinkCluster to FlinkSessionCluster
  • New CRD for native k8s Flink cluster: FlinkCluster
  • New CRDs for various job types: FlinkJob, BeamJob (spec includes FlinkCluster or FlinkSessionCluster template for dedicated use or link existing cluster as clusterName to share it). I think new FlinkJob could support various modes like application, per-job on native k8s Flink cluster.

There are existing issues that had been created. I hope you review it.

@elanv
Copy link
Contributor

elanv commented Sep 28, 2021

There is some interference between my PR and current controller. Therefore if new CRD is going to be added soon, it would be efficient to apply my PR there than reworking the PR.

@regadas In that sense, any specific work or plan for this issue? If not, I would like to proceed with this work first. How is it?
The basic idea is to keep the current FlinkCluster but create new v1beta2 FlinkJob CRD with the job routines extracted from it. If we need Flink k8s native cluster, we can leave the current FlinkCluster as it is and add a new CRD, and among them, we can select the Flink cluster CRD where FlinkJob will run.

@canghai118
Copy link

It would probably be nice to add a new CRD to support Flink-native k8s cluster. I think also implementing the application mode with new CRD will also help lower the complexity.

API improvements, for example:

  • Rename current FlinkCluster to FlinkSessionCluster
  • New CRD for native k8s Flink cluster: FlinkCluster
  • New CRDs for various job types: FlinkJob, BeamJob (spec includes FlinkCluster or FlinkSessionCluster template for dedicated use or link existing cluster as clusterName to share it). I think new FlinkJob could support various modes like application, per-job on native k8s Flink cluster.

Is there any progress or plan on this feature @elanv thanks

@haoxins
Copy link
Contributor

haoxins commented Jan 20, 2022

It would probably be nice to add a new CRD to support Flink-native k8s cluster. I think also implementing the application mode with new CRD will also help lower the complexity.
API improvements, for example:

  • Rename current FlinkCluster to FlinkSessionCluster
  • New CRD for native k8s Flink cluster: FlinkCluster
  • New CRDs for various job types: FlinkJob, BeamJob (spec includes FlinkCluster or FlinkSessionCluster template for dedicated use or link existing cluster as clusterName to share it). I think new FlinkJob could support various modes like application, per-job on native k8s Flink cluster.

Is there any progress or plan on this feature @elanv thanks

Is there any one working on this?
I think we can split this issue into multi issues/steps so that others can help with it.

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

No branches or pull requests

5 participants