[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4892: Session ID included in error message .. IMPALA-4892: Session ID included in error message Included the Session ID in the invalid

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4892: Session ID included in error message .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7054 To unsubscribe,

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4892: Session ID included in error message .. Patch Set 5: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/708/ -- To

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Steve Carlin (Code Review)
Steve Carlin has uploaded a new patch set (#5). Change subject: IMPALA-4892: Session ID included in error message .. IMPALA-4892: Session ID included in error message Included the Session ID in the invalid session ID message.

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Steve Carlin (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7054 to look at the new patch set (#5). Change subject: IMPALA-4892: Session ID included in error message ..

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4892: Session ID included in error message .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7054/4/be/src/service/impala-hs2-server.cc

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Steve Carlin (Code Review)
Steve Carlin has uploaded a new patch set (#4). Change subject: IMPALA-4892: Session ID included in error message .. IMPALA-4892: Session ID included in error message Included the Session ID in the invalid session ID message.

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Steve Carlin (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7054 to look at the new patch set (#4). Change subject: IMPALA-4892: Session ID included in error message ..

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-08 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4892: Session ID included in error message .. Patch Set 3: Code-Review+2 (1 comment) Looks good - please fix the long line, then I'll commit. Thanks!

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-06 Thread Steve Carlin (Code Review)
Steve Carlin has uploaded a new patch set (#3). Change subject: IMPALA-4892: Session ID included in error message .. IMPALA-4892: Session ID included in error message Included the Session ID in the invalid session ID message.

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-04 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4892: Session ID included in error message .. Patch Set 2: Thanks Steve! This is fine to go in as is. I noticed that there are a few other places in

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-02 Thread Steve Carlin (Code Review)
Steve Carlin has posted comments on this change. Change subject: IMPALA-4892: Session ID included in error message .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7054/1/be/src/service/impala-server.cc File

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-01 Thread Steve Carlin (Code Review)
Steve Carlin has abandoned this change. Change subject: IMPALA-4892: Session ID included in error message .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7051 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-01 Thread Steve Carlin (Code Review)
Steve Carlin has uploaded a new patch set (#2). Change subject: IMPALA-4892: Session ID included in error message .. IMPALA-4892: Session ID included in error message Included the Session ID in the invalid session ID message.

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-01 Thread Steve Carlin (Code Review)
Steve Carlin has uploaded a new patch set (#2). Change subject: IMPALA-4892: Session ID included in error message .. IMPALA-4892: Session ID included in error message Included the Session ID in the invalid session ID message.

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-01 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4892: Session ID included in error message .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7054/1/be/src/service/impala-server.cc File

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-01 Thread Steve Carlin (Code Review)
Steve Carlin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7054 Change subject: IMPALA-4892: Session ID included in error message .. IMPALA-4892: Session ID included in error message Included the

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-01 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4892: Session ID included in error message .. Patch Set 1: (1 comment) Thanks for the patch! Just one minor comment.

[Impala-ASF-CR] IMPALA-4892: Session ID included in error message

2017-06-01 Thread Steve Carlin (Code Review)
Steve Carlin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7051 Change subject: IMPALA-4892: Session ID included in error message .. IMPALA-4892: Session ID included in error message Included the