chunjun
chunjun copied to clipboard
support elasticsearch6 & support memory prop in yarn per-job mode
trafficstars
你好,感谢提供的pr。首先第一个commit,我们已经支持了es6,我们注释掉的原因是因为在session模式下,会出现类冲突问题,所以选择注释掉;第二个能否提供一个issue 描述下呢?同时commit 信息修改下,pr 模版是[hotfix-#issueID][fix-module] fix-commit.
- es6,除了启用之外,还修复了format空指针问题。session模式下的类冲突问题不清楚,yarn perJob模式下测试正常。
- 第二个commit:“ jobManagerMemoryMb = jobManagerMemoryMb >> 20” 不确定其意图,故去除;支持传参带单位(形如4096m),保持与flink原本的行为一致;去除最小1024m的限制,改为默认1024m
翻了一下issue,没看到有类似的,故没带issueID
感谢pr,对于这两个问题,能否建立一下issue描述下并关联下呢?为了合理的规划commit信息,我们的规范是一个issue 对应一个pr。没有issue 是需要提交人创建一个issue 来描述修复内容的,否则review 者不清楚上下文,不能很好的理解这个pr。感谢理解