[jira] [Resolved] (LIVY-533) Spark jobs submitted via programmatic API cannot always be canceled

2018-12-03 Thread Marcelo Vanzin (JIRA)


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

Marcelo Vanzin resolved LIVY-533.
-
   Resolution: Fixed
Fix Version/s: 0.6.0

Issue resolved by pull request 128
[https://github.com/apache/incubator-livy/pull/128]

>  Spark jobs submitted via programmatic API cannot always be canceled 
> -
>
> Key: LIVY-533
> URL: https://issues.apache.org/jira/browse/LIVY-533
> Project: Livy
>  Issue Type: Bug
>  Components: RSC
>Affects Versions: 0.5.0
>Reporter: Björn Lohrmann
>Priority: Major
>  Labels: pull-request-available
> Fix For: 0.6.0
>
>
> Running stages of Spark jobs submitted via Livy' programmatic API cannot 
> (always) be successfully cancelled.
> The current implementation of .JobWrapper.cancel() interrupts the worker 
> thread on the Spark driver (via Future.cancel(true)):
> [https://github.com/apache/incubator-livy/blob/4cfb6bcb8fb9ac6b2d6c8b3d04b20f647b507e1f/rsc/src/main/java/org/apache/livy/rsc/driver/JobWrapper.java#L84]
> This does not always cancel all activity in Spark, e.g. long-running stages 
> may remain unaffected.
> The Spark-way of cancelling jobs seems to be via 
> SparkContext.setJobGroup()/cancelJobGroup(), which is also being used in 
> Livy's REPL Session:
> [https://github.com/apache/incubator-livy/blob/4cfb6bcb8fb9ac6b2d6c8b3d04b20f647b507e1f/repl/src/main/scala/org/apache/livy/repl/Session.scala#L164]
> I have opened a PR that invokes setJobGroup()/cancelJobGroup() in addition to 
> interrupting the worker thread running on the driver:
> [https://github.com/apache/incubator-livy/pull/128]
>  
> It would be great if the fix could make it into the 0.6 release.
>  



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


[jira] [Assigned] (LIVY-538) Add integration tests for the thrift server

2018-12-03 Thread Marco Gaido (JIRA)


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

Marco Gaido reassigned LIVY-538:


Assignee: Marco Gaido

> Add integration tests for the thrift server
> ---
>
> Key: LIVY-538
> URL: https://issues.apache.org/jira/browse/LIVY-538
> Project: Livy
>  Issue Type: Sub-task
>  Components: Server, Tests
>Affects Versions: 0.6.0
>Reporter: Marcelo Vanzin
>Assignee: Marco Gaido
>Priority: Major
>
> Currently there are no integration tests for the thrift server part. The UTs 
> are run on a different environment than ITs, which are more similar to a real 
> cluster, and having ITs would allow us to catch issues in the code that are 
> masked in UTs, such as serialization issues or referencing unavailable 
> classes.



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


[jira] [Commented] (LIVY-538) Add integration tests for the thrift server

2018-12-03 Thread Marco Gaido (JIRA)


[ 
https://issues.apache.org/jira/browse/LIVY-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16707357#comment-16707357
 ] 

Marco Gaido commented on LIVY-538:
--

Created PR: https://github.com/apache/incubator-livy/pull/131.

> Add integration tests for the thrift server
> ---
>
> Key: LIVY-538
> URL: https://issues.apache.org/jira/browse/LIVY-538
> Project: Livy
>  Issue Type: Sub-task
>  Components: Server, Tests
>Affects Versions: 0.6.0
>Reporter: Marcelo Vanzin
>Assignee: Marco Gaido
>Priority: Major
>
> Currently there are no integration tests for the thrift server part. The UTs 
> are run on a different environment than ITs, which are more similar to a real 
> cluster, and having ITs would allow us to catch issues in the code that are 
> masked in UTs, such as serialization issues or referencing unavailable 
> classes.



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