chunjun icon indicating copy to clipboard operation
chunjun copied to clipboard

support elasticsearch6 & support memory prop in yarn per-job mode

Open wwuht opened this issue 3 years ago • 3 comments
trafficstars

wwuht avatar Jun 02 '22 10:06 wwuht

你好,感谢提供的pr。首先第一个commit,我们已经支持了es6,我们注释掉的原因是因为在session模式下,会出现类冲突问题,所以选择注释掉;第二个能否提供一个issue 描述下呢?同时commit 信息修改下,pr 模版是[hotfix-#issueID][fix-module] fix-commit.

FlechazoW avatar Jun 02 '22 13:06 FlechazoW

  1. es6,除了启用之外,还修复了format空指针问题。session模式下的类冲突问题不清楚,yarn perJob模式下测试正常。
  2. 第二个commit:“ jobManagerMemoryMb = jobManagerMemoryMb >> 20” 不确定其意图,故去除;支持传参带单位(形如4096m),保持与flink原本的行为一致;去除最小1024m的限制,改为默认1024m

翻了一下issue,没看到有类似的,故没带issueID

wwuht avatar Jun 04 '22 03:06 wwuht

感谢pr,对于这两个问题,能否建立一下issue描述下并关联下呢?为了合理的规划commit信息,我们的规范是一个issue 对应一个pr。没有issue 是需要提交人创建一个issue 来描述修复内容的,否则review 者不清楚上下文,不能很好的理解这个pr。感谢理解

FlechazoW avatar Jun 05 '22 07:06 FlechazoW