[Impala-ASF-CR] IMPALA-3882: Simplify some query exec state locking

2017-03-07 Thread Henry Robinson (Code Review)
Henry Robinson has abandoned this change. Change subject: IMPALA-3882: Simplify some query exec state locking .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4935 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3882: Simplify some query exec state locking

2017-01-06 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3882: Simplify some query exec state locking .. Patch Set 6: (1 comment) Abandoned this for now: Sailesh's comment lead me down a rat hole that I don't

[Impala-ASF-CR] IMPALA-3882: Simplify some query exec state locking

2016-11-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3882: Simplify some query exec state locking .. Patch Set 6: (3 comments)

[Impala-ASF-CR] IMPALA-3882: Simplify some query exec state locking

2016-11-14 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3882: Simplify some query exec state locking .. Patch Set 5: (9 comments)

[Impala-ASF-CR] IMPALA-3882: Simplify some query exec state locking

2016-11-10 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3882: Simplify some query exec state locking .. Patch Set 4: (9 comments) Here's how I think about the lock safety (sorry, long, but hopefully helps to

[Impala-ASF-CR] IMPALA-3882: Simplify some query exec state locking

2016-11-04 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3882: Simplify some query exec state locking .. Patch Set 4: (6 comments)

[Impala-ASF-CR] IMPALA-3882: Simplify some query exec state locking

2016-11-03 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new patch set (#4). Change subject: IMPALA-3882: Simplify some query exec state locking .. IMPALA-3882: Simplify some query exec state locking query_exec_state_map_lock_ is implicated in many

[Impala-ASF-CR] IMPALA-3882: Simplify some query exec state locking

2016-11-03 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/4935 Change subject: IMPALA-3882: Simplify some query exec state locking .. IMPALA-3882: Simplify some query exec state locking