[jira] [Commented] (FLINK-12818) Improve stability of twoInputMapSink benchmark

2019-08-22 Thread Haibo Sun (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-12818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913989#comment-16913989
 ] 

Haibo Sun commented on FLINK-12818:
---

{quote}Alternatively until this is confirmed as a production code issue, we can 
just try to live with this benchmark instability and relay on long term trends.
{quote}
+1.   I think we can adopt this way first.

 
{quote}I think if we want to investigate this further, we would need to print 
compiled byte code and try to understand the difference.
{quote}
I agree. I'll try when I have time.

> Improve stability of twoInputMapSink benchmark
> --
>
> Key: FLINK-12818
> URL: https://issues.apache.org/jira/browse/FLINK-12818
> Project: Flink
>  Issue Type: Sub-task
>  Components: Benchmarks
>Reporter: Piotr Nowojski
>Priority: Critical
> Attachments: RecordWriter-emit.png
>
>
> The {{twoInputMapSink}} benchmark is very unstable over time:
> http://codespeed.dak8s.net:8000/timeline/#/?exe=1&ben=twoInputMapSink&env=2&revs=200&equid=off&quarts=on&extr=on
> It should be fixed, otherwise the benchmark can not be used.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] lirui-apache commented on issue #9502: [FLINK-13814][hive] HiveTableSink should strip quotes from partition …

2019-08-22 Thread GitBox
lirui-apache commented on issue #9502: [FLINK-13814][hive] HiveTableSink should 
strip quotes from partition …
URL: https://github.com/apache/flink/pull/9502#issuecomment-524194414
 
 
   PR updated to get unquoted partition values in `RichSqlInsert`, so that we 
don't have to do it in HiveTableSink.
   @danny0405 better if you can also take a look. Thanks.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9517: [FLINK-13779][metrics] PrometheusPushGatewayReporter support push metrics with groupingKey

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9517: [FLINK-13779][metrics] 
PrometheusPushGatewayReporter support push metrics with groupingKey
URL: https://github.com/apache/flink/pull/9517#issuecomment-524187390
 
 
   
   ## CI report:
   
   * cdbd732fd99f2d2b5ad3cac2d119ef201079aafa : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124323759)
   * 19d2487dac6bcdd9308ab9948d610b3b3edf5b8f : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124325244)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9516: [FLINK-13815][State Backends] Implement the SpaceAllocator

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9516: [FLINK-13815][State Backends] 
Implement the SpaceAllocator
URL: https://github.com/apache/flink/pull/9516#issuecomment-524185347
 
 
   
   ## CI report:
   
   * bde14e569d1d3c7ada5054bcf1acea1c8b41bc2d : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124323122)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9502: [FLINK-13814][hive] HiveTableSink should strip quotes from partition …

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9502: [FLINK-13814][hive] HiveTableSink 
should strip quotes from partition …
URL: https://github.com/apache/flink/pull/9502#issuecomment-523778174
 
 
   
   ## CI report:
   
   * 22278924cd49e4fab6eabb9bf4f6ec8310894808 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124170593)
   * 5ad14b3392b439f2b63b9fb079bf5546c874607d : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124325231)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9517: [FLINK-13779][metrics] PrometheusPushGatewayReporter support push metrics with groupingKey

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9517: [FLINK-13779][metrics] 
PrometheusPushGatewayReporter support push metrics with groupingKey
URL: https://github.com/apache/flink/pull/9517#issuecomment-524187390
 
 
   
   ## CI report:
   
   * cdbd732fd99f2d2b5ad3cac2d119ef201079aafa : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124323759)
   * 19d2487dac6bcdd9308ab9948d610b3b3edf5b8f : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9502: [FLINK-13814][hive] HiveTableSink should strip quotes from partition …

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9502: [FLINK-13814][hive] HiveTableSink 
should strip quotes from partition …
URL: https://github.com/apache/flink/pull/9502#issuecomment-523778174
 
 
   
   ## CI report:
   
   * 22278924cd49e4fab6eabb9bf4f6ec8310894808 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124170593)
   * 5ad14b3392b439f2b63b9fb079bf5546c874607d : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Closed] (FLINK-13365) Add documentation for reusing upstream operations when multi-sink is applied

2019-08-22 Thread Jark Wu (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13365?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jark Wu closed FLINK-13365.
---
  Assignee: (was: Jing Zhang)
Resolution: Fixed

This has been introduced in FLINK-13354.

> Add documentation for reusing upstream operations when multi-sink is applied
> 
>
> Key: FLINK-13365
> URL: https://issues.apache.org/jira/browse/FLINK-13365
> Project: Flink
>  Issue Type: Sub-task
>  Components: Documentation, Table SQL / Planner
>Reporter: Jark Wu
>Priority: Major
> Fix For: 1.9.1
>
>
> Add documentation for reuse upstream operations when multi-sink
> - “Performance Tuning / Multi-Sink Optimization”: How to avoid executing 
> common operators repeatedly if there are multiple TableSinks in a job.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot edited a comment on issue #9416: [FLINK-13646][build system] Add basic ARM CI job definition

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9416: [FLINK-13646][build system] Add basic 
ARM CI job definition
URL: https://github.com/apache/flink/pull/9416#issuecomment-520285701
 
 
   
   ## CI report:
   
   * dbf749f0d0c3167265287b182dd1ea7b4aed84c1 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/122738631)
   * 141896dbe8372442f95f7b6425b23a7cbb4cb624 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/122757944)
   * df40969ada1c232fb00e6aea33f9a1c88b07855c : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124324441)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Closed] (FLINK-13364) Add documentation for setting resources in blink planner

2019-08-22 Thread Jark Wu (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jark Wu closed FLINK-13364.
---
Resolution: Fixed

This has been introduced in FLINK-13119.

> Add documentation for setting resources in blink planner
> 
>
> Key: FLINK-13364
> URL: https://issues.apache.org/jira/browse/FLINK-13364
> Project: Flink
>  Issue Type: Sub-task
>  Components: Documentation, Table SQL / Planner
>Reporter: Jark Wu
>Assignee: xupingyong
>Priority: Major
> Fix For: 1.9.1
>
>
> Add documentation for setting resources.
> - “Performance Tuning / Resource Setting”: how to set resources in blink 
> planner, explain the configs and how they work together.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Resolved] (FLINK-13105) Add documentation for blink planner's built-in functions

2019-08-22 Thread Jark Wu (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13105?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jark Wu resolved FLINK-13105.
-
Resolution: Fixed

master: c2b30d6a0c0c36ee85cf91c2b129302dea59ac70

1.9.1: 60c92bd77103651860e6d99d133aaffb463de11d

> Add documentation for blink planner's built-in functions
> 
>
> Key: FLINK-13105
> URL: https://issues.apache.org/jira/browse/FLINK-13105
> Project: Flink
>  Issue Type: Sub-task
>  Components: Documentation, Table SQL / Planner
>Reporter: Zhenghua Gao
>Assignee: Zhenghua Gao
>Priority: Critical
>  Labels: pull-request-available
> Fix For: 1.9.1
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> Blink planner intros some built-in functions which need to be documented.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot edited a comment on issue #9517: [FLINK-13779][metrics] PrometheusPushGatewayReporter support push metrics with groupingKey

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9517: [FLINK-13779][metrics] 
PrometheusPushGatewayReporter support push metrics with groupingKey
URL: https://github.com/apache/flink/pull/9517#issuecomment-524187390
 
 
   
   ## CI report:
   
   * cdbd732fd99f2d2b5ad3cac2d119ef201079aafa : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124323759)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9416: [FLINK-13646][build system] Add basic ARM CI job definition

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9416: [FLINK-13646][build system] Add basic 
ARM CI job definition
URL: https://github.com/apache/flink/pull/9416#issuecomment-520285701
 
 
   
   ## CI report:
   
   * dbf749f0d0c3167265287b182dd1ea7b4aed84c1 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/122738631)
   * 141896dbe8372442f95f7b6425b23a7cbb4cb624 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/122757944)
   * df40969ada1c232fb00e6aea33f9a1c88b07855c : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] asfgit closed pull request #9002: [FLINK-13105][table][doc] Add documentation for blink planner's built-in functions

2019-08-22 Thread GitBox
asfgit closed pull request #9002: [FLINK-13105][table][doc] Add documentation 
for blink planner's built-in functions
URL: https://github.com/apache/flink/pull/9002
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot commented on issue #9517: [FLINK-13779][metrics] PrometheusPushGatewayReporter support push metrics with groupingKey

2019-08-22 Thread GitBox
flinkbot commented on issue #9517: [FLINK-13779][metrics] 
PrometheusPushGatewayReporter support push metrics with groupingKey
URL: https://github.com/apache/flink/pull/9517#issuecomment-524187390
 
 
   
   ## CI report:
   
   * cdbd732fd99f2d2b5ad3cac2d119ef201079aafa : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9516: [FLINK-13815][State Backends] Implement the SpaceAllocator

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9516: [FLINK-13815][State Backends] 
Implement the SpaceAllocator
URL: https://github.com/apache/flink/pull/9516#issuecomment-524185347
 
 
   
   ## CI report:
   
   * bde14e569d1d3c7ada5054bcf1acea1c8b41bc2d : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124323122)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot commented on issue #9517: [FLINK-13779][metrics] PrometheusPushGatewayReporter support push metrics with groupingKey

2019-08-22 Thread GitBox
flinkbot commented on issue #9517: [FLINK-13779][metrics] 
PrometheusPushGatewayReporter support push metrics with groupingKey
URL: https://github.com/apache/flink/pull/9517#issuecomment-524185771
 
 
   Thanks a lot for your contribution to the Apache Flink project. I'm the 
@flinkbot. I help the community
   to review your pull request. We will use this comment to track the progress 
of the review.
   
   
   ## Automated Checks
   Last check on commit cdbd732fd99f2d2b5ad3cac2d119ef201079aafa (Fri Aug 23 
06:09:22 UTC 2019)
   
   **Warnings:**
* No documentation files were touched! Remember to keep the Flink docs up 
to date!
   
   
   Mention the bot in a comment to re-run the automated checks.
   ## Review Progress
   
   * ❓ 1. The [description] looks good.
   * ❓ 2. There is [consensus] that the contribution should go into to Flink.
   * ❓ 3. Needs [attention] from.
   * ❓ 4. The change fits into the overall [architecture].
   * ❓ 5. Overall code [quality] is good.
   
   Please see the [Pull Request Review 
Guide](https://flink.apache.org/contributing/reviewing-prs.html) for a full 
explanation of the review process.
The Bot is tracking the review progress through labels. Labels are applied 
according to the order of the review items. For consensus, approval by a Flink 
committer of PMC member is required Bot commands
 The @flinkbot bot supports the following commands:
   
- `@flinkbot approve description` to approve one or more aspects (aspects: 
`description`, `consensus`, `architecture` and `quality`)
- `@flinkbot approve all` to approve all aspects
- `@flinkbot approve-until architecture` to approve everything until 
`architecture`
- `@flinkbot attention @username1 [@username2 ..]` to require somebody's 
attention
- `@flinkbot disapprove architecture` to remove an approval you gave earlier
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Updated] (FLINK-13779) PrometheusPushGatewayReporter support push metrics with groupingKey

2019-08-22 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13779?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

ASF GitHub Bot updated FLINK-13779:
---
Labels: pull-request-available  (was: )

> PrometheusPushGatewayReporter support push metrics with groupingKey
> ---
>
> Key: FLINK-13779
> URL: https://issues.apache.org/jira/browse/FLINK-13779
> Project: Flink
>  Issue Type: New Feature
>  Components: Runtime / Metrics
>Reporter: Kaibo Zhou
>Assignee: Kaibo Zhou
>Priority: Minor
>  Labels: pull-request-available
>
> The Prometheus push gateway java SDK support send metrics with _groupingKey_, 
> see 
> [doc|https://prometheus.github.io/client_java/io/prometheus/client/exporter/PushGateway.html#push-io.prometheus.client.CollectorRegistry-java.lang.String-java.util.Map-].
>  
> This feature will make it convenient for users to identify, group or filter 
> their metrics by defining _groupingKey (optional)_. The user does not need to 
> configure this by default, and the default behavior remains the same.
>  



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot commented on issue #9516: [FLINK-13815][State Backends] Implement the SpaceAllocator

2019-08-22 Thread GitBox
flinkbot commented on issue #9516: [FLINK-13815][State Backends] Implement the 
SpaceAllocator
URL: https://github.com/apache/flink/pull/9516#issuecomment-524185347
 
 
   
   ## CI report:
   
   * bde14e569d1d3c7ada5054bcf1acea1c8b41bc2d : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] kaibozhou opened a new pull request #9517: [FLINK-13779][metrics] PrometheusPushGatewayReporter support push metrics with groupingKey

2019-08-22 Thread GitBox
kaibozhou opened a new pull request #9517: [FLINK-13779][metrics] 
PrometheusPushGatewayReporter support push metrics with groupingKey
URL: https://github.com/apache/flink/pull/9517
 
 
   ## What is the purpose of the change
   
   The pushgateway already support send metrics with 
[groupingKey](https://prometheus.github.io/client_java/io/prometheus/client/exporter/PushGateway.html#push-io.prometheus.client.CollectorRegistry-java.lang.String-java.util.Map-),
 PrometheusPushGatewayReporter can support push metrics with groupingKey, which 
will be convenient for users to identify, group or filter their metrics.
   
   ## Brief change log
   
 - *parser groupingKey from user's config: k1=v1,k2=v2 to Map*
 - *Use PushGateway Java SDK to push with groupingKey*
 - *Use PushGateway Java SDK to delete with groupingKey**
   
   
   ## Verifying this change
   Run *PrometheusPushGatewayReporterTest* to verify the groupingKey config.
   
   ## Does this pull request potentially affect one of the following parts:
   
 - Dependencies (does it add or upgrade a dependency): (no)
 - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (no)
 - The serializers: (no)
 - The runtime per-record code paths (performance sensitive): (no)
 - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
 - The S3 file system connector: (no)
   
   ## Documentation
   
 - Does this pull request introduce a new feature? (yes)
 - If yes, how is the feature documented? (docs will auto generated)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot commented on issue #9516: [FLINK-13815][State Backends] Implement the SpaceAllocator

2019-08-22 Thread GitBox
flinkbot commented on issue #9516: [FLINK-13815][State Backends] Implement the 
SpaceAllocator
URL: https://github.com/apache/flink/pull/9516#issuecomment-524185018
 
 
   Thanks a lot for your contribution to the Apache Flink project. I'm the 
@flinkbot. I help the community
   to review your pull request. We will use this comment to track the progress 
of the review.
   
   
   ## Automated Checks
   Last check on commit bde14e569d1d3c7ada5054bcf1acea1c8b41bc2d (Fri Aug 23 
06:05:58 UTC 2019)
   
   **Warnings:**
* **1 pom.xml files were touched**: Check for build and licensing issues.
* No documentation files were touched! Remember to keep the Flink docs up 
to date!
* **This pull request references an unassigned [Jira 
ticket](https://issues.apache.org/jira/browse/FLINK-13815).** According to the 
[code contribution 
guide](https://flink.apache.org/contributing/contribute-code.html), tickets 
need to be assigned before starting with the implementation work.
   
   
   Mention the bot in a comment to re-run the automated checks.
   ## Review Progress
   
   * ❓ 1. The [description] looks good.
   * ❓ 2. There is [consensus] that the contribution should go into to Flink.
   * ❓ 3. Needs [attention] from.
   * ❓ 4. The change fits into the overall [architecture].
   * ❓ 5. Overall code [quality] is good.
   
   Please see the [Pull Request Review 
Guide](https://flink.apache.org/contributing/reviewing-prs.html) for a full 
explanation of the review process.
The Bot is tracking the review progress through labels. Labels are applied 
according to the order of the review items. For consensus, approval by a Flink 
committer of PMC member is required Bot commands
 The @flinkbot bot supports the following commands:
   
- `@flinkbot approve description` to approve one or more aspects (aspects: 
`description`, `consensus`, `architecture` and `quality`)
- `@flinkbot approve all` to approve all aspects
- `@flinkbot approve-until architecture` to approve everything until 
`architecture`
- `@flinkbot attention @username1 [@username2 ..]` to require somebody's 
attention
- `@flinkbot disapprove architecture` to remove an approval you gave earlier
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] carp84 opened a new pull request #9516: [FLINK-13815][State Backends] Implement the SpaceAllocator

2019-08-22 Thread GitBox
carp84 opened a new pull request #9516: [FLINK-13815][State Backends] Implement 
the SpaceAllocator
URL: https://github.com/apache/flink/pull/9516
 
 
   
   ## What is the purpose of the change
   
   Adding allocators for off-heap byte buffer allocation, which is required by 
storing state on disk.
   
   
   ## Brief change log
   
   Mainly introduces two types of buffer allocators: `DirectBucketAllocator` 
and `PowerTwoBucketAllocator`. The former is for relatively big data, and the 
latter is a simple buddy-like allocator for small data. Also assures java 9+ 
compatibility referring to Hadoop's 
[CleanerUtil](https://github.com/apache/hadoop/blob/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/CleanerUtil.java).
   
   
   ## Verifying this change
   
   This change added tests under `org.apache.flink.runtime.state.heap.space` 
package.
   
   ## Does this pull request potentially affect one of the following parts:
   
 - Dependencies (does it add or upgrade a dependency): (no)
 - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (no)
 - The serializers: (no)
 - The runtime per-record code paths (performance sensitive): (no)
 - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
 - The S3 file system connector: (no)
   
   ## Documentation
   
 - Does this pull request introduce a new feature? (no)
 - If yes, how is the feature documented? (docs)
- Added a design doc for `PowerTwoBucketAllocator`
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Updated] (FLINK-13815) Implement the SpaceAllocator

2019-08-22 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13815?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

ASF GitHub Bot updated FLINK-13815:
---
Labels: pull-request-available  (was: )

> Implement the SpaceAllocator
> 
>
> Key: FLINK-13815
> URL: https://issues.apache.org/jira/browse/FLINK-13815
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / State Backends
>Reporter: Yu Li
>Priority: Major
>  Labels: pull-request-available
>
> As described in the design doc, we need a {{SpaceAllocator}} to allocate 
> space on off-heap/disk to store the spilled key-group data.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Commented] (FLINK-7151) Add a function SQL DDL

2019-08-22 Thread Zhenqiu Huang (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-7151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913959#comment-16913959
 ] 

Zhenqiu Huang commented on FLINK-7151:
--

[~phoenixjiangnan] As it is already in AthenaX. We would like to contribute 
back. I will work with Shuyi on this. Thanks

> Add a function SQL DDL
> --
>
> Key: FLINK-7151
> URL: https://issues.apache.org/jira/browse/FLINK-7151
> Project: Flink
>  Issue Type: Sub-task
>  Components: Table SQL / API
>Reporter: yuemeng
>Assignee: Shuyi Chen
>Priority: Critical
> Fix For: 1.10.0
>
>
> Based on create temporary function and table.we can register a udf,udaf,udtf 
> use sql:
> {code}
> CREATE FUNCTION [IF NOT EXISTS] [catalog_name.db_name.]function_name AS 
> class_name;
> DROP FUNCTION [IF EXISTS] [catalog_name.db_name.]function_name;
> ALTER FUNCTION [IF EXISTS] [catalog_name.db_name.]function_name RENAME TO 
> new_name;
> {code}
> {code}
> CREATE function 'TOPK' AS 
> 'com..aggregate.udaf.distinctUdaf.topk.ITopKUDAF';
> INSERT INTO db_sink SELECT id, TOPK(price, 5, 'DESC') FROM kafka_source GROUP 
> BY id;
> {code}



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot edited a comment on issue #9002: [FLINK-13105][table][doc] Add documentation for blink planner's built-in functions

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9002: [FLINK-13105][table][doc] Add 
documentation for blink planner's built-in functions
URL: https://github.com/apache/flink/pull/9002#issuecomment-513721231
 
 
   
   ## CI report:
   
   * 40477a632625f9cb7ebce8ed4488a99b7b4f5093 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/119984603)
   * 64ad34dd196a8941ff9f92dd0b389a22796650f8 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/123663046)
   * c9a49c38c20a47954412605ac151416eb1f3bea6 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/123673855)
   * f6079350661e096dea2505aaaf29466aeee53ce4 : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124054282)
   * 6dc82ded39c33867d6d3de364aa1aa3d5a654ddd : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124068158)
   * 32730d0b0f4ebc1d1c5ff2f677d6d46a1dc36273 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124317952)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #7713: [FLINK-10995][network] Copy intermediate serialization results only once for broadcast mode

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #7713: [FLINK-10995][network] Copy 
intermediate serialization results only once for broadcast mode
URL: https://github.com/apache/flink/pull/7713#issuecomment-524181331
 
 
   
   ## CI report:
   
   * 512499aae1ab33c5ae96c8a2100016e40d83b654 : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124321949)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #7631: [FLINK-11391][shuffle] Introduce PartitionShuffleDescriptor and ShuffleDeploymentDescriptor

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #7631: [FLINK-11391][shuffle] Introduce 
PartitionShuffleDescriptor and ShuffleDeploymentDescriptor
URL: https://github.com/apache/flink/pull/7631#issuecomment-524181162
 
 
   
   ## CI report:
   
   * 48e3aacae9724023140ea5d4fc8007a1550477b1 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124321935)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot commented on issue #7713: [FLINK-10995][network] Copy intermediate serialization results only once for broadcast mode

2019-08-22 Thread GitBox
flinkbot commented on issue #7713: [FLINK-10995][network] Copy intermediate 
serialization results only once for broadcast mode
URL: https://github.com/apache/flink/pull/7713#issuecomment-524181331
 
 
   
   ## CI report:
   
   * 512499aae1ab33c5ae96c8a2100016e40d83b654 : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot commented on issue #7631: [FLINK-11391][shuffle] Introduce PartitionShuffleDescriptor and ShuffleDeploymentDescriptor

2019-08-22 Thread GitBox
flinkbot commented on issue #7631: [FLINK-11391][shuffle] Introduce 
PartitionShuffleDescriptor and ShuffleDeploymentDescriptor
URL: https://github.com/apache/flink/pull/7631#issuecomment-524181162
 
 
   
   ## CI report:
   
   * 48e3aacae9724023140ea5d4fc8007a1550477b1 : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] pnowojski commented on issue #7713: [FLINK-10995][network] Copy intermediate serialization results only once for broadcast mode

2019-08-22 Thread GitBox
pnowojski commented on issue #7713: [FLINK-10995][network] Copy intermediate 
serialization results only once for broadcast mode
URL: https://github.com/apache/flink/pull/7713#issuecomment-524180337
 
 
   @zhijiangW do we want to revisit this one? I came across this, when I was 
looking for this code in the master branch (I was sure that it has been merged) 
while checking out things for:
   > [DISCUSS] Enhance Support for Multicast Communication Pattern
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] pnowojski commented on issue #7631: [FLINK-11391][shuffle] Introduce PartitionShuffleDescriptor and ShuffleDeploymentDescriptor

2019-08-22 Thread GitBox
pnowojski commented on issue #7631: [FLINK-11391][shuffle] Introduce 
PartitionShuffleDescriptor and ShuffleDeploymentDescriptor
URL: https://github.com/apache/flink/pull/7631#issuecomment-524179915
 
 
   @azagrebin @zhijiangW is this PR still valid? Or was it subsumed by 
something else?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] pnowojski commented on issue #7037: [FLINK-10790][network] Refactor StreamPartitioner implementations into runtime module

2019-08-22 Thread GitBox
pnowojski commented on issue #7037: [FLINK-10790][network] Refactor 
StreamPartitioner implementations into runtime module
URL: https://github.com/apache/flink/pull/7037#issuecomment-524179740
 
 
   I'm closing this PR @zhijiangW for now. If you will want to re-open it, 
please feel free to do so.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] pnowojski closed pull request #7037: [FLINK-10790][network] Refactor StreamPartitioner implementations into runtime module

2019-08-22 Thread GitBox
pnowojski closed pull request #7037: [FLINK-10790][network] Refactor 
StreamPartitioner implementations into runtime module
URL: https://github.com/apache/flink/pull/7037
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Comment Edited] (FLINK-13025) Elasticsearch 7.x support

2019-08-22 Thread Dongwon Kim (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-13025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913944#comment-16913944
 ] 

Dongwon Kim edited comment on FLINK-13025 at 8/23/19 5:25 AM:
--

[1] says flink-connector-elasticsearch6_2.11 can be used with Elasticsearch 6 
and later versions as shown in the last row of the table, which implies that it 
can support Elasticsearch 7.x but we failed.

Therefore, [1] needs to be updated accordingly: "6 and later versions" -> "6"

[1] 
https://ci.apache.org/projects/flink/flink-docs-stable/dev/connectors/elasticsearch.html#elasticsearch-connector


was (Author: eastcirclek):
[1] says flink-connector-elasticsearch6_2.11 can be used "6 and later versions" 
as shown in the last row of the table in [1], which implies that it can support 
Elasticsearch 7.x but we failed.

Therefore, [1] needs to be updated accordingly: "6 and later versions" -> "6"

[1] 
https://ci.apache.org/projects/flink/flink-docs-stable/dev/connectors/elasticsearch.html#elasticsearch-connector

> Elasticsearch 7.x support
> -
>
> Key: FLINK-13025
> URL: https://issues.apache.org/jira/browse/FLINK-13025
> Project: Flink
>  Issue Type: New Feature
>  Components: Connectors / ElasticSearch
>Affects Versions: 1.8.0
>Reporter: Keegan Standifer
>Priority: Major
>
> Elasticsearch 7.0.0 was released in April of 2019: 
> [https://www.elastic.co/blog/elasticsearch-7-0-0-released]
> The latest elasticsearch connector is 
> [flink-connector-elasticsearch6|https://github.com/apache/flink/tree/master/flink-connectors/flink-connector-elasticsearch6]



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Commented] (FLINK-13025) Elasticsearch 7.x support

2019-08-22 Thread Dongwon Kim (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-13025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913944#comment-16913944
 ] 

Dongwon Kim commented on FLINK-13025:
-

[1] says flink-connector-elasticsearch6_2.11 can be used "6 and later versions" 
as shown in the last row of the table in [1], which implies that it can support 
Elasticsearch 7.x but we failed.

Therefore, [1] needs to be updated accordingly: "6 and later versions" -> "6"

[1] 
https://ci.apache.org/projects/flink/flink-docs-stable/dev/connectors/elasticsearch.html#elasticsearch-connector

> Elasticsearch 7.x support
> -
>
> Key: FLINK-13025
> URL: https://issues.apache.org/jira/browse/FLINK-13025
> Project: Flink
>  Issue Type: New Feature
>  Components: Connectors / ElasticSearch
>Affects Versions: 1.8.0
>Reporter: Keegan Standifer
>Priority: Major
>
> Elasticsearch 7.0.0 was released in April of 2019: 
> [https://www.elastic.co/blog/elasticsearch-7-0-0-released]
> The latest elasticsearch connector is 
> [flink-connector-elasticsearch6|https://github.com/apache/flink/tree/master/flink-connectors/flink-connector-elasticsearch6]



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot edited a comment on issue #8303: [FLINK-12343] [flink-yarn] add file replication config for yarn configuration

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #8303: [FLINK-12343] [flink-yarn] add file 
replication config for yarn configuration
URL: https://github.com/apache/flink/pull/8303#issuecomment-511684151
 
 
   
   ## CI report:
   
   * 6a7ca58b4a04f6dce250045e021702e67e82b893 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/119421914)
   * 4d38a8df0d59734c4b2386689a2f17b9f2b44b12 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/119441376)
   * 9c14836f8639e98d58cf7bb32e38b938b3843994 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/119577044)
   * 76186776c5620598a19234245bbd05dfdfb1c62c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/120113740)
   * 628ca7b316ad3968c90192a47a84dd01f26e2578 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/122381349)
   * d204a725ff3c8a046cbd1b84e34d9e3ae8aafeac : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123620485)
   * 143efadbdb6c4681569d5b412a175edfb1633b85 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123637809)
   * b78b64a82ed2a9a92886095ec42f06d5082ad830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123671219)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] HuangZhenQiu commented on a change in pull request #8303: [FLINK-12343] [flink-yarn] add file replication config for yarn configuration

2019-08-22 Thread GitBox
HuangZhenQiu commented on a change in pull request #8303: [FLINK-12343] 
[flink-yarn] add file replication config for yarn configuration
URL: https://github.com/apache/flink/pull/8303#discussion_r316976697
 
 

 ##
 File path: flink-yarn-tests/src/test/java/org/apache/flink/yarn/YARNITCase.java
 ##
 @@ -69,6 +74,7 @@ public void testPerJobMode() throws Exception {
runTest(() -> {
Configuration configuration = new Configuration();
configuration.setString(AkkaOptions.ASK_TIMEOUT, "30 
s");
+   
configuration.setString(YarnConfigOptions.FILE_REPLICATION, "5");
 
 Review comment:
   Sure.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] HuangZhenQiu commented on a change in pull request #8303: [FLINK-12343] [flink-yarn] add file replication config for yarn configuration

2019-08-22 Thread GitBox
HuangZhenQiu commented on a change in pull request #8303: [FLINK-12343] 
[flink-yarn] add file replication config for yarn configuration
URL: https://github.com/apache/flink/pull/8303#discussion_r316976653
 
 

 ##
 File path: 
flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java
 ##
 @@ -746,6 +746,17 @@ public ApplicationReport startAppMaster(
// ship list that enables reuse of resources for task manager 
containers
StringBuilder envShipFileList = new StringBuilder();
 
+   int fileReplication = 
yarnConfiguration.getInt(DFSConfigKeys.DFS_REPLICATION_KEY, 
DFSConfigKeys.DFS_REPLICATION_DEFAULT);
+   if 
(flinkConfiguration.contains(YarnConfigOptions.FILE_REPLICATION)) {
+   try {
+   final String replication = 
flinkConfiguration.getString(YarnConfigOptions.FILE_REPLICATION);
 
 Review comment:
   ConfigOption is not nullable. Per your last suggestion, if the 
FILE_REPLICATION is not set, then we will use the default value.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9002: [FLINK-13105][table][doc] Add documentation for blink planner's built-in functions

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9002: [FLINK-13105][table][doc] Add 
documentation for blink planner's built-in functions
URL: https://github.com/apache/flink/pull/9002#issuecomment-513721231
 
 
   
   ## CI report:
   
   * 40477a632625f9cb7ebce8ed4488a99b7b4f5093 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/119984603)
   * 64ad34dd196a8941ff9f92dd0b389a22796650f8 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/123663046)
   * c9a49c38c20a47954412605ac151416eb1f3bea6 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/123673855)
   * f6079350661e096dea2505aaaf29466aeee53ce4 : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124054282)
   * 6dc82ded39c33867d6d3de364aa1aa3d5a654ddd : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124068158)
   * 32730d0b0f4ebc1d1c5ff2f677d6d46a1dc36273 : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124317952)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9002: [FLINK-13105][table][doc] Add documentation for blink planner's built-in functions

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9002: [FLINK-13105][table][doc] Add 
documentation for blink planner's built-in functions
URL: https://github.com/apache/flink/pull/9002#issuecomment-513721231
 
 
   
   ## CI report:
   
   * 40477a632625f9cb7ebce8ed4488a99b7b4f5093 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/119984603)
   * 64ad34dd196a8941ff9f92dd0b389a22796650f8 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/123663046)
   * c9a49c38c20a47954412605ac151416eb1f3bea6 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/123673855)
   * f6079350661e096dea2505aaaf29466aeee53ce4 : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124054282)
   * 6dc82ded39c33867d6d3de364aa1aa3d5a654ddd : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124068158)
   * 32730d0b0f4ebc1d1c5ff2f677d6d46a1dc36273 : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] docete commented on issue #9002: [FLINK-13105][table][doc] Add documentation for blink planner's built-in functions

2019-08-22 Thread GitBox
docete commented on issue #9002: [FLINK-13105][table][doc] Add documentation 
for blink planner's built-in functions
URL: https://github.com/apache/flink/pull/9002#issuecomment-524167790
 
 
   @wuchong Thanks for your comments and LGTM for your changes expect for 
parameters of STR_TO_MAP. I have fixed that. Please have a look at it. 


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] docete commented on a change in pull request #9002: [FLINK-13105][table][doc] Add documentation for blink planner's built-in functions

2019-08-22 Thread GitBox
docete commented on a change in pull request #9002: [FLINK-13105][table][doc] 
Add documentation for blink planner's built-in functions
URL: https://github.com/apache/flink/pull/9002#discussion_r316965281
 
 

 ##
 File path: docs/dev/table/functions.md
 ##
 @@ -1466,6 +1466,18 @@ TRUNCATE(numeric1, integer2)
 E.g. truncate(42.345, 2) to 42.34. and 
truncate(42.345) to 42.0.
   
 
+
+
+  
+{% highlight text %}
+PI()
+{% endhighlight %}
+  
+  
+  Returns a value that is closer than any other values to pi.
 
 Review comment:
   LGTM.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9515: [hotfix][table-blink] Ignore tests in GroupWindowTableAggregateITCase and TableAggregateITCase

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9515: [hotfix][table-blink] Ignore tests in 
GroupWindowTableAggregateITCase and TableAggregateITCase
URL: https://github.com/apache/flink/pull/9515#issuecomment-524154431
 
 
   
   ## CI report:
   
   * b6645c0abcbbe6a94586916772b15bea15818d95 : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124313513)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Resolved] (FLINK-13354) Add documentation for how to use blink planner

2019-08-22 Thread Jark Wu (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13354?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jark Wu resolved FLINK-13354.
-
Resolution: Fixed

master: 57bb00a19f296312b3c06599901f4ef529d2feaf

1.9.1: 1dce49445263cca3733c3923c129c7c3c1f9b581

> Add documentation for how to use blink planner
> --
>
> Key: FLINK-13354
> URL: https://issues.apache.org/jira/browse/FLINK-13354
> Project: Flink
>  Issue Type: Sub-task
>  Components: Documentation, Table SQL / Planner
>Reporter: Jark Wu
>Assignee: godfrey he
>Priority: Critical
>  Labels: pull-request-available
> Fix For: 1.9.1
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> Add documentation for how to use different planner
> - “Overview”: add pom dependency
> -  “Concepts & Common API”:  add description about how to use different 
> planner in code



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] asfgit closed pull request #9362: [FLINK-13354] [docs] Add documentation for how to use blink planner

2019-08-22 Thread GitBox
asfgit closed pull request #9362: [FLINK-13354] [docs] Add documentation for 
how to use blink planner
URL: https://github.com/apache/flink/pull/9362
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] wuchong commented on issue #9362: [FLINK-13354] [docs] Add documentation for how to use blink planner

2019-08-22 Thread GitBox
wuchong commented on issue #9362: [FLINK-13354] [docs] Add documentation for 
how to use blink planner
URL: https://github.com/apache/flink/pull/9362#issuecomment-524160913
 
 
   I will merge this PR first to make users can know how to use blink planner 
for the release. We can improve the content in following PRs. 


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-523852882
 
 
   
   ## CI report:
   
   * a5a0a74cb3e2bef0cd7352a6cb8e7697cbd79f75 : UNKNOWN
   * bde5de0cc669dec593e090e5425f39d56c220830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124201954)
   * 63ab3d431bf8256dfb071574e013f0e878e40cef : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124203053)
   * fe6f2ab1d566100cd63941e890e2d34947ea43b8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124205327)
   * ccb85242c0c0a5f2fec18f7d67dd5293e9ad2dce : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124238221)
   * 57a5ab12b5cfca106a2352892d2334f83273d86c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124300218)
   * 694ad929be390ad0a37406b499f8e8ba68346399 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124310900)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] hequn8128 closed pull request #9515: [hotfix][table-blink] Ignore tests in GroupWindowTableAggregateITCase and TableAggregateITCase

2019-08-22 Thread GitBox
hequn8128 closed pull request #9515: [hotfix][table-blink] Ignore tests in 
GroupWindowTableAggregateITCase and TableAggregateITCase
URL: https://github.com/apache/flink/pull/9515
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (FLINK-13740) TableAggregateITCase.testNonkeyedFlatAggregate failed on Travis

2019-08-22 Thread Hequn Cheng (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-13740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913889#comment-16913889
 ] 

Hequn Cheng commented on FLINK-13740:
-

The hotfix(b6645c0abcbbe6a94586916772b15bea15818d95) has been merged into the 
master.

> TableAggregateITCase.testNonkeyedFlatAggregate failed on Travis
> ---
>
> Key: FLINK-13740
> URL: https://issues.apache.org/jira/browse/FLINK-13740
> Project: Flink
>  Issue Type: Bug
>  Components: Table SQL / Planner
>Affects Versions: 1.10.0
>Reporter: Till Rohrmann
>Priority: Critical
>  Labels: test-stability
> Fix For: 1.10.0
>
>
> The {{TableAggregateITCase.testNonkeyedFlatAggregate}} failed on Travis with 
> {code}
> org.apache.flink.runtime.client.JobExecutionException: Job execution failed.
>   at 
> org.apache.flink.table.planner.runtime.stream.table.TableAggregateITCase.testNonkeyedFlatAggregate(TableAggregateITCase.scala:93)
> Caused by: java.lang.Exception: Artificial Failure
> {code}
> https://api.travis-ci.com/v3/job/225551182/log.txt



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] docete commented on a change in pull request #9002: [FLINK-13105][table][doc] Add documentation for blink planner's built-in functions

2019-08-22 Thread GitBox
docete commented on a change in pull request #9002: [FLINK-13105][table][doc] 
Add documentation for blink planner's built-in functions
URL: https://github.com/apache/flink/pull/9002#discussion_r316963583
 
 

 ##
 File path: docs/dev/table/functions.md
 ##
 @@ -1466,6 +1466,18 @@ TRUNCATE(numeric1, integer2)
 E.g. truncate(42.345, 2) to 42.34. and 
truncate(42.345) to 42.0.
   
 
+
+
+  
+{% highlight text %}
+PI()
+{% endhighlight %}
+  
+  
+  Returns a value that is closer than any other values to pi.
+  Only supported in blink planner. Only supported in SQL api.
 
 Review comment:
   OK.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] docete commented on a change in pull request #9002: [FLINK-13105][table][doc] Add documentation for blink planner's built-in functions

2019-08-22 Thread GitBox
docete commented on a change in pull request #9002: [FLINK-13105][table][doc] 
Add documentation for blink planner's built-in functions
URL: https://github.com/apache/flink/pull/9002#discussion_r316963527
 
 

 ##
 File path: docs/dev/table/functions.md
 ##
 @@ -2689,6 +2701,177 @@ TO_BASE64(string)
 E.g., TO_BASE64('hello world') returns 
"aGVsbG8gd29ybGQ=".
   
 
+
+
+  
+{% highlight text %}
+ASCII(string)
+{% endhighlight %}
+  
+  
+Returns the numeric value of the first character of string. 
Returns NULL if string is NULL.
+Only supported in blink planner. Only supported in SQL api.
+E.g., ascii('abc') returns 97, and 
ascii(CAST(NULL AS VARCHAR)) returns NULL.
+  
+
+
+
+  
+{% highlight text %}
+CHR(integer)
+{% endhighlight %}
+  
+  
+Returns the ASCII character having the binary equivalent to 
integer. If integer is larger than 255 the result is equivalent 
to chr(bitand(integer,256). Returns NULL if integer is NULL.
 
 Review comment:
   Make sense, will fix it soon.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9515: [hotfix][table-blink] Ignore tests in GroupWindowTableAggregateITCase and TableAggregateITCase

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9515: [hotfix][table-blink] Ignore tests in 
GroupWindowTableAggregateITCase and TableAggregateITCase
URL: https://github.com/apache/flink/pull/9515#issuecomment-524154431
 
 
   
   ## CI report:
   
   * b6645c0abcbbe6a94586916772b15bea15818d95 : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124313513)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] vthinkxie commented on a change in pull request #9504: [FLINK-13799][web]: fix error message web submit is disables in the c…

2019-08-22 Thread GitBox
vthinkxie commented on a change in pull request #9504: [FLINK-13799][web]: fix 
error message web submit is disables in the c…
URL: https://github.com/apache/flink/pull/9504#discussion_r316959338
 
 

 ##
 File path: flink-runtime-web/web-dashboard/src/app/app.interceptor.ts
 ##
 @@ -32,7 +32,7 @@ export class AppInterceptor implements HttpInterceptor {
  * Error response from below url should be ignored
  */
 const ignoreErrorUrlEndsList = ['checkpoints/config', 'checkpoints'];
-const ignoreErrorMessage = ['File not found.'];
+const ignoreErrorMessage = ['File not found.', 'Unable to load requested 
file /jars.'];
 
 Review comment:
   the upload button in the submit page is already hidden when getting the 
error message, so the user could not use the submit page in the case.
   I think we should wait for the proper solution if not hurry.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Closed] (FLINK-13772) GroupWindowTableAggregateITCase tests fail on Travis

2019-08-22 Thread Hequn Cheng (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13772?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Hequn Cheng closed FLINK-13772.
---
Resolution: Fixed

> GroupWindowTableAggregateITCase tests fail on Travis
> 
>
> Key: FLINK-13772
> URL: https://issues.apache.org/jira/browse/FLINK-13772
> Project: Flink
>  Issue Type: Bug
>  Components: Table SQL / Planner
>Affects Versions: 1.10.0
>Reporter: Till Rohrmann
>Priority: Critical
>  Labels: test-stability
> Fix For: 1.10.0
>
>
> The 
> {{GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane}}
>  fails on Travis:
> {code}
> 05:21:41.532 [ERROR] Tests run: 18, Failures: 0, Errors: 1, Skipped: 0, Time 
> elapsed: 14.039 s <<< FAILURE! - in 
> org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase
> 05:21:41.532 [ERROR] 
> testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane[StateBackend=HEAP](org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase)
>   Time elapsed: 0.573 s  <<< ERROR!
> org.apache.flink.runtime.client.JobExecutionException: Job execution failed.
>   at 
> org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane(GroupWindowTableAggregateITCase.scala:301)
> Caused by: java.lang.Exception: Artificial Failure
> {code}
> https://api.travis-ci.org/v3/job/573615974/log.txt



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Comment Edited] (FLINK-13772) GroupWindowTableAggregateITCase tests fail on Travis

2019-08-22 Thread Hequn Cheng (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-13772?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913849#comment-16913849
 ] 

Hequn Cheng edited comment on FLINK-13772 at 8/23/19 3:03 AM:
--

As discussed in FLINK-13740. The serializer logic contains a bug which leads to 
the test failure in TableAggregate tests.
I will open a hotfix to ignore the tests for the time being. We can recover the 
tests once the bug is fixed in blink-planner.



was (Author: hequn8128):
Same problem as in FLINK-13740

> GroupWindowTableAggregateITCase tests fail on Travis
> 
>
> Key: FLINK-13772
> URL: https://issues.apache.org/jira/browse/FLINK-13772
> Project: Flink
>  Issue Type: Bug
>  Components: Table SQL / Planner
>Affects Versions: 1.10.0
>Reporter: Till Rohrmann
>Priority: Critical
>  Labels: test-stability
> Fix For: 1.10.0
>
>
> The 
> {{GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane}}
>  fails on Travis:
> {code}
> 05:21:41.532 [ERROR] Tests run: 18, Failures: 0, Errors: 1, Skipped: 0, Time 
> elapsed: 14.039 s <<< FAILURE! - in 
> org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase
> 05:21:41.532 [ERROR] 
> testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane[StateBackend=HEAP](org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase)
>   Time elapsed: 0.573 s  <<< ERROR!
> org.apache.flink.runtime.client.JobExecutionException: Job execution failed.
>   at 
> org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane(GroupWindowTableAggregateITCase.scala:301)
> Caused by: java.lang.Exception: Artificial Failure
> {code}
> https://api.travis-ci.org/v3/job/573615974/log.txt



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot commented on issue #9515: [hotfix][table-blink] Ignore tests in GroupWindowTableAggregateITCase and TableAggregateITCase

2019-08-22 Thread GitBox
flinkbot commented on issue #9515: [hotfix][table-blink] Ignore tests in 
GroupWindowTableAggregateITCase and TableAggregateITCase
URL: https://github.com/apache/flink/pull/9515#issuecomment-524154431
 
 
   
   ## CI report:
   
   * b6645c0abcbbe6a94586916772b15bea15818d95 : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] vthinkxie commented on a change in pull request #9504: [FLINK-13799][web]: fix error message web submit is disables in the c…

2019-08-22 Thread GitBox
vthinkxie commented on a change in pull request #9504: [FLINK-13799][web]: fix 
error message web submit is disables in the c…
URL: https://github.com/apache/flink/pull/9504#discussion_r316959338
 
 

 ##
 File path: flink-runtime-web/web-dashboard/src/app/app.interceptor.ts
 ##
 @@ -32,7 +32,7 @@ export class AppInterceptor implements HttpInterceptor {
  * Error response from below url should be ignored
  */
 const ignoreErrorUrlEndsList = ['checkpoints/config', 'checkpoints'];
-const ignoreErrorMessage = ['File not found.'];
+const ignoreErrorMessage = ['File not found.', 'Unable to load requested 
file /jars.'];
 
 Review comment:
   the upload button in the submit page is already hidden when getting the 
error message, so the user could not use the submit page in the case.
   I think we should wait for the proper solution.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot commented on issue #9515: [hotfix][table-blink] Ignore tests in GroupWindowTableAggregateITCase and TableAggregateITCase

2019-08-22 Thread GitBox
flinkbot commented on issue #9515: [hotfix][table-blink] Ignore tests in 
GroupWindowTableAggregateITCase and TableAggregateITCase
URL: https://github.com/apache/flink/pull/9515#issuecomment-524153818
 
 
   Thanks a lot for your contribution to the Apache Flink project. I'm the 
@flinkbot. I help the community
   to review your pull request. We will use this comment to track the progress 
of the review.
   
   
   ## Automated Checks
   Last check on commit b6645c0abcbbe6a94586916772b15bea15818d95 (Fri Aug 23 
03:00:10 UTC 2019)
   
   **Warnings:**
* No documentation files were touched! Remember to keep the Flink docs up 
to date!
   
   
   Mention the bot in a comment to re-run the automated checks.
   ## Review Progress
   
   * ❓ 1. The [description] looks good.
   * ❓ 2. There is [consensus] that the contribution should go into to Flink.
   * ❓ 3. Needs [attention] from.
   * ❓ 4. The change fits into the overall [architecture].
   * ❓ 5. Overall code [quality] is good.
   
   Please see the [Pull Request Review 
Guide](https://flink.apache.org/contributing/reviewing-prs.html) for a full 
explanation of the review process.
The Bot is tracking the review progress through labels. Labels are applied 
according to the order of the review items. For consensus, approval by a Flink 
committer of PMC member is required Bot commands
 The @flinkbot bot supports the following commands:
   
- `@flinkbot approve description` to approve one or more aspects (aspects: 
`description`, `consensus`, `architecture` and `quality`)
- `@flinkbot approve all` to approve all aspects
- `@flinkbot approve-until architecture` to approve everything until 
`architecture`
- `@flinkbot attention @username1 [@username2 ..]` to require somebody's 
attention
- `@flinkbot disapprove architecture` to remove an approval you gave earlier
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] hequn8128 opened a new pull request #9515: [hotfix][table-blink] Ignore tests in GroupWindowTableAggregateITCase and TableAggregateITCase

2019-08-22 Thread GitBox
hequn8128 opened a new pull request #9515: [hotfix][table-blink] Ignore tests 
in GroupWindowTableAggregateITCase and TableAggregateITCase
URL: https://github.com/apache/flink/pull/9515
 
 
   
   ## What is the purpose of the change
   
   As discussed in 
[FLINK-13740](https://issues.apache.org/jira/browse/FLINK-13740). The 
serializer logic contains a bug which leads to the test failure in 
TableAggregate tests. 
   
   This pr tries to ignore the tests in TableAggregate and recover it once the 
bug is fixed in blink-planner.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9497: [FLINK-13794][client] Remove unused logic of printStatusDuringExecution

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9497: [FLINK-13794][client] Remove unused 
logic of printStatusDuringExecution
URL: https://github.com/apache/flink/pull/9497#issuecomment-523364967
 
 
   
   ## CI report:
   
   * b383982c0f9353b799ba85ca2635cd9b5f3e24a8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124020443)
   * dec3458bf0f82c673cbcc9b686f590a83062 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124183689)
   * f97118b164679e2f771ab242c57dcbcff8d51053 : PENDING [Build](TBD)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] TisonKun commented on issue #9497: [FLINK-13794][client] Remove unused logic of printStatusDuringExecution

2019-08-22 Thread GitBox
TisonKun commented on issue #9497: [FLINK-13794][client] Remove unused logic of 
printStatusDuringExecution
URL: https://github.com/apache/flink/pull/9497#issuecomment-524150322
 
 
   @flinkbot run travis


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] TisonKun commented on issue #9488: [hotfix][docs] Update local setup tutorials to fit new log messages

2019-08-22 Thread GitBox
TisonKun commented on issue #9488: [hotfix][docs] Update local setup tutorials 
to fit new log messages
URL: https://github.com/apache/flink/pull/9488#issuecomment-524150272
 
 
   @rmetzger could you help merge this hotfix?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] TisonKun commented on issue #9510: [FLINK-13807][tests] Read file with UTF-8 charset in TestBaseUtils.getResultReader

2019-08-22 Thread GitBox
TisonKun commented on issue #9510: [FLINK-13807][tests] Read file with UTF-8 
charset in TestBaseUtils.getResultReader
URL: https://github.com/apache/flink/pull/9510#issuecomment-524149924
 
 
   @zentol travis fails on a known issue 
[FLINK-13769](https://issues.apache.org/jira/browse/FLINK-13769). Shall we 
retest this?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-523852882
 
 
   
   ## CI report:
   
   * a5a0a74cb3e2bef0cd7352a6cb8e7697cbd79f75 : UNKNOWN
   * bde5de0cc669dec593e090e5425f39d56c220830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124201954)
   * 63ab3d431bf8256dfb071574e013f0e878e40cef : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124203053)
   * fe6f2ab1d566100cd63941e890e2d34947ea43b8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124205327)
   * ccb85242c0c0a5f2fec18f7d67dd5293e9ad2dce : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124238221)
   * 57a5ab12b5cfca106a2352892d2334f83273d86c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124300218)
   * 694ad929be390ad0a37406b499f8e8ba68346399 : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124310900)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (FLINK-13750) Separate HA services between client-/ and server-side

2019-08-22 Thread TisonKun (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-13750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913856#comment-16913856
 ] 

TisonKun commented on FLINK-13750:
--

Hi [~till.rohrmann][~Zentol], I sent a pull request GH-9509 follow our 
discussion above. Please give it a look :- )

Besides, here I'd like to start a new sub-topic, shall we expose "leaderInfo" 
to client? The only usage I see is to log in {{FlinkYarnSessionCli}} and 
extract dispatcher host:port in {{FlinkShell}} under YARN scenario. I think of 
an alternative instead of expose "leaderInfo" to client. Or we just design to 
expose it here.

> Separate HA services between client-/ and server-side
> -
>
> Key: FLINK-13750
> URL: https://issues.apache.org/jira/browse/FLINK-13750
> Project: Flink
>  Issue Type: Improvement
>  Components: Command Line Client, Runtime / Coordination
>Reporter: Chesnay Schepler
>Assignee: TisonKun
>Priority: Major
>  Labels: pull-request-available
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Currently, we use the same {{HighAvailabilityServices}} on the client and 
> server. However, the client does not need several of the features that the 
> services currently provide (access to the blobstore or checkpoint metadata).
> Additionally, due to how these services are setup they also require the 
> client to have access to the blob storage, despite it never actually being 
> used, which can cause issues, like FLINK-13500.
> [~Tison] Would be be interested in this issue?



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-523852882
 
 
   
   ## CI report:
   
   * a5a0a74cb3e2bef0cd7352a6cb8e7697cbd79f75 : UNKNOWN
   * bde5de0cc669dec593e090e5425f39d56c220830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124201954)
   * 63ab3d431bf8256dfb071574e013f0e878e40cef : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124203053)
   * fe6f2ab1d566100cd63941e890e2d34947ea43b8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124205327)
   * ccb85242c0c0a5f2fec18f7d67dd5293e9ad2dce : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124238221)
   * 57a5ab12b5cfca106a2352892d2334f83273d86c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124300218)
   * 694ad929be390ad0a37406b499f8e8ba68346399 : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Closed] (FLINK-13822) TableAggregateITCase.testGroupByFlatAggregate test failure: IllegalStateException: Concurrent access to KryoSerializer.

2019-08-22 Thread Jark Wu (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13822?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jark Wu closed FLINK-13822.
---
Resolution: Duplicate

> TableAggregateITCase.testGroupByFlatAggregate test failure: 
> IllegalStateException: Concurrent access to KryoSerializer.
> ---
>
> Key: FLINK-13822
> URL: https://issues.apache.org/jira/browse/FLINK-13822
> Project: Flink
>  Issue Type: Bug
>Reporter: Alex
>Priority: Major
>
> {code}
> [ERROR] 
> testGroupByFlatAggregate[StateBackend=HEAP](org.apache.flink.table.planner.runtime.stream.table.TableAggregateITCase)
>   Time elapsed: 0.643 s  <<< ERROR!
> org.apache.flink.runtime.client.JobExecutionException: Job execution failed.
>   at 
> org.apache.flink.table.planner.runtime.stream.table.TableAggregateITCase.testGroupByFlatAggregate(TableAggregateITCase.scala:59)
> Caused by: java.lang.IllegalStateException: Concurrent access to 
> KryoSerializer. Thread 1: GroupTableAggregate -> Calc(select=[b AS category, 
> f0 AS v1, f1 AS v2]) -> SinkConversionToTuple2 (1/4) , Thread 2: 
> AsyncOperations-thread-1
> {code}
> CI log: https://api.travis-ci.com/v3/job/227362742/log.txt



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Commented] (FLINK-13740) TableAggregateITCase.testNonkeyedFlatAggregate failed on Travis

2019-08-22 Thread Jark Wu (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-13740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913853#comment-16913853
 ] 

Jark Wu commented on FLINK-13740:
-

Agree with you [~hequn8128]. Let's ignore the tests first. And discuss the 
solution under this Jira issue. 

> TableAggregateITCase.testNonkeyedFlatAggregate failed on Travis
> ---
>
> Key: FLINK-13740
> URL: https://issues.apache.org/jira/browse/FLINK-13740
> Project: Flink
>  Issue Type: Bug
>  Components: Table SQL / Planner
>Affects Versions: 1.10.0
>Reporter: Till Rohrmann
>Priority: Critical
>  Labels: test-stability
> Fix For: 1.10.0
>
>
> The {{TableAggregateITCase.testNonkeyedFlatAggregate}} failed on Travis with 
> {code}
> org.apache.flink.runtime.client.JobExecutionException: Job execution failed.
>   at 
> org.apache.flink.table.planner.runtime.stream.table.TableAggregateITCase.testNonkeyedFlatAggregate(TableAggregateITCase.scala:93)
> Caused by: java.lang.Exception: Artificial Failure
> {code}
> https://api.travis-ci.com/v3/job/225551182/log.txt



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Updated] (FLINK-13790) Support -q option with a sql query statement as input

2019-08-22 Thread Zhenghua Gao (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13790?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Zhenghua Gao updated FLINK-13790:
-
Summary: Support -q option with a sql query statement as input  (was: 
Support -e option with a sql script file as input)

> Support -q option with a sql query statement as input
> -
>
> Key: FLINK-13790
> URL: https://issues.apache.org/jira/browse/FLINK-13790
> Project: Flink
>  Issue Type: Sub-task
>  Components: Command Line Client
>Reporter: Bowen Li
>Assignee: Zhenghua Gao
>Priority: Major
> Fix For: 1.10.0
>
>
> We expect user to run SQL directly on the command line. Something like: 
> sql-client embedded -e "query in string", which will execute the given query 
> without entering interactive mode
> This is related to FLINK-12828.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Updated] (FLINK-13790) Support -q option with a sql query statement as input

2019-08-22 Thread Zhenghua Gao (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13790?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Zhenghua Gao updated FLINK-13790:
-
Description: 
We expect user to run SQL directly on the command line. Something like: 
sql-client embedded -q "query in string", which will execute the given query 
without entering interactive mode

This is related to FLINK-12828.

  was:
We expect user to run SQL directly on the command line. Something like: 
sql-client embedded -e "query in string", which will execute the given query 
without entering interactive mode

This is related to FLINK-12828.


> Support -q option with a sql query statement as input
> -
>
> Key: FLINK-13790
> URL: https://issues.apache.org/jira/browse/FLINK-13790
> Project: Flink
>  Issue Type: Sub-task
>  Components: Command Line Client
>Reporter: Bowen Li
>Assignee: Zhenghua Gao
>Priority: Major
> Fix For: 1.10.0
>
>
> We expect user to run SQL directly on the command line. Something like: 
> sql-client embedded -q "query in string", which will execute the given query 
> without entering interactive mode
> This is related to FLINK-12828.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Commented] (FLINK-13772) GroupWindowTableAggregateITCase tests fail on Travis

2019-08-22 Thread Hequn Cheng (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-13772?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913849#comment-16913849
 ] 

Hequn Cheng commented on FLINK-13772:
-

Same problem as in FLINK-13740

> GroupWindowTableAggregateITCase tests fail on Travis
> 
>
> Key: FLINK-13772
> URL: https://issues.apache.org/jira/browse/FLINK-13772
> Project: Flink
>  Issue Type: Bug
>  Components: Table SQL / Planner
>Affects Versions: 1.10.0
>Reporter: Till Rohrmann
>Priority: Critical
>  Labels: test-stability
> Fix For: 1.10.0
>
>
> The 
> {{GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane}}
>  fails on Travis:
> {code}
> 05:21:41.532 [ERROR] Tests run: 18, Failures: 0, Errors: 1, Skipped: 0, Time 
> elapsed: 14.039 s <<< FAILURE! - in 
> org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase
> 05:21:41.532 [ERROR] 
> testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane[StateBackend=HEAP](org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase)
>   Time elapsed: 0.573 s  <<< ERROR!
> org.apache.flink.runtime.client.JobExecutionException: Job execution failed.
>   at 
> org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane(GroupWindowTableAggregateITCase.scala:301)
> Caused by: java.lang.Exception: Artificial Failure
> {code}
> https://api.travis-ci.org/v3/job/573615974/log.txt



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Commented] (FLINK-13740) TableAggregateITCase.testNonkeyedFlatAggregate failed on Travis

2019-08-22 Thread Hequn Cheng (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-13740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913845#comment-16913845
 ] 

Hequn Cheng commented on FLINK-13740:
-

Hi, [~jark] [~lzljs3620320] the same problem is reported by another jira 
FLINK-13822. Would be great if you can take another look and fix it. 
Another option is ignoring the test case for the time being if you find fixing 
the problem in blink planner needs more time to discuss. (I can do it if it is 
true.)
Thanks, Hequn

> TableAggregateITCase.testNonkeyedFlatAggregate failed on Travis
> ---
>
> Key: FLINK-13740
> URL: https://issues.apache.org/jira/browse/FLINK-13740
> Project: Flink
>  Issue Type: Bug
>  Components: Table SQL / Planner
>Affects Versions: 1.10.0
>Reporter: Till Rohrmann
>Priority: Critical
>  Labels: test-stability
> Fix For: 1.10.0
>
>
> The {{TableAggregateITCase.testNonkeyedFlatAggregate}} failed on Travis with 
> {code}
> org.apache.flink.runtime.client.JobExecutionException: Job execution failed.
>   at 
> org.apache.flink.table.planner.runtime.stream.table.TableAggregateITCase.testNonkeyedFlatAggregate(TableAggregateITCase.scala:93)
> Caused by: java.lang.Exception: Artificial Failure
> {code}
> https://api.travis-ci.com/v3/job/225551182/log.txt



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-523852882
 
 
   
   ## CI report:
   
   * a5a0a74cb3e2bef0cd7352a6cb8e7697cbd79f75 : UNKNOWN
   * bde5de0cc669dec593e090e5425f39d56c220830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124201954)
   * 63ab3d431bf8256dfb071574e013f0e878e40cef : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124203053)
   * fe6f2ab1d566100cd63941e890e2d34947ea43b8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124205327)
   * ccb85242c0c0a5f2fec18f7d67dd5293e9ad2dce : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124238221)
   * 57a5ab12b5cfca106a2352892d2334f83273d86c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124300218)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9514: [FLINK-13573][coordination] Merge SubmittedJobGraph into JobGraph

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9514: [FLINK-13573][coordination] Merge 
SubmittedJobGraph into JobGraph
URL: https://github.com/apache/flink/pull/9514#issuecomment-52458
 
 
   
   ## CI report:
   
   * 6092a6154f8275957d306c9d04b780ccd374b5d9 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124258242)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-523852882
 
 
   
   ## CI report:
   
   * a5a0a74cb3e2bef0cd7352a6cb8e7697cbd79f75 : UNKNOWN
   * bde5de0cc669dec593e090e5425f39d56c220830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124201954)
   * 63ab3d431bf8256dfb071574e013f0e878e40cef : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124203053)
   * fe6f2ab1d566100cd63941e890e2d34947ea43b8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124205327)
   * ccb85242c0c0a5f2fec18f7d67dd5293e9ad2dce : CANCELED 
[Build](https://travis-ci.com/flink-ci/flink/builds/124238221)
   * 57a5ab12b5cfca106a2352892d2334f83273d86c : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124300218)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9476: [FLINK-13764][task, metrics] Pass the counter of numRecordsIn into the constructor of StreamInputProcessor

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9476: [FLINK-13764][task, metrics] Pass the 
counter of numRecordsIn into the constructor of StreamInputProcessor
URL: https://github.com/apache/flink/pull/9476#issuecomment-522355867
 
 
   
   ## CI report:
   
   * df58cd55f5ca284e412ebbfc35c336e4f012974c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123646651)
   * 870e9199aa5afbbdea0bccffcb2fa4c2fa26ec8f : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124293088)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-523852882
 
 
   
   ## CI report:
   
   * a5a0a74cb3e2bef0cd7352a6cb8e7697cbd79f75 : UNKNOWN
   * bde5de0cc669dec593e090e5425f39d56c220830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124201954)
   * 63ab3d431bf8256dfb071574e013f0e878e40cef : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124203053)
   * fe6f2ab1d566100cd63941e890e2d34947ea43b8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124205327)
   * ccb85242c0c0a5f2fec18f7d67dd5293e9ad2dce : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124238221)
   * 57a5ab12b5cfca106a2352892d2334f83273d86c : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] TisonKun commented on a change in pull request #9512: [FLINK-13819][coordination] Introduce State for RpcEndpoint

2019-08-22 Thread GitBox
TisonKun commented on a change in pull request #9512: 
[FLINK-13819][coordination] Introduce State for RpcEndpoint
URL: https://github.com/apache/flink/pull/9512#discussion_r316920238
 
 

 ##
 File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/rpc/RpcEndpoint.java
 ##
 @@ -137,18 +153,26 @@ public final void start() {
 * @throws Exception indicating that the rpc endpoint could not be 
started. If an exception occurs,
 * then the rpc endpoint will automatically terminate.
 */
-   public void onStart() throws Exception {}
+   public final void doStart() throws Exception {
 
 Review comment:
   Make sense.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9478: [FLINK-13766][task] Refactor the implementation of StreamInputProcessor based on StreamTaskInput#emitNext

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9478: [FLINK-13766][task] Refactor the 
implementation of StreamInputProcessor based on StreamTaskInput#emitNext
URL: https://github.com/apache/flink/pull/9478#issuecomment-522374042
 
 
   
   ## CI report:
   
   * 1530600eaf36324966f343c277437e48c2416dc2 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123653758)
   * 8d08aaa1fedf7338ef3de48991e841f9baf7018b : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123700944)
   * 9791697c559fa39cd9f7a1c574420d62a1193545 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124210361)
   * fbd14e2320371a3cd6615deee0498d477644b954 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124297960)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9514: [FLINK-13573][coordination] Merge SubmittedJobGraph into JobGraph

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9514: [FLINK-13573][coordination] Merge 
SubmittedJobGraph into JobGraph
URL: https://github.com/apache/flink/pull/9514#issuecomment-52458
 
 
   
   ## CI report:
   
   * 6092a6154f8275957d306c9d04b780ccd374b5d9 : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124258242)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-523852882
 
 
   
   ## CI report:
   
   * a5a0a74cb3e2bef0cd7352a6cb8e7697cbd79f75 : UNKNOWN
   * bde5de0cc669dec593e090e5425f39d56c220830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124201954)
   * 63ab3d431bf8256dfb071574e013f0e878e40cef : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124203053)
   * fe6f2ab1d566100cd63941e890e2d34947ea43b8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124205327)
   * ccb85242c0c0a5f2fec18f7d67dd5293e9ad2dce : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124238221)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9478: [FLINK-13766][task] Refactor the implementation of StreamInputProcessor based on StreamTaskInput#emitNext

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9478: [FLINK-13766][task] Refactor the 
implementation of StreamInputProcessor based on StreamTaskInput#emitNext
URL: https://github.com/apache/flink/pull/9478#issuecomment-522374042
 
 
   
   ## CI report:
   
   * 1530600eaf36324966f343c277437e48c2416dc2 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123653758)
   * 8d08aaa1fedf7338ef3de48991e841f9baf7018b : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123700944)
   * 9791697c559fa39cd9f7a1c574420d62a1193545 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124210361)
   * fbd14e2320371a3cd6615deee0498d477644b954 : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124297960)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9478: [FLINK-13766][task] Refactor the implementation of StreamInputProcessor based on StreamTaskInput#emitNext

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9478: [FLINK-13766][task] Refactor the 
implementation of StreamInputProcessor based on StreamTaskInput#emitNext
URL: https://github.com/apache/flink/pull/9478#issuecomment-522374042
 
 
   
   ## CI report:
   
   * 1530600eaf36324966f343c277437e48c2416dc2 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123653758)
   * 8d08aaa1fedf7338ef3de48991e841f9baf7018b : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123700944)
   * 9791697c559fa39cd9f7a1c574420d62a1193545 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124210361)
   * fbd14e2320371a3cd6615deee0498d477644b954 : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] TisonKun commented on issue #9514: [FLINK-13573][coordination] Merge SubmittedJobGraph into JobGraph

2019-08-22 Thread GitBox
TisonKun commented on issue #9514: [FLINK-13573][coordination] Merge 
SubmittedJobGraph into JobGraph
URL: https://github.com/apache/flink/pull/9514#issuecomment-524105211
 
 
   @flinkbot run travis


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] TisonKun commented on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
TisonKun commented on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-524105145
 
 
   @flinkbot run travis


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9476: [FLINK-13764][task, metrics] Pass the counter of numRecordsIn into the constructor of StreamInputProcessor

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9476: [FLINK-13764][task, metrics] Pass the 
counter of numRecordsIn into the constructor of StreamInputProcessor
URL: https://github.com/apache/flink/pull/9476#issuecomment-522355867
 
 
   
   ## CI report:
   
   * df58cd55f5ca284e412ebbfc35c336e4f012974c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123646651)
   * 870e9199aa5afbbdea0bccffcb2fa4c2fa26ec8f : PENDING 
[Build](https://travis-ci.com/flink-ci/flink/builds/124293088)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9476: [FLINK-13764][task, metrics] Pass the counter of numRecordsIn into the constructor of StreamInputProcessor

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9476: [FLINK-13764][task, metrics] Pass the 
counter of numRecordsIn into the constructor of StreamInputProcessor
URL: https://github.com/apache/flink/pull/9476#issuecomment-522355867
 
 
   
   ## CI report:
   
   * df58cd55f5ca284e412ebbfc35c336e4f012974c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/123646651)
   * 870e9199aa5afbbdea0bccffcb2fa4c2fa26ec8f : UNKNOWN
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9514: [FLINK-13573][coordination] Merge SubmittedJobGraph into JobGraph

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9514: [FLINK-13573][coordination] Merge 
SubmittedJobGraph into JobGraph
URL: https://github.com/apache/flink/pull/9514#issuecomment-52458
 
 
   
   ## CI report:
   
   * 6092a6154f8275957d306c9d04b780ccd374b5d9 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124258242)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Assigned] (FLINK-11241) Enhance TableEnvironment to connect to a catalog via a descriptor

2019-08-22 Thread Eron Wright (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-11241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eron Wright  reassigned FLINK-11241:


Assignee: (was: Eron Wright )

> Enhance TableEnvironment to connect to a catalog via a descriptor
> -
>
> Key: FLINK-11241
> URL: https://issues.apache.org/jira/browse/FLINK-11241
> Project: Flink
>  Issue Type: Sub-task
>  Components: Table SQL / API
>Reporter: Eron Wright 
>Priority: Major
>  Labels: pull-request-available
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> Given FLINK-11240, extend {{TableEnvironment}} to connect to an external 
> catalog via an {{ExternalCatalogDescriptor}}.   Consider extending the 
> existing {{connect()}} method.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Assigned] (FLINK-11238) Enhance SQL-Client to recursively list tables

2019-08-22 Thread Eron Wright (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-11238?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eron Wright  reassigned FLINK-11238:


Assignee: (was: Eron Wright )

> Enhance SQL-Client to recursively list tables
> -
>
> Key: FLINK-11238
> URL: https://issues.apache.org/jira/browse/FLINK-11238
> Project: Flink
>  Issue Type: Sub-task
>  Components: Table SQL / Client
>Reporter: Eron Wright 
>Priority: Major
>
> The SQL Client provides a {{SHOW TABLES}} command.   Tables that are added 
> via an external catalog should be listed (presently, only the root schema is 
> listed).



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] flinkbot edited a comment on issue #8175: [FLINK-9787]Change ExecutionConfig#getGlobalJobParameters to return a…

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #8175: [FLINK-9787]Change 
ExecutionConfig#getGlobalJobParameters to return a…
URL: https://github.com/apache/flink/pull/8175#issuecomment-523993728
 
 
   
   ## CI report:
   
   * fe71dc563e7d69da189c9d74f84f34fa60486121 : SUCCESS 
[Build](https://travis-ci.com/flink-ci/flink/builds/124256028)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9507: [travis][hotfix] Travis scripts refactoring

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9507: [travis][hotfix] Travis scripts 
refactoring
URL: https://github.com/apache/flink/pull/9507#issuecomment-523838542
 
 
   
   ## CI report:
   
   * f95391f21b1fcad234145f13746822decc1bbe8c : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124195773)
   * fbf6d25c4d580a5a63b26df6435d691d0bc9fff5 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124235043)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate client/cluster-side high-availability services

2019-08-22 Thread GitBox
flinkbot edited a comment on issue #9509: [FLINK-13750][client] Separate 
client/cluster-side high-availability services
URL: https://github.com/apache/flink/pull/9509#issuecomment-523852882
 
 
   
   ## CI report:
   
   * a5a0a74cb3e2bef0cd7352a6cb8e7697cbd79f75 : UNKNOWN
   * bde5de0cc669dec593e090e5425f39d56c220830 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124201954)
   * 63ab3d431bf8256dfb071574e013f0e878e40cef : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124203053)
   * fe6f2ab1d566100cd63941e890e2d34947ea43b8 : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124205327)
   * ccb85242c0c0a5f2fec18f7d67dd5293e9ad2dce : FAILURE 
[Build](https://travis-ci.com/flink-ci/flink/builds/124238221)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] xuefuz commented on a change in pull request #9502: [FLINK-13814][hive] HiveTableSink should strip quotes from partition …

2019-08-22 Thread GitBox
xuefuz commented on a change in pull request #9502: [FLINK-13814][hive] 
HiveTableSink should strip quotes from partition …
URL: https://github.com/apache/flink/pull/9502#discussion_r316821021
 
 

 ##
 File path: 
flink-connectors/flink-connector-hive/src/main/java/org/apache/flink/connectors/hive/HiveTableSink.java
 ##
 @@ -170,7 +171,7 @@ public void setStaticPartition(Map 
partitionSpec) {
staticPartitionSpec = new LinkedHashMap<>();
for (String partitionCol : getPartitionFieldNames()) {
if (partitionSpec.containsKey(partitionCol)) {
-   staticPartitionSpec.put(partitionCol, 
partitionSpec.get(partitionCol));
+   staticPartitionSpec.put(partitionCol, 
PlanUtils.stripQuotes(partitionSpec.get(partitionCol)));
 
 Review comment:
   Just curious, partitionSpec is a map of strings, and the value should 
probably have been unquoted before getting here. I'm wondering if the fix 
should be at a lower level than here. Otherwise, similar fix will be needed 
wherever such partition spec is used.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Updated] (FLINK-13814) HiveTableSink should strip quotes from partition values

2019-08-22 Thread Bowen Li (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13814?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bowen Li updated FLINK-13814:
-
Fix Version/s: 1.9.1

> HiveTableSink should strip quotes from partition values
> ---
>
> Key: FLINK-13814
> URL: https://issues.apache.org/jira/browse/FLINK-13814
> Project: Flink
>  Issue Type: Bug
>  Components: Connectors / Hive
>Affects Versions: 1.9.0
>Reporter: Rui Li
>Assignee: Rui Li
>Priority: Major
>  Labels: pull-request-available
> Fix For: 1.10.0, 1.9.1
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Updated] (FLINK-13814) HiveTableSink should strip quotes from partition values

2019-08-22 Thread Bowen Li (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13814?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bowen Li updated FLINK-13814:
-
Affects Version/s: 1.9.0

> HiveTableSink should strip quotes from partition values
> ---
>
> Key: FLINK-13814
> URL: https://issues.apache.org/jira/browse/FLINK-13814
> Project: Flink
>  Issue Type: Bug
>  Components: Connectors / Hive
>Affects Versions: 1.9.0
>Reporter: Rui Li
>Assignee: Rui Li
>Priority: Major
>  Labels: pull-request-available
> Fix For: 1.10.0
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Assigned] (FLINK-13814) HiveTableSink should strip quotes from partition values

2019-08-22 Thread Bowen Li (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13814?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bowen Li reassigned FLINK-13814:


Assignee: Rui Li

> HiveTableSink should strip quotes from partition values
> ---
>
> Key: FLINK-13814
> URL: https://issues.apache.org/jira/browse/FLINK-13814
> Project: Flink
>  Issue Type: Bug
>  Components: Connectors / Hive
>Reporter: Rui Li
>Assignee: Rui Li
>Priority: Major
>  Labels: pull-request-available
> Fix For: 1.10.0
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Updated] (FLINK-13814) HiveTableSink should strip quotes from partition values

2019-08-22 Thread Bowen Li (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13814?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bowen Li updated FLINK-13814:
-
Fix Version/s: 1.10.0

> HiveTableSink should strip quotes from partition values
> ---
>
> Key: FLINK-13814
> URL: https://issues.apache.org/jira/browse/FLINK-13814
> Project: Flink
>  Issue Type: Bug
>  Components: Connectors / Hive
>Reporter: Rui Li
>Priority: Major
>  Labels: pull-request-available
> Fix For: 1.10.0
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[GitHub] [flink] bowenli86 edited a comment on issue #9494: [FLINK-12847] [Connectors / Kinesis] update flink-connector-kinesis to use Apache 2.0 licensed code

2019-08-22 Thread GitBox
bowenli86 edited a comment on issue #9494: [FLINK-12847] [Connectors / Kinesis] 
update flink-connector-kinesis to use Apache 2.0 licensed code
URL: https://github.com/apache/flink/pull/9494#issuecomment-523667337
 
 
   cc @zentol @tzulitai @tweise 


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Updated] (FLINK-13772) GroupWindowTableAggregateITCase tests fail on Travis

2019-08-22 Thread Alex (Jira)


 [ 
https://issues.apache.org/jira/browse/FLINK-13772?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex updated FLINK-13772:
-
Summary: GroupWindowTableAggregateITCase tests fail on Travis  (was: 
GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane)

> GroupWindowTableAggregateITCase tests fail on Travis
> 
>
> Key: FLINK-13772
> URL: https://issues.apache.org/jira/browse/FLINK-13772
> Project: Flink
>  Issue Type: Bug
>  Components: Table SQL / Planner
>Affects Versions: 1.10.0
>Reporter: Till Rohrmann
>Priority: Critical
>  Labels: test-stability
> Fix For: 1.10.0
>
>
> The 
> {{GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane}}
>  fails on Travis:
> {code}
> 05:21:41.532 [ERROR] Tests run: 18, Failures: 0, Errors: 1, Skipped: 0, Time 
> elapsed: 14.039 s <<< FAILURE! - in 
> org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase
> 05:21:41.532 [ERROR] 
> testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane[StateBackend=HEAP](org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase)
>   Time elapsed: 0.573 s  <<< ERROR!
> org.apache.flink.runtime.client.JobExecutionException: Job execution failed.
>   at 
> org.apache.flink.table.planner.runtime.stream.table.GroupWindowTableAggregateITCase.testEventTimeSlidingGroupWindowOverTimeNonOverlappingSplitPane(GroupWindowTableAggregateITCase.scala:301)
> Caused by: java.lang.Exception: Artificial Failure
> {code}
> https://api.travis-ci.org/v3/job/573615974/log.txt



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


  1   2   3   4   5   >