Github user zentol commented on the issue:
https://github.com/apache/flink/pull/6378
merging.
---
Github user YCjia commented on the issue:
https://github.com/apache/flink/pull/6378
@greghogan Thanks for your reminder. Since version 20 was released on
July 7, 2018, I think it is more appropriate to use version 20. I have modified
the code I submitted. I am a newbie here, please
Github user YCjia commented on the issue:
https://github.com/apache/flink/pull/6378
@zentol Thank you for your reminder, I will pay attention in the future.
---
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/6378
@zhangminglei @YCjia Please do not push changes for the sole purpose of
re-triggering Travis. We already know that they passed (due to the travis
fork), and realistically there's no way for the parent
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/6378
Could you please push code and trigger the travis again ?
---
Github user YCjia commented on the issue:
https://github.com/apache/flink/pull/6378
@TisonShadowsong @zhangminglei I ran these tests locally, but they all
succeeded, and the Travis CI build for my fork repository passed too.
https://travis-ci.com/YCjia/flink/builds
---
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/6378
@TisonShadowsong is right. You can verify that and i think it might be a
bug.
---
Github user TisonShadowsong commented on the issue:
https://github.com/apache/flink/pull/6378
The test that fails is `WindowCheckpointingITCase`, not sure but @YCjia you
can run it locally to figure out what happened.
---
Github user tedyu commented on the issue:
https://github.com/apache/flink/pull/6378
No particular reason - considering the date when JIRA was first logged :-)
---
Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/6378
Any reason not to use version 20 or are we just being conservative?
---
Github user tedyu commented on the issue:
https://github.com/apache/flink/pull/6378
lgtm
Pending QA bot
---
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/6378
@YCjia The travis test failed. Could you take a look on what is happening ?
---
12 matches
Mail list logo