[GitHub] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/993 ---

[GitHub] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-21 Thread sohami
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-18 Thread sohami
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-18 Thread sohami
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-18 Thread sohami
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-18 Thread sohami
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-18 Thread sohami
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-18 Thread sohami
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-18 Thread sohami
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-16 Thread paul-rogers
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-16 Thread paul-rogers
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-16 Thread paul-rogers
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-16 Thread paul-rogers
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-16 Thread paul-rogers
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-16 Thread paul-rogers
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] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

2017-10-13 Thread sohami
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