Yangze Guo
Yangze Guo
@xinbinhuang There are failure tests that seem to be related to this change. Would you like to take a look first?
@xinbinhuang If you need any help in fixing those tests, feel free to ask :).
@xinbinhuang Just a kindly reminder, please rebase your PR to the latest master instead of merging it.
Hmm. I'm afraid I can not help a lot. Maybe you can follow [this guide](https://nightlies.apache.org/flink/flink-docs-release-1.14/docs/flinkdev/ide_setup/#intellij-idea) to reimport your project. BTW, the CI is currently unstable, you can retrigger it later.
@xinbinhuang Thanks for the update. Could you explain why you need to derive the total task slots required from the JSON graph?
> > @xinbinhuang Thanks for the update. Could you explain why you need to derive the total task slots required from the JSON graph? > > Our team wants to...
> > https://nightlies.apache.org/flink/flink-docs-master/docs/deployment/finegrained_resource/#notice > > ah, wasn't realized this! thanks for the reference! I'll give a through read over the weekend. > > From the doc section you link, it...
> > > > https://nightlies.apache.org/flink/flink-docs-master/docs/deployment/finegrained_resource/#notice > > > > > > > > > ah, wasn't realized this! thanks for the reference! I'll give a through read over the weekend....
JFYI, I've refactor this PR, addressed all the exisiting comments and rebase on the lastest master in #23565
> @RocMarshal Just be curious about the progress, does this PR still wait for some comments to be addressed before it could be merged? This PR is in progress now....