[GitHub] flink pull request: Improved exception for missing type of InputFo...

2015-01-29 Thread hsaputra
Github user hsaputra commented on the pull request: https://github.com/apache/flink/pull/351#issuecomment-72162452 To reduce code duplication and tracing, we could just throw the exception inside createInput(InputFormat inputFormat, TypeInformation producedType) by wrapping IllegalArg

[GitHub] flink pull request: Add support for Subclasses, Interfaces, Abstra...

2015-01-29 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/236#issuecomment-72161285 Then it fails at runtime, which makes me very uneasy. But then again, stuff can always fail at runtime when the user uses some strange subclass. Even more so without this

[jira] [Commented] (FLINK-1452) Add "flink-contrib" maven module and README.md with the rules

2015-01-29 Thread Henry Saputra (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297785#comment-14297785 ] Henry Saputra commented on FLINK-1452: -- I like {{flink-contrib}} and it reflect the c

[jira] [Commented] (FLINK-1460) Typo fixes

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297756#comment-14297756 ] ASF GitHub Bot commented on FLINK-1460: --- Github user hsaputra commented on the pull

[GitHub] flink pull request: [FLINK-1460] fix typos

2015-01-29 Thread hsaputra
Github user hsaputra commented on the pull request: https://github.com/apache/flink/pull/346#issuecomment-72111491 @coderxiang, ah that the tests do comparison in the after method so it is hard to figure out which method fail. But seemed like other builds passed and I am sure that you

[jira] [Commented] (FLINK-1460) Typo fixes

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297727#comment-14297727 ] ASF GitHub Bot commented on FLINK-1460: --- Github user hsaputra commented on the pull

[GitHub] flink pull request: [FLINK-1460] fix typos

2015-01-29 Thread hsaputra
Github user hsaputra commented on the pull request: https://github.com/apache/flink/pull/346#issuecomment-72109489 HI @aljoscha, thanks. If you could merge it before later tonight please do. Otherwise I could do it later tonight if you did not have time to get to this one =) ---

[jira] [Commented] (FLINK-1460) Typo fixes

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297713#comment-14297713 ] ASF GitHub Bot commented on FLINK-1460: --- Github user coderxiang commented on the pul

[GitHub] flink pull request: [FLINK-1460] fix typos

2015-01-29 Thread coderxiang
Github user coderxiang commented on the pull request: https://github.com/apache/flink/pull/346#issuecomment-72107415 @hsaputra @aljoscha I failed one check which I don't have much idea about. Could you please retest? --- If your project is set up for it, you can reply to this email

[jira] [Commented] (FLINK-1452) Add "flink-contrib" maven module and README.md with the rules

2015-01-29 Thread Chiwan Park (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297692#comment-14297692 ] Chiwan Park commented on FLINK-1452: +1 {{flink-staging}} And, I think we can refer t

[jira] [Commented] (FLINK-1422) Missing usage example for "withParameters"

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297687#comment-14297687 ] ASF GitHub Bot commented on FLINK-1422: --- Github user zentol commented on the pull re

[GitHub] flink pull request: [FLINK-1422] Add withParameters() to documenta...

2015-01-29 Thread zentol
Github user zentol commented on the pull request: https://github.com/apache/flink/pull/350#issuecomment-72105374 gotcha, I'll address the points mentioned. --- 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 d

[GitHub] flink pull request: Add support for Subclasses, Interfaces, Abstra...

2015-01-29 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/236#issuecomment-72103112 What would happen if a base class fulfills the POJO requirements, but the subclass does not? --- If your project is set up for it, you can reply to this email and hav

[jira] [Commented] (FLINK-1422) Missing usage example for "withParameters"

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297602#comment-14297602 ] ASF GitHub Bot commented on FLINK-1422: --- Github user fhueske commented on the pull r

[GitHub] flink pull request: [FLINK-1422] Add withParameters() to documenta...

2015-01-29 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/350#issuecomment-72098340 Very good addition to the documentation! I also agree with Alex (see FLINK-1422) to describe how functions can be configured via the constructor, i.e., that function o

[GitHub] flink pull request: Improved exception for missing type of InputFo...

2015-01-29 Thread twalthr
GitHub user twalthr opened a pull request: https://github.com/apache/flink/pull/351 Improved exception for missing type of InputFormat As per discussion on the dev mailing list. You can merge this pull request into a Git repository by running: $ git pull https://github.com/twal

[GitHub] flink pull request: Added ResultTypeQueryable interface to TypeSer...

2015-01-29 Thread twalthr
Github user twalthr commented on the pull request: https://github.com/apache/flink/pull/349#issuecomment-72042165 The change makes sense. The TypeExtractor can never get the types from `new TypeSerializerInputFormat()` automatically, so passing them in the constructor is correct.

[jira] [Created] (FLINK-1462) Add documentation guide for the graph API

2015-01-29 Thread Vasia Kalavri (JIRA)
Vasia Kalavri created FLINK-1462: Summary: Add documentation guide for the graph API Key: FLINK-1462 URL: https://issues.apache.org/jira/browse/FLINK-1462 Project: Flink Issue Type: Task

[jira] [Commented] (FLINK-1422) Missing usage example for "withParameters"

2015-01-29 Thread Alexander Alexandrov (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296986#comment-14296986 ] Alexander Alexandrov commented on FLINK-1422: - +1 from me as well. I recently

[jira] [Updated] (FLINK-1376) SubSlots are not properly released in case that a TaskManager fatally fails, leaving the system in a corrupted state

2015-01-29 Thread Chesnay Schepler (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1376?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chesnay Schepler updated FLINK-1376: Assignee: Till Rohrmann > SubSlots are not properly released in case that a TaskManager fata

[jira] [Commented] (FLINK-1193) InvalidTypesException for Array[Array[Tuple2[Int, Double]]]

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296972#comment-14296972 ] ASF GitHub Bot commented on FLINK-1193: --- Github user aljoscha commented on a diff in

[GitHub] flink pull request: [FLINK-1193][java-api][scala-api] Complete sup...

2015-01-29 Thread aljoscha
Github user aljoscha commented on a diff in the pull request: https://github.com/apache/flink/pull/348#discussion_r23773125 --- Diff: flink-tests/src/test/scala/org/apache/flink/api/scala/types/TypeInformationGenTest.scala --- @@ -255,6 +255,53 @@ class TypeInformationGenTest {

[GitHub] flink pull request: [FLINK-1422] Add withParameters() to documenta...

2015-01-29 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/350#issuecomment-72039131 +1 looks good 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 fea

[jira] [Commented] (FLINK-1422) Missing usage example for "withParameters"

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296971#comment-14296971 ] ASF GitHub Bot commented on FLINK-1422: --- Github user aljoscha commented on the pull

[jira] [Commented] (FLINK-1450) Add Fold operator to the Streaming api

2015-01-29 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296968#comment-14296968 ] Aljoscha Krettek commented on FLINK-1450: - I see the necessity in the streaming AP

[jira] [Commented] (FLINK-1422) Missing usage example for "withParameters"

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296966#comment-14296966 ] ASF GitHub Bot commented on FLINK-1422: --- GitHub user zentol opened a pull request:

[GitHub] flink pull request: [FLINK-1422] Add withParameters() to documenta...

2015-01-29 Thread zentol
GitHub user zentol opened a pull request: https://github.com/apache/flink/pull/350 [FLINK-1422] Add withParameters() to documentation You can merge this pull request into a Git repository by running: $ git pull https://github.com/zentol/incubator-flink docu_parameters Alterna

[GitHub] flink pull request: Added ResultTypeQueryable interface to TypeSer...

2015-01-29 Thread aalexandrov
GitHub user aalexandrov opened a pull request: https://github.com/apache/flink/pull/349 Added ResultTypeQueryable interface to TypeSerializerInputFormat. It is currently impossible to use the `TypeSerializerInputFormat` with generic Tuple types. For example, [this example

[jira] [Commented] (FLINK-1456) Projection to fieldnames, keyselectors

2015-01-29 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296957#comment-14296957 ] Aljoscha Krettek commented on FLINK-1456: - I don't know if this is really necessar

[jira] [Commented] (FLINK-1460) Typo fixes

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296949#comment-14296949 ] ASF GitHub Bot commented on FLINK-1460: --- Github user aljoscha commented on the pull

[GitHub] flink pull request: [FLINK-1460] fix typos

2015-01-29 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/346#issuecomment-72036147 +1, can you merge it @hsaputra or should I? --- 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 projec

[jira] [Commented] (FLINK-1452) Add "flink-contrib" maven module and README.md with the rules

2015-01-29 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296912#comment-14296912 ] Aljoscha Krettek commented on FLINK-1452: - Or {{flink-staging}}. > Add "flink-con

[jira] [Commented] (FLINK-1447) Flink Web Submission Client does not work on Ubuntu 14.04 firefox browser

2015-01-29 Thread Robert Metzger (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296902#comment-14296902 ] Robert Metzger commented on FLINK-1447: --- I just ran into the issue as well, on the l

[GitHub] flink pull request: Add support for Subclasses, Interfaces, Abstra...

2015-01-29 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/236#issuecomment-72020646 I took @fhueske's rebase from #316. I also added a switch to the TypeExtractor to force it to use Kryo, even though it could analyse a type as a POJO. In some cas

[jira] [Commented] (FLINK-1193) InvalidTypesException for Array[Array[Tuple2[Int, Double]]]

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296788#comment-14296788 ] ASF GitHub Bot commented on FLINK-1193: --- GitHub user twalthr opened a pull request:

[GitHub] flink pull request: [FLINK-1193][java-api][scala-api] Complete sup...

2015-01-29 Thread twalthr
GitHub user twalthr opened a pull request: https://github.com/apache/flink/pull/348 [FLINK-1193][java-api][scala-api] Complete support for multidimensional arrays This PR introduces official support for multidimensional arrays. It contains bug fixes for TypeExtractor and S

[jira] [Created] (FLINK-1461) Add sortPartition operator

2015-01-29 Thread Fabian Hueske (JIRA)
Fabian Hueske created FLINK-1461: Summary: Add sortPartition operator Key: FLINK-1461 URL: https://issues.apache.org/jira/browse/FLINK-1461 Project: Flink Issue Type: New Feature Co

[jira] [Commented] (FLINK-1105) Add support for locally sorted output

2015-01-29 Thread Fabian Hueske (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1105?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296712#comment-14296712 ] Fabian Hueske commented on FLINK-1105: -- [~StephanEwen] Yes, I thought about adding a

[jira] [Commented] (FLINK-1105) Add support for locally sorted output

2015-01-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1105?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296708#comment-14296708 ] ASF GitHub Bot commented on FLINK-1105: --- GitHub user fhueske opened a pull request:

[GitHub] flink pull request: [FLINK-1105] Add support for locally sorted ou...

2015-01-29 Thread fhueske
GitHub user fhueske opened a pull request: https://github.com/apache/flink/pull/347 [FLINK-1105] Add support for locally sorted output You can merge this pull request into a Git repository by running: $ git pull https://github.com/fhueske/flink locallySortedOutput Alternative

[GitHub] flink pull request: Add support for Subclasses, Interfaces, Abstra...

2015-01-29 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/236#issuecomment-72006969 We could try and analyse the type as a POJO. If it is a POJO add it to the PojoSerializer, otherwise, add it to the KryoSerializer. --- If your project is set up for it

[jira] [Commented] (FLINK-1459) Collect DataSet to client

2015-01-29 Thread Stephan Ewen (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1459?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14296610#comment-14296610 ] Stephan Ewen commented on FLINK-1459: - [~jsand] Stay tuned for a few days, it is going