[GitHub] incubator-livy issue #116: [LIVY-520] Add log when session is GC-ed

2018-10-01 Thread codecov-io
Github user codecov-io commented on the issue: https://github.com/apache/incubator-livy/pull/116 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/116?src=pr=h1) Report > Merging [#116](https://codecov.io/gh/apache/incubator-livy/pull/116?src=pr=desc) into

[GitHub] incubator-livy pull request #116: [LIVY-520] Add log when session is GC-ed

2018-10-01 Thread mgaido91
Github user mgaido91 closed the pull request at: https://github.com/apache/incubator-livy/pull/116 ---

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221768562 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/auth/PlainSaslServer.scala --- @@ -0,0 +1,184 @@ +/* + * Licensed

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221771929 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/cli/ThriftHttpCLIService.scala --- @@ -0,0 +1,210 @@ +/* + *

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221755078 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/LivyOperationManager.scala --- @@ -233,12 +232,5 @@ class

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221768243 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/auth/LivyDelegationTokenSecretManager.scala --- @@ -0,0 +1,81 @@ +/*

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221764487 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/auth/AuthBridgeServer.scala --- @@ -0,0 +1,296 @@ +/* + * Licensed

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221771643 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/cli/ThriftCLIService.scala --- @@ -0,0 +1,745 @@ +/* + * Licensed

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221772645 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/cli/ThriftHttpServlet.scala --- @@ -0,0 +1,500 @@ +/* + * Licensed

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221764252 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/auth/AuthBridgeServer.scala --- @@ -0,0 +1,296 @@ +/* + * Licensed

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221767779 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/auth/AuthenticationProvider.scala --- @@ -0,0 +1,74 @@ +/* + *

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221772907 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/operation/GetTypeInfoOperation.scala --- @@ -0,0 +1,130 @@ +/* + *

[GitHub] incubator-livy pull request #117: [WIP][LIVY-502] Remove dependency on hive-...

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r221762741 --- Diff: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/LivyThriftServer.scala --- @@ -114,24 +98,56 @@ object LivyThriftServer

[GitHub] incubator-livy pull request #113: [LIVY-500] Add beeline client for dev

2018-10-01 Thread mgaido91
Github user mgaido91 closed the pull request at: https://github.com/apache/incubator-livy/pull/113 ---

[GitHub] incubator-livy pull request #114: [LIVY-495] Add thriftserver UI

2018-10-01 Thread mgaido91
GitHub user mgaido91 reopened a pull request: https://github.com/apache/incubator-livy/pull/114 [LIVY-495] Add thriftserver UI ## What changes were proposed in this pull request? The PR adds a new UI tab dedicated to thrift-server sessions. The UI shows the active sessions

[GitHub] incubator-livy pull request #113: [LIVY-500] Add beeline client for dev

2018-10-01 Thread mgaido91
GitHub user mgaido91 reopened a pull request: https://github.com/apache/incubator-livy/pull/113 [LIVY-500] Add beeline client for dev ## What changes were proposed in this pull request? The PR adds a new `thriftserver/client` module and a `beeline` script which are useful

[GitHub] incubator-livy pull request #114: [LIVY-495] Add thriftserver UI

2018-10-01 Thread mgaido91
Github user mgaido91 closed the pull request at: https://github.com/apache/incubator-livy/pull/114 ---

[GitHub] incubator-livy pull request #116: [LIVY-520] Add log when session is GC-ed

2018-10-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-livy/pull/116 ---

[GitHub] incubator-livy pull request #113: [LIVY-500] Add beeline client for dev

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/113#discussion_r221750636 --- Diff: thriftserver/client/pom.xml --- @@ -0,0 +1,66 @@ + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] incubator-livy pull request #113: [LIVY-500] Add beeline client for dev

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/113#discussion_r221750204 --- Diff: thriftserver/client/pom.xml --- @@ -0,0 +1,66 @@ + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] incubator-livy issue #118: [LIVY-503] Separate thrift server session code in...

2018-10-01 Thread codecov-io
Github user codecov-io commented on the issue: https://github.com/apache/incubator-livy/pull/118 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/118?src=pr=h1) Report > Merging [#118](https://codecov.io/gh/apache/incubator-livy/pull/118?src=pr=desc) into

[GitHub] incubator-livy issue #116: [LIVY-520] Add log when session is GC-ed

2018-10-01 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/incubator-livy/pull/116 Merging to master. ---

[GitHub] incubator-livy pull request #114: [LIVY-495] Add thriftserver UI

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/114#discussion_r221751897 --- Diff: server/src/main/scala/org/apache/livy/server/LivyServer.scala --- @@ -211,10 +217,13 @@ class LivyServer extends Logging {

[GitHub] incubator-livy pull request #114: [LIVY-495] Add thriftserver UI

2018-10-01 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/114#discussion_r221752067 --- Diff: server/src/main/scala/org/apache/livy/server/ThriftServerFactory.scala --- @@ -30,6 +32,8 @@ trait ThriftServerFactory {

[GitHub] incubator-livy pull request #118: [LIVY-503] Separate thrift server session ...

2018-10-01 Thread vanzin
GitHub user vanzin opened a pull request: https://github.com/apache/incubator-livy/pull/118 [LIVY-503] Separate thrift server session code in separate module. This change creates a new module ("livy-thriftserver-session") with the code related to the Thrift server that is