Github user asfgit closed the pull request at:
https://github.com/apache/drill/pull/993
---
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r146112692
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/WebUserConnection.java
---
@@ -124,6 +124,13 @@ public void
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r145565896
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRestServer.java
---
@@ -204,9 +217,15 @@ public WebUserConnection provide() {
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r145565968
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRestServer.java
---
@@ -260,8 +282,14 @@ public WebUserConnection provide() {
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r145568772
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/WebUserConnection.java
---
@@ -124,6 +124,13 @@ public void
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r145565944
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRestServer.java
---
@@ -204,9 +217,15 @@ public WebUserConnection provide() {
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r145571583
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/server/rest/WebSessionResourcesTest.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r145566366
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/WebSessionResources.java
---
@@ -68,16 +68,19 @@ public SocketAddress
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r145495729
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRestServer.java
---
@@ -204,9 +217,15 @@ public WebUserConnection provide() {
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r144930316
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/WebUserConnection.java
---
@@ -124,6 +124,13 @@ public void
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r144930573
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/server/rest/WebSessionResourcesTest.java
---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r144929270
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRestServer.java
---
@@ -260,8 +282,14 @@ public WebUserConnection
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r144929150
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRestServer.java
---
@@ -204,9 +217,15 @@ public WebUserConnection
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r144928933
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRestServer.java
---
@@ -204,9 +217,15 @@ public WebUserConnection
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r144929812
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/WebSessionResources.java
---
@@ -68,16 +68,19 @@ public SocketAddress
GitHub user sohami opened a pull request:
https://github.com/apache/drill/pull/993
DRILL-5874: NPE in AnonWebUserConnection.cleanupSession()
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sohami/drill DRILL-5874
Alternatively
16 matches
Mail list logo