Zakelly
Zakelly
@spoon-lz sorry for the late reply. I agreed with @masteryhx that we'd better create another new class and deprecate the old one in python.
I modified the commit message and remove those 'cherry-picked from xxxxx'. Already merged into master.
@drymatini I think your commits include the wrong files by mistake, maybe you amend commit on top of my previous one?
Thanks for the update! I'll take a look this week.
> @Zakelly any comments/thought? Thanks for the heads up, looking into this...
@HuangZhenQiu Thanks for your answer. I see your point about the metadata controller (e.g. NameNode) being the bottleneck. However this is still under the speed control story, given QPS/RPS and...
> It will take longer end to end time for single TM to finish the state upload if reduce thread num from 4 to 2. It would cause checkpoint timeout...
> I've updated the PR to completely remove the standard YAML flag from the code. Additionally, I've removed the setStandardYaml calls from `RocksDBStateBackendConfigTest#testConfigureForStCompressionPerLevel `and `ForStStateBackendConfigTest#testConfigureForStCompressionPerLevel`, as discussed with @Zakelly. Although...
And a rebase is needed since there are conflicts with master branch.
And BTW, you should run `mvn package -Dgenerate-config-docs -pl flink-docs -am -nsu -DskipTests -Pskip-webui-build` if you add/change the configurations. Please check the instructions in `flink-docs/README.md`.