[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-16 Thread aljoscha
GitHub user aljoscha opened a pull request: https://github.com/apache/flink/pull/405 [FLINK-947] Add a declarative expression API This one is quite big. So you should check out the documentation, skaldic, examples and test cases to see how the API works. You can merge this pull req

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-16 Thread mbalassi
Github user mbalassi commented on the pull request: https://github.com/apache/flink/pull/405#issuecomment-74579541 Great additions! Really looking to trying 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 yo

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-16 Thread mbalassi
Github user mbalassi commented on a diff in the pull request: https://github.com/apache/flink/pull/405#discussion_r24778932 --- Diff: flink-staging/flink-linq/pom.xml --- @@ -0,0 +1,227 @@ + + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSch

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-17 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/405#discussion_r24857326 --- Diff: flink-staging/flink-linq/pom.xml --- @@ -0,0 +1,227 @@ + + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSch

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-17 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/405#discussion_r24858527 --- Diff: docs/linq.md --- @@ -0,0 +1,65 @@ +--- +title: "Language-Integrated Queries" +--- + + +* This will be replaced by the TOC

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-17 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/405#discussion_r24862039 --- Diff: flink-staging/flink-linq/src/main/scala/org/apache/flink/api/scala/expressions/package.scala --- @@ -0,0 +1,104 @@ +/* + * Licensed to the

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-17 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/405#issuecomment-74773124 Very very nice work! I've played around a bit with it and the first impression is very good. +1 to merge the pull request. The change is very big, but stable enou

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-18 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/405#issuecomment-74832989 One more thing, the maven module is called "flink-linq". Are we certain that we can use the name LINQ without problems here? --- If your project is set up for it, you ca

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-18 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/405#issuecomment-74842606 Yeah, I'm not sure about linq as well. I like the name but realise that it might be problematic. What do the others think. I could call it "flink-expressions". I

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-18 Thread mxm
Github user mxm commented on the pull request: https://github.com/apache/flink/pull/405#issuecomment-74845397 To me, `flink-expressions` sounds much better than `linq` and it mitigates the risk of law suites :) --- If your project is set up for it, you can reply to this email and hav

[GitHub] flink pull request: [FLINK-947] Add a declarative expression API

2015-02-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/405 --- 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