Dagang Wei
Dagang Wei
Are you using different emails for your GitHub account and the commit? They need to be the same.
Sounds like a great feature to add. IIUC, it leverages the update cluster feature we added recently when the autoscaler decides TM should be scaled. But I have a concern...
Sounds good. Go ahead adding this feature, I will review your PR once it is ready.
Is this a helm chart specific problem? Does it manifest with `make deploy`?
Originally it is supported, then replaced by the current approach, we can consider adding it back as an option.
cert-manager is not mandatory, but some users might want to use it for the reasons mentioned by @elanv.
Thanks for the bug report, but according to the [doc](https://ci.apache.org/projects/flink/flink-docs-release-1.10/ops/config.html), Flink >=1.10 supports/requires both `jobmanager.heap.size` and `taskmanager.memory.process.size`: > jobmanager.heap.size: Sets the size of the Flink Master (JobManager / ResourceManager /...
Actually I couldn't find `taskmanager.heap.size` in the [Flink 1.10 doc](https://ci.apache.org/projects/flink/flink-docs-release-1.10/ops/config.html#memory-configuration), and there are many new `taskmanager.memory.*` properties. So, I don't understand the reason > I found "taskmanager.heap.size" is set as...
Gotcha. What if you specify `taskmanager.memory.flink.size` manually in `flinkProperties`? That should solve the problem I think. I think instead of adding a field `useProcessSize`, we might want to add a...