[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245421#comment-16245421 ] ASF GitHub Bot commented on FLINK-7704: --- Github user tillrohrmann commented on the issue:

[GitHub] flink issue #4978: [FLINK-7704][hotfix][flip6] Fix JobPlanInfoTest package p...

2017-11-09 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4978 LGTM. Thanks for your contribution @yew1eb. Merging. ---

[jira] [Commented] (FLINK-8025) Let DashboardConfigHandler extend from AbstractRestHandler

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245428#comment-16245428 ] ASF GitHub Bot commented on FLINK-8025: --- Github user tillrohrmann commented on the issue:

[jira] [Commented] (FLINK-8026) Let ClusterConfigHandler extend from AbstractRestHandler

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245430#comment-16245430 ] ASF GitHub Bot commented on FLINK-8026: --- Github user tillrohrmann commented on the issue:

[GitHub] flink issue #4984: [FLINK-8026] Let ClusterConfigHandler directly extend Abs...

2017-11-09 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4984 Thanks for your review @zentol. Merging. ---

[jira] [Commented] (FLINK-7419) Shade jackson dependency in flink-avro

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7419?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245495#comment-16245495 ] ASF GitHub Bot commented on FLINK-7419: --- Github user aljoscha commented on the issue:

[GitHub] flink issue #4981: [FLINK-7419][build][avro] Relocate jackson in flink-dist

2017-11-09 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/4981 Ok, sounds good. 👌 ---

[jira] [Commented] (FLINK-8008) PojoTypeInfo should sort fields fields based on boolean

2017-11-09 Thread Fabian Hueske (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245509#comment-16245509 ] Fabian Hueske commented on FLINK-8008: -- Depends on the TableSink. {{CsvTableSink}} will write the

[jira] [Commented] (FLINK-7517) let NettyBufferPool extend PooledByteBufAllocator

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245348#comment-16245348 ] ASF GitHub Bot commented on FLINK-7517: --- Github user zhijiangW commented on the issue:

[jira] [Comment Edited] (FLINK-7003) "select * from" in Flink SQL should not flatten all fields in the table by default

2017-11-09 Thread Shuyi Chen (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245361#comment-16245361 ] Shuyi Chen edited comment on FLINK-7003 at 11/9/17 8:28 AM: added [pull

[jira] [Commented] (FLINK-7003) "select * from" in Flink SQL should not flatten all fields in the table by default

2017-11-09 Thread Shuyi Chen (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245361#comment-16245361 ] Shuyi Chen commented on FLINK-7003: --- added [link pull

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245484#comment-16245484 ] ASF GitHub Bot commented on FLINK-8005: --- Github user aljoscha commented on a diff in the pull

[GitHub] flink pull request #4980: [FLINK-8005] [runtime] Set user code class loader ...

2017-11-09 Thread aljoscha
Github user aljoscha commented on a diff in the pull request: https://github.com/apache/flink/pull/4980#discussion_r149926517 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/taskmanager/TaskAsyncCallTest.java --- @@ -58,99 +59,144 @@ import org.junit.Before;

[jira] [Commented] (FLINK-8009) flink-dist pulls in flink-runtime's transitive avro/jackson dependency

2017-11-09 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8009?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245498#comment-16245498 ] Aljoscha Krettek commented on FLINK-8009: - It's a weird situation but if someone looks at the

[GitHub] flink issue #4983: [FLINK-8025] Let DashboardConfigHandler directly extend A...

2017-11-09 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4983 Thanks for your review @zentol. Merging. ---

[jira] [Commented] (FLINK-8024) Let ClusterOverviewHandler directly extend from AbstractRestHandler

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245426#comment-16245426 ] ASF GitHub Bot commented on FLINK-8024: --- Github user tillrohrmann commented on the issue:

[GitHub] flink issue #4982: [FLINK-8024] Let ClusterOverviewHandler directly extend f...

2017-11-09 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4982 Thanks for your review @zentol. Merging. ---

[jira] [Commented] (FLINK-8000) Sort REST handler URLs in RestServerEndpoint

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245425#comment-16245425 ] ASF GitHub Bot commented on FLINK-8000: --- Github user tillrohrmann commented on the issue:

[GitHub] flink issue #4958: [FLINK-8000] Sort Rest handler URLS in RestServerEndpoint

2017-11-09 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4958 Will remove the comment and then merge the PR. Thanks for your review @zentol. ---

[jira] [Commented] (FLINK-5633) ClassCastException: X cannot be cast to X when re-submitting a job.

2017-11-09 Thread Erik van Oosten (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5633?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245474#comment-16245474 ] Erik van Oosten commented on FLINK-5633: bq. Just curious, why are you creating a new reader for

[GitHub] flink pull request #4594: [FLINK-7517][network] let NettyBufferPool extend P...

2017-11-09 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4594#discussion_r149889426 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/netty/NettyBufferPool.java --- @@ -52,51 +48,61 @@ /** Configured chunk

[GitHub] flink issue #4594: [FLINK-7517][network] let NettyBufferPool extend PooledBy...

2017-11-09 Thread zhijiangW
Github user zhijiangW commented on the issue: https://github.com/apache/flink/pull/4594 This improvement is indeed very clean and reduce many redundant `Override` methods by extending `PooledByteBufAllocator` directly. But I am still confused of one thing. In previous way,

[jira] [Commented] (FLINK-7517) let NettyBufferPool extend PooledByteBufAllocator

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245347#comment-16245347 ] ASF GitHub Bot commented on FLINK-7517: --- Github user NicoK commented on a diff in the pull request:

[GitHub] flink pull request #4989: [Flink 7003] [Table API & SQL] use PEEK_FIELDS_NO_...

2017-11-09 Thread suez1224
GitHub user suez1224 opened a pull request: https://github.com/apache/flink/pull/4989 [Flink 7003] [Table API & SQL] use PEEK_FIELDS_NO_EXPAND for CompositeRelDataType ## What is the purpose of the change This pull request change CompositeRelDataType to use the newly added

[jira] [Commented] (FLINK-7977) bump version of compatibility check for Flink 1.4

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7977?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245496#comment-16245496 ] ASF GitHub Bot commented on FLINK-7977: --- Github user aljoscha commented on the issue:

[GitHub] flink issue #4945: [FLINK-7977][build] bump version of compatibility check f...

2017-11-09 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/4945 @greghogan That makes sense, I didn't think about added APIs. ---

[GitHub] flink pull request #4978: [FLINK-7704][hotfix][flip6] Fix JobPlanInfoTest pa...

2017-11-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4978 ---

[jira] [Commented] (FLINK-8008) PojoTypeInfo should sort fields fields based on boolean

2017-11-09 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245504#comment-16245504 ] Aljoscha Krettek commented on FLINK-8008: - [~fhueske] Is there a way of specifying the order of

[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245423#comment-16245423 ] ASF GitHub Bot commented on FLINK-7704: --- Github user asfgit closed the pull request at:

[jira] [Commented] (FLINK-8036) Consider using gradle to build Flink

2017-11-09 Thread Fabian Hueske (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8036?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245390#comment-16245390 ] Fabian Hueske commented on FLINK-8036: -- I think this will be very hard. Flink's build management

[jira] [Commented] (FLINK-8024) Let ClusterOverviewHandler directly extend from AbstractRestHandler

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245442#comment-16245442 ] ASF GitHub Bot commented on FLINK-8024: --- Github user asfgit closed the pull request at:

[jira] [Commented] (FLINK-8000) Sort REST handler URLs in RestServerEndpoint

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245441#comment-16245441 ] ASF GitHub Bot commented on FLINK-8000: --- Github user asfgit closed the pull request at:

[GitHub] flink pull request #4984: [FLINK-8026] Let ClusterConfigHandler directly ext...

2017-11-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4984 ---

[jira] [Closed] (FLINK-8026) Let ClusterConfigHandler extend from AbstractRestHandler

2017-11-09 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8026?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann closed FLINK-8026. Resolution: Fixed Fixed via 541fe43663d2a24b1ae66bc2b5228c49dfd43e7b > Let ClusterConfigHandler

[jira] [Commented] (FLINK-8026) Let ClusterConfigHandler extend from AbstractRestHandler

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245444#comment-16245444 ] ASF GitHub Bot commented on FLINK-8026: --- Github user asfgit closed the pull request at:

[jira] [Commented] (FLINK-8025) Let DashboardConfigHandler extend from AbstractRestHandler

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245443#comment-16245443 ] ASF GitHub Bot commented on FLINK-8025: --- Github user asfgit closed the pull request at:

[GitHub] flink pull request #4983: [FLINK-8025] Let DashboardConfigHandler directly e...

2017-11-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4983 ---

[GitHub] flink pull request #4958: [FLINK-8000] Sort Rest handler URLS in RestServerE...

2017-11-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4958 ---

[jira] [Closed] (FLINK-8025) Let DashboardConfigHandler extend from AbstractRestHandler

2017-11-09 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8025?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann closed FLINK-8025. Resolution: Fixed Fixed via fa967dfc0cddec0a5d0c7d76e564a8307ec3fb35 > Let DashboardConfigHandler

[GitHub] flink pull request #4982: [FLINK-8024] Let ClusterOverviewHandler directly e...

2017-11-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4982 ---

[jira] [Commented] (FLINK-7980) Bump joda-time to 2.9.9

2017-11-09 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7980?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245492#comment-16245492 ] Aljoscha Krettek commented on FLINK-7980: - Yep, I think getting rid of it makes sense. > Bump

[jira] [Commented] (FLINK-8008) PojoTypeInfo should sort fields fields based on boolean

2017-11-09 Thread Fabian Hueske (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245513#comment-16245513 ] Fabian Hueske commented on FLINK-8008: -- Except for nested field, {{CsvTableSink}} simply calls

[jira] [Closed] (FLINK-8000) Sort REST handler URLs in RestServerEndpoint

2017-11-09 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8000?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann closed FLINK-8000. Resolution: Fixed Fix Version/s: 1.5.0 Fixed via 34fdf569d7defb4393849fb9ecb2763b14532cc6 >

[jira] [Closed] (FLINK-8024) Let ClusterOverviewHandler directly extend from AbstractRestHandler

2017-11-09 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8024?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann closed FLINK-8024. Resolution: Fixed Fixed via 34fdf569d7defb4393849fb9ecb2763b14532cc6 > Let ClusterOverviewHandler

[GitHub] flink issue #4976: [FLINK-8017] Fix High availability cluster-id key in docu...

2017-11-09 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/4976 Thanks for spotting and fixing this! 👍 I'm merging. ---

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245630#comment-16245630 ] ASF GitHub Bot commented on FLINK-8005: --- Github user GJL commented on the issue:

[GitHub] flink issue #4980: [FLINK-8005] [runtime] Set user code class loader before ...

2017-11-09 Thread GJL
Github user GJL commented on the issue: https://github.com/apache/flink/pull/4980 There is only one thread dispatching the calls: ``` executor = Executors.newSingleThreadExecutor( new DispatcherThreadFactory(TASK_THREADS_GROUP,

[jira] [Commented] (FLINK-7765) Enable dependency convergence

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245631#comment-16245631 ] ASF GitHub Bot commented on FLINK-7765: --- Github user aljoscha commented on the issue:

[jira] [Updated] (FLINK-8040) Test instability ResourceGuard#testCloseBlockIfAcquired

2017-11-09 Thread Gary Yao (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8040?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gary Yao updated FLINK-8040: Description: Test {{ResourceGuard#testCloseBlockIfAcquired()}} fails sporadically with {noformat}

[jira] [Created] (FLINK-8040) Test instability ResourceGuard#testCloseBlockIfAcquired

2017-11-09 Thread Gary Yao (JIRA)
Gary Yao created FLINK-8040: --- Summary: Test instability ResourceGuard#testCloseBlockIfAcquired Key: FLINK-8040 URL: https://issues.apache.org/jira/browse/FLINK-8040 Project: Flink Issue Type: Bug

[GitHub] flink issue #4989: [Flink 7003] [Table API & SQL] use PEEK_FIELDS_NO_EXPAND ...

2017-11-09 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/4989 Thanks for the PR @suez1224. I will merge it. Thanks, Fabian ---

[jira] [Commented] (FLINK-7765) Enable dependency convergence

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245753#comment-16245753 ] ASF GitHub Bot commented on FLINK-7765: --- Github user aljoscha commented on the issue:

[jira] [Closed] (FLINK-7886) Enable dependency convergence for flink-core

2017-11-09 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7886?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aljoscha Krettek closed FLINK-7886. --- Resolution: Fixed Implemented on master in bdbca37b01254aef9ddd6943d34dab1838c2a9f1 > Enable

[GitHub] flink pull request #4939: [FLINK-4228][yarn/s3a] fix yarn resource upload s3...

2017-11-09 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/4939#discussion_r149983029 --- Diff: flink-yarn/pom.xml --- @@ -153,6 +159,63 @@ under the License.

[jira] [Commented] (FLINK-4228) YARN artifact upload does not work with S3AFileSystem

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245769#comment-16245769 ] ASF GitHub Bot commented on FLINK-4228: --- Github user tillrohrmann commented on a diff in the pull

[jira] [Commented] (FLINK-4228) YARN artifact upload does not work with S3AFileSystem

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245768#comment-16245768 ] ASF GitHub Bot commented on FLINK-4228: --- Github user tillrohrmann commented on a diff in the pull

[GitHub] flink pull request #4939: [FLINK-4228][yarn/s3a] fix yarn resource upload s3...

2017-11-09 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/4939#discussion_r149983025 --- Diff: flink-yarn/pom.xml --- @@ -153,6 +159,63 @@ under the License.

[jira] [Assigned] (FLINK-8040) Test instability ResourceGuard#testCloseBlockIfAcquired

2017-11-09 Thread Gary Yao (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8040?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gary Yao reassigned FLINK-8040: --- Assignee: Stefan Richter > Test instability ResourceGuard#testCloseBlockIfAcquired >

[GitHub] flink pull request #4980: [FLINK-8005] [runtime] Set user code class loader ...

2017-11-09 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/4980#discussion_r149985097 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/taskmanager/TaskAsyncCallTest.java --- @@ -277,8 +287,12 @@ public void invoke() throws

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245593#comment-16245593 ] ASF GitHub Bot commented on FLINK-8005: --- Github user GJL commented on a diff in the pull request:

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245592#comment-16245592 ] ASF GitHub Bot commented on FLINK-8005: --- Github user GJL commented on a diff in the pull request:

[GitHub] flink pull request #4980: [FLINK-8005] [runtime] Set user code class loader ...

2017-11-09 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/4980#discussion_r149954849 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/taskmanager/TaskAsyncCallTest.java --- @@ -254,12 +300,10 @@ else if (this.error == null) {

[GitHub] flink issue #4777: [FLINK-7765] Enable dependency convergence

2017-11-09 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/4777 I'll merge, yes, what are the follow-up issues you created? ---

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245765#comment-16245765 ] ASF GitHub Bot commented on FLINK-8005: --- Github user kl0u commented on a diff in the pull request:

[GitHub] flink pull request #4939: [FLINK-4228][yarn/s3a] fix yarn resource upload s3...

2017-11-09 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/4939#discussion_r149980302 --- Diff: flink-yarn/pom.xml --- @@ -153,6 +159,63 @@ under the License.

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245781#comment-16245781 ] ASF GitHub Bot commented on FLINK-8005: --- Github user GJL commented on a diff in the pull request:

[jira] [Commented] (FLINK-4228) YARN artifact upload does not work with S3AFileSystem

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245767#comment-16245767 ] ASF GitHub Bot commented on FLINK-4228: --- Github user tillrohrmann commented on a diff in the pull

[GitHub] flink issue #4980: [FLINK-8005] [runtime] Set user code class loader before ...

2017-11-09 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/4980 Yes, but I think this is making an assumption about the internal implementation. If someone changes that the test could break/not test the right thing anymore. ---

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245681#comment-16245681 ] ASF GitHub Bot commented on FLINK-8005: --- Github user aljoscha commented on the issue:

[GitHub] flink issue #4777: [FLINK-7765] Enable dependency convergence

2017-11-09 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/4777 Thanks! Could you please close if GH doesn't auto-close? ---

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245776#comment-16245776 ] ASF GitHub Bot commented on FLINK-8005: --- Github user kl0u commented on a diff in the pull request:

[jira] [Commented] (FLINK-7979) Use Log.*(Object, Throwable) overload to log exceptions

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245575#comment-16245575 ] ASF GitHub Bot commented on FLINK-7979: --- Github user StephanEwen commented on the issue:

[GitHub] flink issue #4948: [FLINK-7979][minor] Use Log.*(Object, Throwable) overload...

2017-11-09 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/4948 Thanks for understanding! ---

[GitHub] flink issue #4777: [FLINK-7765] Enable dependency convergence

2017-11-09 Thread pnowojski
Github user pnowojski commented on the issue: https://github.com/apache/flink/pull/4777 Thanks! ---

[jira] [Comment Edited] (FLINK-6022) Don't serialise Schema when serialising Avro GenericRecord

2017-11-09 Thread Stephan Ewen (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245577#comment-16245577 ] Stephan Ewen edited comment on FLINK-6022 at 11/9/17 12:48 PM: --- We are not

[jira] [Closed] (FLINK-8017) High availability cluster-id option incorrect in documentation

2017-11-09 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8017?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aljoscha Krettek closed FLINK-8017. --- Resolution: Fixed Fixed on release-1.4 in 02a19a14fad1ef928038f4971bdcacf4d0642d88 Fixed on

[jira] [Commented] (FLINK-8017) High availability cluster-id option incorrect in documentation

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245603#comment-16245603 ] ASF GitHub Bot commented on FLINK-8017: --- Github user aljoscha commented on the issue:

[jira] [Updated] (FLINK-8017) High availability cluster-id option incorrect in documentation

2017-11-09 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8017?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aljoscha Krettek updated FLINK-8017: Fix Version/s: 1.4.0 > High availability cluster-id option incorrect in documentation >

[GitHub] flink issue #4976: [FLINK-8017] Fix High availability cluster-id key in docu...

2017-11-09 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/4976 Could you please close this if it doesn't auto-close? ---

[jira] [Updated] (FLINK-8040) Test instability ResourceGuard#testCloseBlockIfAcquired

2017-11-09 Thread Gary Yao (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8040?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gary Yao updated FLINK-8040: Description: Test {{ResourceGuard#testCloseBlockIfAcquired()}} fails sporadically with {noformat}

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245773#comment-16245773 ] ASF GitHub Bot commented on FLINK-8005: --- Github user GJL commented on a diff in the pull request:

[GitHub] flink pull request #4980: [FLINK-8005] [runtime] Set user code class loader ...

2017-11-09 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/4980#discussion_r149983878 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/taskmanager/TaskAsyncCallTest.java --- @@ -277,8 +287,12 @@ public void invoke() throws

[jira] [Commented] (FLINK-7765) Enable dependency convergence

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245810#comment-16245810 ] ASF GitHub Bot commented on FLINK-7765: --- Github user pnowojski closed the pull request at:

[jira] [Commented] (FLINK-7765) Enable dependency convergence

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245808#comment-16245808 ] ASF GitHub Bot commented on FLINK-7765: --- Github user pnowojski commented on the issue:

[GitHub] flink pull request #4980: [FLINK-8005] [runtime] Set user code class loader ...

2017-11-09 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/4980#discussion_r149947069 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/taskmanager/TaskAsyncCallTest.java --- @@ -254,12 +300,10 @@ else if (this.error == null) {

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245569#comment-16245569 ] ASF GitHub Bot commented on FLINK-8005: --- Github user kl0u commented on a diff in the pull request:

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245570#comment-16245570 ] ASF GitHub Bot commented on FLINK-8005: --- Github user kl0u commented on a diff in the pull request:

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245572#comment-16245572 ] ASF GitHub Bot commented on FLINK-8005: --- Github user kl0u commented on a diff in the pull request:

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245571#comment-16245571 ] ASF GitHub Bot commented on FLINK-8005: --- Github user kl0u commented on a diff in the pull request:

[jira] [Commented] (FLINK-6022) Don't serialise Schema when serialising Avro GenericRecord

2017-11-09 Thread Stephan Ewen (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245577#comment-16245577 ] Stephan Ewen commented on FLINK-6022: - We are not serializing the schema in the Avro Serializer. If

[GitHub] flink pull request #4777: [FLINK-7765] Enable dependency convergence

2017-11-09 Thread pnowojski
Github user pnowojski closed the pull request at: https://github.com/apache/flink/pull/4777 ---

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245825#comment-16245825 ] ASF GitHub Bot commented on FLINK-8005: --- Github user aljoscha commented on a diff in the pull

[GitHub] flink pull request #4980: [FLINK-8005] [runtime] Set user code class loader ...

2017-11-09 Thread aljoscha
Github user aljoscha commented on a diff in the pull request: https://github.com/apache/flink/pull/4980#discussion_r149993559 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/taskmanager/TaskAsyncCallTest.java --- @@ -277,8 +287,12 @@ public void invoke() throws

[GitHub] flink pull request #4980: [FLINK-8005] [runtime] Set user code class loader ...

2017-11-09 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/4980#discussion_r149954576 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/taskmanager/TaskAsyncCallTest.java --- @@ -58,99 +59,144 @@ import org.junit.Before;

[GitHub] flink pull request #4980: [FLINK-8005] [runtime] Set user code class loader ...

2017-11-09 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/4980#discussion_r149954606 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/taskmanager/DispatcherThreadFactory.java --- @@ -29,21 +31,41 @@ private final

[jira] [Assigned] (FLINK-8021) End-to-end tests may not shutdown cluster on failure

2017-11-09 Thread Aljoscha Krettek (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8021?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aljoscha Krettek reassigned FLINK-8021: --- Assignee: Aljoscha Krettek > End-to-end tests may not shutdown cluster on failure >

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245600#comment-16245600 ] ASF GitHub Bot commented on FLINK-8005: --- Github user GJL commented on the issue:

[jira] [Commented] (FLINK-8017) High availability cluster-id option incorrect in documentation

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245599#comment-16245599 ] ASF GitHub Bot commented on FLINK-8017: --- Github user aljoscha commented on the issue:

[GitHub] flink issue #4980: [FLINK-8005] [runtime] Set user code class loader before ...

2017-11-09 Thread GJL
Github user GJL commented on the issue: https://github.com/apache/flink/pull/4980 I addressed the comments. Let's wait for Travis and let me know if something else needs to be changed. @aljoscha @kl0u ---

[jira] [Commented] (FLINK-8005) Snapshotting FlinkKafkaProducer011 fails due to ClassLoader issues

2017-11-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245623#comment-16245623 ] ASF GitHub Bot commented on FLINK-8005: --- Github user aljoscha commented on the issue:

  1   2   >