pyscala commented on pull request #12566:
URL: https://github.com/apache/flink/pull/12566#issuecomment-654058889
> Thanks for the contribution. Why default value is 1? Why not keep
Int.Max?
@JingsongLi Thanks for your reply , The details link
pyscala commented on pull request #12566:
URL: https://github.com/apache/flink/pull/12566#issuecomment-654045783
@JingsongLi @wuchong Can you review this PR for me? thanks.
This is an automated message from the Apache Git
pyscala commented on pull request #12566:
URL: https://github.com/apache/flink/pull/12566#issuecomment-651727239
@becketqin The problems have been solved, and the current error seems to
have nothing to do with our changes .
Looking forward to your reply , thanks
pyscala commented on pull request #12566:
URL: https://github.com/apache/flink/pull/12566#issuecomment-651554556
@flinkbot run azure
This is an automated message from the Apache Git Service.
To respond to the message, please
pyscala commented on pull request #12566:
URL: https://github.com/apache/flink/pull/12566#issuecomment-651156382
> @pyscala Thanks for the patch. LGTM. Can we add a unit test for this?
thanks @becketqin for your reply ,i will add a unit test for this
pyscala commented on pull request #12566:
URL: https://github.com/apache/flink/pull/12566#issuecomment-641707884
@flinkbot run travis
This is an automated message from the Apache Git Service.
To respond to the
pyscala commented on pull request #12566:
URL: https://github.com/apache/flink/pull/12566#issuecomment-641693599
Hi @becketqin ,appreciate for your time to review this PR,thanks
This is an automated message from the Apache
