[Impala-ASF-CR] IMPALA-5427: Fix order of status handling in ClientRequestState

2017-06-12 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5427: Fix order of status handling in ClientRequestState .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-5427: Fix order of status handling in ClientRequestState

2017-06-12 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-5427: Fix order of status handling in ClientRequestState .. IMPALA-5427: Fix order of status handling in ClientRequestState There was a race in

[Impala-ASF-CR] IMPALA-5427: Fix order of status handling in ClientRequestState

2017-06-12 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5427: Fix order of status handling in ClientRequestState .. Patch Set 1: > > (1 comment) > > > > This is really subtle. In order to be correct, you'd need

[Impala-ASF-CR] IMPALA-5427: Fix order of status handling in ClientRequestState

2017-06-12 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5427: Fix order of status handling in ClientRequestState .. Patch Set 1: -Code-Review > (1 comment) > > This is really subtle. In order to be correct,

[Impala-ASF-CR] IMPALA-5427: Fix order of status handling in ClientRequestState

2017-06-12 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5427: Fix order of status handling in ClientRequestState .. Patch Set 1: (1 comment) This is really subtle. In order to be correct, you'd need at least a

[Impala-ASF-CR] IMPALA-5427: Fix order of status handling in ClientRequestState

2017-06-12 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5427: Fix order of status handling in ClientRequestState .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7155 To

[Impala-ASF-CR] IMPALA-5427: Fix order of status handling in ClientRequestState

2017-06-12 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/7155 Change subject: IMPALA-5427: Fix order of status handling in ClientRequestState .. IMPALA-5427: Fix order of status handling in