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
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 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 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 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 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 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 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 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 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 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
11 matches
Mail list logo