[GitHub] flink issue #4534: [FLINK-7358][table]Add implicitly converts support for Us...

2017-09-22 Thread sunjincheng121
Github user sunjincheng121 commented on the issue: https://github.com/apache/flink/pull/4534 @fhueske Because the 11.11, I have not update this PR in time. sorry about that. Now, I have updated the PR. and PR description. Please look at the PR. :) Thanks, jincheng ---

[GitHub] flink issue #4534: [FLINK-7358][table]Add implicitly converts support for Us...

2017-08-28 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/4534 The text above `"What is the purpose of the change"` should be removed such that only the relevant info is in the PR description. --- If your project is set up for it, you can reply to this email

[GitHub] flink issue #4534: [FLINK-7358][table]Add implicitly converts support for Us...

2017-08-18 Thread sunjincheng121
Github user sunjincheng121 commented on the issue: https://github.com/apache/flink/pull/4534 Hi, @fhueske Thank you for reminding me :-). I updated the description. and feel free to tell me if there is any inappropriate description. Thanks, jincheng --- If your project is

[GitHub] flink issue #4534: [FLINK-7358][table]Add implicitly converts support for Us...

2017-08-17 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/4534 Hi @sunjincheng121, can you please update the PR description and fill out the PR template? The community decided to extend the PR template and to be more strict about the PR submission