[GitHub] flink issue #3915: [FLINK-6352] Support to use timestamp to set the initial ...

2017-05-18 Thread zjureel
Github user zjureel commented on the issue: https://github.com/apache/flink/pull/3915 @tzulitai I have fixed `setStartFromSpecificDate` problem, and updated `FlinkKafkaConsumer08` so that it supports to set start offsets of Kafka by date --- If your project is set up for it, you can

[jira] [Commented] (FLINK-6352) FlinkKafkaConsumer should support to use timestamp to set up start offset

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015307#comment-16015307 ] ASF GitHub Bot commented on FLINK-6352: --- Github user zjureel commented on the issue:

[GitHub] flink pull request #3937: [FLINK-6482] [core] Add nested serializers to conf...

2017-05-18 Thread tzulitai
GitHub user tzulitai opened a pull request: https://github.com/apache/flink/pull/3937 [FLINK-6482] [core] Add nested serializers to config snapshots of composite serializers This commit adds also the nested serializers themselves to the configuration snapshots of composite serializ

[jira] [Commented] (FLINK-6482) Add nested serializers into configuration snapshots of composite serializers

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015315#comment-16015315 ] ASF GitHub Bot commented on FLINK-6482: --- GitHub user tzulitai opened a pull request:

[GitHub] flink pull request #3915: [FLINK-6352] Support to use timestamp to set the i...

2017-05-18 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/3915#discussion_r117175199 --- Diff: flink-connectors/flink-connector-kafka-0.8/src/main/java/org/apache/flink/streaming/connectors/kafka/FlinkKafkaConsumer08.java --- @@ -187,31 +19

[GitHub] flink pull request #3915: [FLINK-6352] Support to use timestamp to set the i...

2017-05-18 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/3915#discussion_r117175607 --- Diff: flink-connectors/flink-connector-kafka-0.9/src/main/java/org/apache/flink/streaming/connectors/kafka/FlinkKafkaConsumer09.java --- @@ -171,6 +170

[GitHub] flink pull request #3915: [FLINK-6352] Support to use timestamp to set the i...

2017-05-18 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/3915#discussion_r117175760 --- Diff: flink-connectors/flink-connector-kafka-base/src/main/java/org/apache/flink/streaming/connectors/kafka/FlinkKafkaConsumerBase.java --- @@ -311,12

[jira] [Commented] (FLINK-6352) FlinkKafkaConsumer should support to use timestamp to set up start offset

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015323#comment-16015323 ] ASF GitHub Bot commented on FLINK-6352: --- Github user tzulitai commented on a diff in

[jira] [Commented] (FLINK-6352) FlinkKafkaConsumer should support to use timestamp to set up start offset

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015322#comment-16015322 ] ASF GitHub Bot commented on FLINK-6352: --- Github user tzulitai commented on a diff in

[jira] [Commented] (FLINK-6352) FlinkKafkaConsumer should support to use timestamp to set up start offset

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015324#comment-16015324 ] ASF GitHub Bot commented on FLINK-6352: --- Github user tzulitai commented on a diff in

[GitHub] flink pull request #3525: [FLINK-6020]add a random integer suffix to blob ke...

2017-05-18 Thread WangTaoTheTonic
Github user WangTaoTheTonic closed the pull request at: https://github.com/apache/flink/pull/3525 --- 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 featur

[GitHub] flink issue #3525: [FLINK-6020]add a random integer suffix to blob key to av...

2017-05-18 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3525 @tillrohrmann I've tried with your commit and the issue is resolved, thanks. Closing this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] flink pull request #3715: [FLINK-6232][Table&SQL]Support proctime inner equi...

2017-05-18 Thread hongyuhong
Github user hongyuhong commented on a diff in the pull request: https://github.com/apache/flink/pull/3715#discussion_r117176330 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/rules/datastream/DataStreamJoinRule.scala --- @@ -0,0 +1,69 @@ +/*

[jira] [Commented] (FLINK-6020) Blob Server cannot handle multiple job submits (with same content) parallelly

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6020?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015328#comment-16015328 ] ASF GitHub Bot commented on FLINK-6020: --- Github user WangTaoTheTonic closed the pull

[jira] [Commented] (FLINK-6020) Blob Server cannot handle multiple job submits (with same content) parallelly

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6020?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015327#comment-16015327 ] ASF GitHub Bot commented on FLINK-6020: --- Github user WangTaoTheTonic commented on th

[jira] [Commented] (FLINK-6232) Support proctime inner equi-join between two streams in the SQL API

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015329#comment-16015329 ] ASF GitHub Bot commented on FLINK-6232: --- Github user hongyuhong commented on a diff

[GitHub] flink issue #3525: [FLINK-6020]add a random integer suffix to blob key to av...

2017-05-18 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/3525 Great to hear @WangTaoTheTonic. Thanks a lot for testing. --- 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

[jira] [Commented] (FLINK-6020) Blob Server cannot handle multiple job submits (with same content) parallelly

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6020?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015364#comment-16015364 ] ASF GitHub Bot commented on FLINK-6020: --- Github user tillrohrmann commented on the i

[GitHub] flink pull request #3926: [FLINK-6570] QueryableStateClient docs with matchi...

2017-05-18 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/3926#discussion_r117180703 --- Diff: docs/dev/stream/queryable_state.md --- @@ -208,10 +208,16 @@ Once used in a job, you can retrieve the job ID and then query any key's current

[GitHub] flink issue #3926: [FLINK-6570] QueryableStateClient docs with matching cons...

2017-05-18 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/3926 Thanks for your contribution @gosubpl. Changes look good to me. Merging this and addressing my comment while doing so. --- If your project is set up for it, you can reply to this email and have

[jira] [Commented] (FLINK-6570) QueryableStateClient constructor in documentation doesn't match actual signature

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015372#comment-16015372 ] ASF GitHub Bot commented on FLINK-6570: --- Github user tillrohrmann commented on a dif

[jira] [Commented] (FLINK-6570) QueryableStateClient constructor in documentation doesn't match actual signature

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015373#comment-16015373 ] ASF GitHub Bot commented on FLINK-6570: --- Github user tillrohrmann commented on the i

[GitHub] flink pull request #3926: [FLINK-6570] QueryableStateClient docs with matchi...

2017-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3926 --- 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 enab

[jira] [Resolved] (FLINK-6570) QueryableStateClient constructor in documentation doesn't match actual signature

2017-05-18 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6570?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann resolved FLINK-6570. -- Resolution: Fixed Fix Version/s: 1.4.0 1.3.0 1.4.0: 85e281be3835fe8bb0

[jira] [Commented] (FLINK-6570) QueryableStateClient constructor in documentation doesn't match actual signature

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015380#comment-16015380 ] ASF GitHub Bot commented on FLINK-6570: --- Github user asfgit closed the pull request

[GitHub] flink pull request #3889: [FLINK-6075] - Support Limit/Top(Sort) for Stream ...

2017-05-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3889#discussion_r117182703 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/RowTimeSortProcessFunction.scala --- @@ -0,0 +1,169 @@ +/*

[jira] [Commented] (FLINK-6075) Support Limit/Top(Sort) for Stream SQL

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6075?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015385#comment-16015385 ] ASF GitHub Bot commented on FLINK-6075: --- Github user fhueske commented on a diff in

[GitHub] flink pull request #3919: [FLINK-6583][talbe]Enable QueryConfig in count bas...

2017-05-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3919#discussion_r117183206 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/nodes/datastream/DataStreamGroupWindowAggregate.scala --- @@ -245,6 +264,7

[jira] [Commented] (FLINK-6583) Enable QueryConfig in count base GroupWindow

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6583?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015387#comment-16015387 ] ASF GitHub Bot commented on FLINK-6583: --- Github user fhueske commented on a diff in

[GitHub] flink pull request #3919: [FLINK-6583][talbe]Enable QueryConfig in count bas...

2017-05-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3919#discussion_r117183489 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/nodes/datastream/DataStreamGroupWindowAggregate.scala --- @@ -131,6 +135,19

[jira] [Commented] (FLINK-6583) Enable QueryConfig in count base GroupWindow

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6583?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015389#comment-16015389 ] ASF GitHub Bot commented on FLINK-6583: --- Github user fhueske commented on a diff in

[jira] [Updated] (FLINK-6619) Check Table API & SQL support for 1.3.0 RC01 Release

2017-05-18 Thread sunjincheng (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6619?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] sunjincheng updated FLINK-6619: --- Description: In this JIRA. I will do the following tasks for Flink 1.3.0 RC01 Release. * Check that th

[GitHub] flink issue #3901: [FLINK-6288] fix FlinkKafkaProducer's custom Partitioner ...

2017-05-18 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3901 This looks good! I'll do some final tweaks to the styling of the code and then merge this :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[jira] [Commented] (FLINK-6288) FlinkKafkaProducer's custom Partitioner is always invoked with number of partitions of default topic

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015411#comment-16015411 ] ASF GitHub Bot commented on FLINK-6288: --- Github user tzulitai commented on the issue

[jira] [Updated] (FLINK-6618) Fix GroupWindowStringExpressionTest testcase bug

2017-05-18 Thread sunjincheng (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6618?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] sunjincheng updated FLINK-6618: --- Description: I find 2 bugs as follows: 1. {{GroupWindowStringExpressionTest}} testcase bug: {{Assert

[jira] [Created] (FLINK-6622) flink master code not compiling

2017-05-18 Thread constantine stanley (JIRA)
constantine stanley created FLINK-6622: -- Summary: flink master code not compiling Key: FLINK-6622 URL: https://issues.apache.org/jira/browse/FLINK-6622 Project: Flink Issue Type: Bug

[jira] [Created] (FLINK-6623) unable to build flink master

2017-05-18 Thread constantine stanley (JIRA)
constantine stanley created FLINK-6623: -- Summary: unable to build flink master Key: FLINK-6623 URL: https://issues.apache.org/jira/browse/FLINK-6623 Project: Flink Issue Type: Bug

[jira] [Closed] (FLINK-6622) flink master code not compiling

2017-05-18 Thread Chesnay Schepler (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6622?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chesnay Schepler closed FLINK-6622. --- Resolution: Duplicate This is not a compilation issue; the problem is that one of the tests ca

[GitHub] flink issue #3746: [FLINK-4022] [kafka] Partition and topic discovery for Fl...

2017-05-18 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3746 Hi @haohui, I revisited this, but I think there is no easy workaround for this, unfortunately. This can only be properly merged in 1.4 after some additional requirements are added to Flink.

[jira] [Commented] (FLINK-4022) Partition discovery / regex topic subscription for the Kafka consumer

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015457#comment-16015457 ] ASF GitHub Bot commented on FLINK-4022: --- Github user tzulitai commented on the issue

[jira] [Updated] (FLINK-6482) Add nested serializers into configuration snapshots of composite serializers

2017-05-18 Thread Tzu-Li (Gordon) Tai (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6482?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tzu-Li (Gordon) Tai updated FLINK-6482: --- Priority: Blocker (was: Critical) > Add nested serializers into configuration snapsho

[jira] [Commented] (FLINK-6623) unable to build flink master

2017-05-18 Thread Chesnay Schepler (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015459#comment-16015459 ] Chesnay Schepler commented on FLINK-6623: - It would be great if you could provide

[jira] [Updated] (FLINK-6623) Several tests fail on Windows

2017-05-18 Thread Chesnay Schepler (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6623?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chesnay Schepler updated FLINK-6623: Summary: Several tests fail on Windows (was: unable to build flink master) > Several tests

[GitHub] flink issue #3905: [FLINK-6585] [table] Table examples are not runnable in I...

2017-05-18 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/3905 LGTM +1 to merge --- 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,

[jira] [Commented] (FLINK-6585) Table examples are not runnable in IDE

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015463#comment-16015463 ] ASF GitHub Bot commented on FLINK-6585: --- Github user fhueske commented on the issue:

[GitHub] flink pull request #3936: [FLINK-6618][table] Fix GroupWindowStringExpressio...

2017-05-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3936#discussion_r117196683 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/stream/table/stringexpr/GroupWindowStringExpressionTest.scala --- @@ -

[jira] [Created] (FLINK-6624) SharedBuffer#hashCode() uses multiplier in wrong way

2017-05-18 Thread Ted Yu (JIRA)
Ted Yu created FLINK-6624: - Summary: SharedBuffer#hashCode() uses multiplier in wrong way Key: FLINK-6624 URL: https://issues.apache.org/jira/browse/FLINK-6624 Project: Flink Issue Type: Bug

[jira] [Commented] (FLINK-6618) Fix GroupWindowStringExpressionTest testcase bug

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015464#comment-16015464 ] ASF GitHub Bot commented on FLINK-6618: --- Github user fhueske commented on a diff in

[jira] [Closed] (FLINK-6621) Legal check for 1.3.0 RC01 Release

2017-05-18 Thread Xiaowei Jiang (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiaowei Jiang closed FLINK-6621. Resolution: Fixed no need to open jira for checks > Legal check for 1.3.0 RC01 Release > --

[GitHub] flink issue #3869: [FLINK-6502] Add support ElasticsearchSink for DataSet

2017-05-18 Thread 397090770
Github user 397090770 commented on the issue: https://github.com/apache/flink/pull/3869 @tzulitai Thank you for your review very much. The current structuring of the ES output format as follow: ![elasticsearch_flink](https://cloud.githubusercontent.com/assets/5170878/26194858/69c

[jira] [Commented] (FLINK-6502) Add support ElasticsearchSink for DataSet

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6502?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015470#comment-16015470 ] ASF GitHub Bot commented on FLINK-6502: --- Github user 397090770 commented on the issu

[jira] [Assigned] (FLINK-6624) SharedBuffer#hashCode() uses multiplier in wrong way

2017-05-18 Thread Kostas Kloudas (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6624?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kostas Kloudas reassigned FLINK-6624: - Assignee: Kostas Kloudas > SharedBuffer#hashCode() uses multiplier in wrong way > ---

[GitHub] flink pull request #3854: [hotfix] [rat] Add exclusion for test state snapsh...

2017-05-18 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/3854#discussion_r117198127 --- Diff: flink-tests/src/test/java/org/apache/flink/test/checkpointing/utils/StatefulJobSavepointFrom11MigrationITCase.java --- @@ -129,7 +129,7 @@ public

[jira] [Commented] (FLINK-6624) SharedBuffer#hashCode() uses multiplier in wrong way

2017-05-18 Thread Kostas Kloudas (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015473#comment-16015473 ] Kostas Kloudas commented on FLINK-6624: --- Thanks for reporting this [~tedyu], I will

[GitHub] flink issue #3854: [hotfix] [rat] Add exclusion for test state snapshots

2017-05-18 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3854 Looks good to me, minus one minor comment. --- 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

[jira] [Updated] (FLINK-6612) ZooKeeperStateHandleStore does not guard against concurrent delete operations

2017-05-18 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6612?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann updated FLINK-6612: - Priority: Blocker (was: Critical) > ZooKeeperStateHandleStore does not guard against concurrent d

[GitHub] flink pull request #3715: [FLINK-6232][Table&SQL]Support proctime inner equi...

2017-05-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3715#discussion_r117200048 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/join/ProcTimeInnerJoin.scala --- @@ -0,0 +1,323 @@ +/* + * Licen

[jira] [Commented] (FLINK-6232) Support proctime inner equi-join between two streams in the SQL API

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015475#comment-16015475 ] ASF GitHub Bot commented on FLINK-6232: --- Github user fhueske commented on a diff in

[jira] [Created] (FLINK-6625) Flink removes HA job data when reaching JobStatus.FAILED

2017-05-18 Thread Till Rohrmann (JIRA)
Till Rohrmann created FLINK-6625: Summary: Flink removes HA job data when reaching JobStatus.FAILED Key: FLINK-6625 URL: https://issues.apache.org/jira/browse/FLINK-6625 Project: Flink Issue

[GitHub] flink pull request #3938: [FLINK-6624] [cep] Fix SharedBuffer#hashCode().

2017-05-18 Thread kl0u
GitHub user kl0u opened a pull request: https://github.com/apache/flink/pull/3938 [FLINK-6624] [cep] Fix SharedBuffer#hashCode(). You can merge this pull request into a Git repository by running: $ git pull https://github.com/kl0u/flink cep-hash Alternatively you can review a

[jira] [Commented] (FLINK-6624) SharedBuffer#hashCode() uses multiplier in wrong way

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015487#comment-16015487 ] ASF GitHub Bot commented on FLINK-6624: --- GitHub user kl0u opened a pull request:

[jira] [Commented] (FLINK-6574) Support nested catalogs in ExternalCatalog

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6574?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015488#comment-16015488 ] ASF GitHub Bot commented on FLINK-6574: --- Github user KurtYoung commented on a diff i

[GitHub] flink pull request #3913: [FLINK-6574] Support nested catalogs in ExternalCa...

2017-05-18 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/3913#discussion_r117201965 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/catalog/ExternalCatalogTable.scala --- @@ -37,25 +36,10 @@ import org.apache.f

[GitHub] flink pull request #3715: [FLINK-6232][Table&SQL]Support proctime inner equi...

2017-05-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3715#discussion_r117202009 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/join/JoinUtil.scala --- @@ -0,0 +1,488 @@ +/* + * Licensed to th

[GitHub] flink pull request #3931: [FLINK-6031][yarn] Add config parameter for user-j...

2017-05-18 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/3931#discussion_r117199375 --- Diff: docs/setup/yarn_setup.md --- @@ -245,6 +245,18 @@ Note: You can use a different configuration directory per job by setting the env Note

[GitHub] flink pull request #3938: [FLINK-6624] [cep] Fix SharedBuffer#hashCode().

2017-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3938 --- 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 enab

[jira] [Commented] (FLINK-6031) Add parameter for per job yarn clusters to control whether the user code jar is included into the system classloader.

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015492#comment-16015492 ] ASF GitHub Bot commented on FLINK-6031: --- Github user rmetzger commented on a diff in

[jira] [Commented] (FLINK-6031) Add parameter for per job yarn clusters to control whether the user code jar is included into the system classloader.

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015490#comment-16015490 ] ASF GitHub Bot commented on FLINK-6031: --- Github user rmetzger commented on a diff in

[GitHub] flink pull request #3931: [FLINK-6031][yarn] Add config parameter for user-j...

2017-05-18 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/3931#discussion_r117199176 --- Diff: flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java --- @@ -665,58 +685,52 @@ public ApplicationReport startAppMast

[GitHub] flink pull request #3931: [FLINK-6031][yarn] Add config parameter for user-j...

2017-05-18 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/3931#discussion_r117198446 --- Diff: flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java --- @@ -200,6 +214,17 @@ public void setTaskManagerMemory(int m

[jira] [Commented] (FLINK-6031) Add parameter for per job yarn clusters to control whether the user code jar is included into the system classloader.

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015491#comment-16015491 ] ASF GitHub Bot commented on FLINK-6031: --- Github user rmetzger commented on a diff in

[jira] [Commented] (FLINK-6618) Fix GroupWindowStringExpressionTest testcase bug

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015493#comment-16015493 ] ASF GitHub Bot commented on FLINK-6618: --- Github user sunjincheng121 commented on a d

[jira] [Commented] (FLINK-6232) Support proctime inner equi-join between two streams in the SQL API

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015489#comment-16015489 ] ASF GitHub Bot commented on FLINK-6232: --- Github user fhueske commented on a diff in

[jira] [Commented] (FLINK-6624) SharedBuffer#hashCode() uses multiplier in wrong way

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015494#comment-16015494 ] ASF GitHub Bot commented on FLINK-6624: --- Github user asfgit closed the pull request

[GitHub] flink pull request #3936: [FLINK-6618][table] Fix GroupWindowStringExpressio...

2017-05-18 Thread sunjincheng121
Github user sunjincheng121 commented on a diff in the pull request: https://github.com/apache/flink/pull/3936#discussion_r117202137 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/stream/table/stringexpr/GroupWindowStringExpressionTest.scala ---

[GitHub] flink issue #3938: [FLINK-6624] [cep] Fix SharedBuffer#hashCode().

2017-05-18 Thread tedyu
Github user tedyu commented on the issue: https://github.com/apache/flink/pull/3938 Lgtm --- 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

[jira] [Commented] (FLINK-6624) SharedBuffer#hashCode() uses multiplier in wrong way

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015495#comment-16015495 ] ASF GitHub Bot commented on FLINK-6624: --- Github user tedyu commented on the issue:

[GitHub] flink issue #3938: [FLINK-6624] [cep] Fix SharedBuffer#hashCode().

2017-05-18 Thread kl0u
Github user kl0u commented on the issue: https://github.com/apache/flink/pull/3938 Merging this! --- 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

[GitHub] flink issue #3901: [FLINK-6288] fix FlinkKafkaProducer's custom Partitioner ...

2017-05-18 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3901 @zjureel I'll be merging this only for 1.4-SNAPSHOT. The main reason is that there is a lot of deprecation / API change with this, and I'll like to ship it along with other API changes we had in min

[jira] [Closed] (FLINK-6624) SharedBuffer#hashCode() uses multiplier in wrong way

2017-05-18 Thread Kostas Kloudas (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6624?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kostas Kloudas closed FLINK-6624. - Resolution: Fixed > SharedBuffer#hashCode() uses multiplier in wrong way > ---

[jira] [Commented] (FLINK-6624) SharedBuffer#hashCode() uses multiplier in wrong way

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015498#comment-16015498 ] ASF GitHub Bot commented on FLINK-6624: --- Github user kl0u commented on the issue:

[jira] [Commented] (FLINK-6288) FlinkKafkaProducer's custom Partitioner is always invoked with number of partitions of default topic

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015499#comment-16015499 ] ASF GitHub Bot commented on FLINK-6288: --- Github user tzulitai commented on the issue

[jira] [Created] (FLINK-6626) Unifying lifecycle management of SubmittedJobGraph- and CompletedCheckpointStore

2017-05-18 Thread Till Rohrmann (JIRA)
Till Rohrmann created FLINK-6626: Summary: Unifying lifecycle management of SubmittedJobGraph- and CompletedCheckpointStore Key: FLINK-6626 URL: https://issues.apache.org/jira/browse/FLINK-6626 Projec

[GitHub] flink issue #3936: [FLINK-6618][table] Fix GroupWindowStringExpressionTest t...

2017-05-18 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/3936 Thanks for the update. +1 to merge --- 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 a

[jira] [Commented] (FLINK-6618) Fix GroupWindowStringExpressionTest testcase bug

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015503#comment-16015503 ] ASF GitHub Bot commented on FLINK-6618: --- Github user fhueske commented on the issue:

[GitHub] flink pull request #3931: [FLINK-6031][yarn] Add config parameter for user-j...

2017-05-18 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/3931#discussion_r117204403 --- Diff: docs/setup/yarn_setup.md --- @@ -245,6 +245,18 @@ Note: You can use a different configuration directory per job by setting the env Note

[jira] [Commented] (FLINK-6031) Add parameter for per job yarn clusters to control whether the user code jar is included into the system classloader.

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015505#comment-16015505 ] ASF GitHub Bot commented on FLINK-6031: --- Github user rmetzger commented on a diff in

[GitHub] flink pull request #3931: [FLINK-6031][yarn] Add config parameter for user-j...

2017-05-18 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/3931#discussion_r117204729 --- Diff: flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java --- @@ -665,58 +685,52 @@ public ApplicationReport startAppMaster

[jira] [Commented] (FLINK-6031) Add parameter for per job yarn clusters to control whether the user code jar is included into the system classloader.

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015509#comment-16015509 ] ASF GitHub Bot commented on FLINK-6031: --- Github user zentol commented on a diff in t

[GitHub] flink pull request #3715: [FLINK-6232][Table&SQL]Support proctime inner equi...

2017-05-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3715#discussion_r117205051 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/rules/datastream/DataStreamJoinRule.scala --- @@ -0,0 +1,69 @@ +/*

[jira] [Commented] (FLINK-6232) Support proctime inner equi-join between two streams in the SQL API

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015510#comment-16015510 ] ASF GitHub Bot commented on FLINK-6232: --- Github user fhueske commented on a diff in

[GitHub] flink pull request #3928: [FLINK-6608] [security, config] Relax Kerberos log...

2017-05-18 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/3928#discussion_r117205738 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/security/SecurityUtils.java --- @@ -230,10 +230,15 @@ private void validate() {

[GitHub] flink issue #3901: [FLINK-6288] fix FlinkKafkaProducer's custom Partitioner ...

2017-05-18 Thread gyfora
Github user gyfora commented on the issue: https://github.com/apache/flink/pull/3901 Well, I can understand it but will mean that we have to keep running with a custom build because there is no way to work around this nicely. --- If your project is set up for it, you can reply to thi

[jira] [Commented] (FLINK-6608) Relax Kerberos login contexts parsing by trimming whitespaces in contexts list

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015513#comment-16015513 ] ASF GitHub Bot commented on FLINK-6608: --- Github user tzulitai commented on a diff in

[jira] [Commented] (FLINK-6288) FlinkKafkaProducer's custom Partitioner is always invoked with number of partitions of default topic

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015515#comment-16015515 ] ASF GitHub Bot commented on FLINK-6288: --- Github user gyfora commented on the issue:

[GitHub] flink issue #3901: [FLINK-6288] fix FlinkKafkaProducer's custom Partitioner ...

2017-05-18 Thread gyfora
Github user gyfora commented on the issue: https://github.com/apache/flink/pull/3901 On the other hand this is potentially causing major data skew or errors for any people who are using the dynamic topics (they might not even be aware of it). --- If your project is set up for it, yo

[jira] [Commented] (FLINK-6288) FlinkKafkaProducer's custom Partitioner is always invoked with number of partitions of default topic

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015517#comment-16015517 ] ASF GitHub Bot commented on FLINK-6288: --- Github user gyfora commented on the issue:

[GitHub] flink pull request #3915: [FLINK-6352] Support to use timestamp to set the i...

2017-05-18 Thread zjureel
Github user zjureel commented on a diff in the pull request: https://github.com/apache/flink/pull/3915#discussion_r117206216 --- Diff: flink-connectors/flink-connector-kafka-0.8/src/main/java/org/apache/flink/streaming/connectors/kafka/FlinkKafkaConsumer08.java --- @@ -187,31 +191

[jira] [Commented] (FLINK-6352) FlinkKafkaConsumer should support to use timestamp to set up start offset

2017-05-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015518#comment-16015518 ] ASF GitHub Bot commented on FLINK-6352: --- Github user zjureel commented on a diff in

[GitHub] flink issue #3928: [FLINK-6608] [security, config] Relax Kerberos login cont...

2017-05-18 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3928 Thanks for the reviews @sunjincheng121 @EronWright. I personally prefer to stick with the regex approach. But really either way is fine :) Don't really expect any test errors on this, bu

  1   2   3   >