You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@livy.apache.org by GitBox <gi...@apache.org> on 2020/02/10 01:20:04 UTC

[GitHub] [incubator-livy] RogPodge opened a new pull request #278: [LIVY-748] Add support for running Livy Integration tests against secure external clusters

RogPodge opened a new pull request #278: [LIVY-748] Add support for running Livy Integration tests against secure external clusters
URL: https://github.com/apache/incubator-livy/pull/278
 
 
   ## What changes were proposed in this pull request?
   
   https://issues.apache.org/jira/projects/LIVY/issues/LIVY-748?filter=reportedbyme
   
   This PR configures the integration tests to work on Livy Servers instances that require basic or kerberos authentication. To do this we had to replace the async httpclient with apaches normal httpclient, as there were issues found getting kerberos authentication to work with the async client.
   
   This PR also contains a minor fix to the InteractiveIT test suite. One of the tests would occasionally fail as a statement submitted to the interactive session would not be finished processing when the result was checked. The correctness check was also changed to correctly check for code completition
   
   ## How was this patch tested?
   This patch was tested on local deployments of livy server, both on a instance that required kerberos authentication and on one that required basic authentication. 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services