[jira] [Commented] (FLINK-10964) sql-client throws exception when paging through finished batch query

2018-12-10 Thread vinoyang (JIRA)


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

vinoyang commented on FLINK-10964:
--

[~twalthr] when you have time, can you help to review the PR of this issue? By 
analyzing the source code, I found that the ResultRestore has been emptied at 
the time of cancelQuery. The main problem with this exception is that when 
stopping the retrieval thread, it still tries to call cancelQuery regardless of 
the running state of the job.

> sql-client throws exception when paging through finished batch query 
> -
>
> Key: FLINK-10964
> URL: https://issues.apache.org/jira/browse/FLINK-10964
> Project: Flink
>  Issue Type: Bug
>  Components: SQL Client
>Reporter: Seth Wiesman
>Assignee: vinoyang
>Priority: Major
>  Labels: pull-request-available
>
> When paging through a batch query in state 'Finished' the sql client throws 
> the following exception: 
> {code:java}
> org.apache.flink.table.client.gateway.SqlExecutionException: Could not find a 
> result with result identifier '0c7dce30d287fdd13b934fbefe5a38d1'.{code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (FLINK-10964) sql-client throws exception when paging through finished batch query

2018-12-09 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on FLINK-10964:


yanghua opened a new pull request #7265: [FLINK-10964] SQL-client throws 
exception when paging through finished batch query
URL: https://github.com/apache/flink/pull/7265
 
 
   ## What is the purpose of the change
   
   *This pull request fixes SQL-client throws exception when paging through 
finished batch query*
   
   
   ## Brief change log
   
 - *Fix SQL-client throws exception when paging through finished batch 
query*
   
   ## Verifying this change
   
   This change added tests and can be verified as follows:
   
 - *testRetrieveBoundedStreamQueryExecutionChangelog()*
 - *testRetrieveBoundedStreamQueryExecutionTable()*
 - *testRetrieveResultAfterBatchQueryFinished*
   
   ## Does this pull request potentially affect one of the following parts:
   
 - Dependencies (does it add or upgrade a dependency): (yes / **no**)
 - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (yes / **no**)
 - The serializers: (yes / **no** / don't know)
 - The runtime per-record code paths (performance sensitive): (yes / **no** 
/ don't know)
 - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / **no** / don't know)
 - The S3 file system connector: (yes / **no** / don't know)
   
   ## Documentation
   
 - Does this pull request introduce a new feature? (yes / **no**)
 - If yes, how is the feature documented? (not applicable / docs / JavaDocs 
/ **not documented**)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


> sql-client throws exception when paging through finished batch query 
> -
>
> Key: FLINK-10964
> URL: https://issues.apache.org/jira/browse/FLINK-10964
> Project: Flink
>  Issue Type: Bug
>  Components: SQL Client
>Reporter: Seth Wiesman
>Assignee: vinoyang
>Priority: Major
>  Labels: pull-request-available
>
> When paging through a batch query in state 'Finished' the sql client throws 
> the following exception: 
> {code:java}
> org.apache.flink.table.client.gateway.SqlExecutionException: Could not find a 
> result with result identifier '0c7dce30d287fdd13b934fbefe5a38d1'.{code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (FLINK-10964) sql-client throws exception when paging through finished batch query

2018-12-06 Thread vinoyang (JIRA)


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

vinoyang commented on FLINK-10964:
--

[~twalthr]  Thank you for providing more information. I personally do not tend 
to introduce the timer, there are already a lot of configuration items, we can 
delete the result of the job directly when calling cacelQuery.

> sql-client throws exception when paging through finished batch query 
> -
>
> Key: FLINK-10964
> URL: https://issues.apache.org/jira/browse/FLINK-10964
> Project: Flink
>  Issue Type: Bug
>  Components: SQL Client
>Reporter: Seth Wiesman
>Assignee: vinoyang
>Priority: Major
>
> When paging through a batch query in state 'Finished' the sql client throws 
> the following exception: 
> {code:java}
> org.apache.flink.table.client.gateway.SqlExecutionException: Could not find a 
> result with result identifier '0c7dce30d287fdd13b934fbefe5a38d1'.{code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (FLINK-10964) sql-client throws exception when paging through finished batch query

2018-12-06 Thread Timo Walther (JIRA)


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

Timo Walther commented on FLINK-10964:
--

I can give more information here. The issue is rather a conceptual problem. 
Currently, we only maintain results until a job has finished. If the job is 
bounded (for batch or streaming with bounded sources), the result store is 
cleaned up even though the CLI still wants to page through the result. A 
solution would be:
- keep results in {{ResultStore}} even if query has been stopped
- delete result when {{Executor#cancelQuery}} is called
- or by a timeout that has been specified using the "execution" properties 
(meaning we need a timer service in the result store).

> sql-client throws exception when paging through finished batch query 
> -
>
> Key: FLINK-10964
> URL: https://issues.apache.org/jira/browse/FLINK-10964
> Project: Flink
>  Issue Type: Bug
>  Components: SQL Client
>Reporter: Seth Wiesman
>Assignee: vinoyang
>Priority: Major
>
> When paging through a batch query in state 'Finished' the sql client throws 
> the following exception: 
> {code:java}
> org.apache.flink.table.client.gateway.SqlExecutionException: Could not find a 
> result with result identifier '0c7dce30d287fdd13b934fbefe5a38d1'.{code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (FLINK-10964) sql-client throws exception when paging through finished batch query

2018-12-06 Thread vinoyang (JIRA)


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

vinoyang commented on FLINK-10964:
--

[~sjwiesman] Can you provide more details and context about the exception? 
thanks.

> sql-client throws exception when paging through finished batch query 
> -
>
> Key: FLINK-10964
> URL: https://issues.apache.org/jira/browse/FLINK-10964
> Project: Flink
>  Issue Type: Bug
>  Components: SQL Client
>Reporter: Seth Wiesman
>Assignee: vinoyang
>Priority: Major
>
> When paging through a batch query in state 'Finished' the sql client throws 
> the following exception: 
> {code:java}
> org.apache.flink.table.client.gateway.SqlExecutionException: Could not find a 
> result with result identifier '0c7dce30d287fdd13b934fbefe5a38d1'.{code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)