[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/667 --- 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 enabl

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-15 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-102374728 I'm running my own tests on travis and then I'll merge it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-14 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-102076699 Can you post a link to the failed run? Or is it on your local machine? Some of the streaming test cases fail sometimes right now, this is a known problem. --- If your

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-13 Thread chhao01
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 rep

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-13 Thread chhao01
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

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-13 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-101617351 For some reason it doesn't like the keywork2Parser when you use String Interpolation. If you replace the line with: ``` ("""(i?)\Q""" + kw.key + """\E""").r

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-13 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-101544907 I'll check it out. --- 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 fea

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-12 Thread chhao01
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

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-12 Thread chhao01
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 r

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-11 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-100950435 I created this issue for the aggregations: https://issues.apache.org/jira/browse/FLINK-2000. I think I can only assign you if you have a Jira account. Do you have one?

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-11 Thread chhao01
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 intereste

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-11 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/667#issuecomment-100940455 Thanks for your contribution! Could you maybe enhance it along these lines: http://stackoverflow.com/questions/6080437/case-insensitive-scala-parser-combinator

[GitHub] flink pull request: [FLINK-1990] [staging table] Support upper cas...

2015-05-11 Thread chhao01
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 Alternat