[GitHub] flink pull request: [FLINK-2000] [table] Add sql style aggregation...
GitHub user chhao01 opened a pull request: https://github.com/apache/flink/pull/782 [FLINK-2000] [table] Add sql style aggregation support https://issues.apache.org/jira/browse/FLINK-2000 You can merge this pull request into a Git repository by running: $ git pull https://github.com/chhao01/flink flink_2000 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/782.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #782 commit 775d6b168b60fae26d2a416258b198fae266 Author: Cheng Hao Date: 2015-06-04T15:17:35Z add sql style aggregation support --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...
Github user chhao01 commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-101719363 Failed in `flink-streaming-connectors` with java 8, does that related to this change? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...
Github user chhao01 commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-101688740 Thank you @aljoscha, eventually I got the IDE works with unit test, and I update the code, let's see what travis will say. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...
Github user chhao01 commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-101466586 @aljoscha there is nothing when I open the Travis CI link, is there any hint why this failed? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...
Github user chhao01 commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-101344334 @aljoscha sorry for the delay, my jira id is `chhao01`, and I can start working on the `FLINK-2000` once this got merged. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...
Github user chhao01 commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-100943527 Thanks for the quick response, I was not sure which is suppose to be. Since it's case-insensitive, I will update the code with unit test soon. Yes, I am interested in adding the SQL style aggregations, it will be great if you can create the jira and assign to me. :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...
GitHub user chhao01 opened a pull request: https://github.com/apache/flink/pull/667 [FLINK-1990] [staging table] Support upper case of `as` for expression You can merge this pull request into a Git repository by running: $ git pull https://github.com/chhao01/flink as Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/667.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #667 commit bd1d059440a23e4b2a725946716f580e168c2a7b Author: Cheng Hao Date: 2015-05-11T14:57:14Z support upper case of 'as' for expression --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---