Github user asfgit closed the pull request at:
https://github.com/apache/incubator-livy/pull/131
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r245124550
--- Diff: server/src/main/scala/org/apache/livy/server/LivyServer.scala ---
@@ -355,6 +355,21 @@ class LivyServer extends Logging {
_serverUrl
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r245102371
--- Diff: server/src/main/scala/org/apache/livy/server/LivyServer.scala ---
@@ -355,6 +355,21 @@ class LivyServer extends Logging {
_serverU
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r245096875
--- Diff: server/src/main/scala/org/apache/livy/server/LivyServer.scala ---
@@ -355,6 +355,21 @@ class LivyServer extends Logging {
_serverUrl
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r244974668
--- Diff:
thriftserver/server/src/main/scala/org/apache/livy/thriftserver/LivyThriftServer.scala
---
@@ -138,6 +139,16 @@ class LivyThriftServer(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r244836308
--- Diff:
thriftserver/server/src/main/scala/org/apache/livy/thriftserver/LivyThriftServer.scala
---
@@ -138,6 +139,16 @@ class LivyThriftServer(
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r243816737
--- Diff:
thriftserver/server/src/main/scala/org/apache/livy/thriftserver/LivyThriftServer.scala
---
@@ -138,6 +139,16 @@ class LivyThriftServer(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r243083151
--- Diff:
integration-test/src/thriftserver/main/scala/org/apache/livy/test/framework/ThriftJdbcClient.scala
---
@@ -0,0 +1,51 @@
+/*
+ * Lic
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r243081435
--- Diff:
integration-test/src/thriftserver/main/scala/org/apache/livy/test/framework/ThriftJdbcClient.scala
---
@@ -0,0 +1,51 @@
+/*
+ * Lic
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r243081744
--- Diff:
integration-test/src/thriftserver/main/scala/org/apache/livy/test/framework/ThriftJdbcClient.scala
---
@@ -0,0 +1,51 @@
+/*
+ * Lic
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r243084756
--- Diff:
thriftserver/server/src/main/scala/org/apache/livy/thriftserver/LivyThriftServer.scala
---
@@ -138,6 +139,16 @@ class LivyThriftServer(
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r242929093
--- Diff:
thriftserver/integration-test/src/test/scala/org/apache/livy/test/JdbcIT.scala
---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apach
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/131#discussion_r242756706
--- Diff:
thriftserver/integration-test/src/test/scala/org/apache/livy/test/JdbcIT.scala
---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache
GitHub user mgaido91 opened a pull request:
https://github.com/apache/incubator-livy/pull/131
[LIVY-538] Add integration tests for thriftserver
## What changes were proposed in this pull request?
Added a new module containing ITs for the thriftserver. The current tests
cont
14 matches
Mail list logo