[Impala-ASF-CR] IMPALA-2397: Use atomics for metrics

2018-01-17 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9012 ) Change subject: IMPALA-2397: Use atomics for metrics .. Patch Set 2: Code-Review+1 I don't have further comments, thanks for the inline r

[Impala-ASF-CR] IMPALA-2397: Use atomics for metrics

2018-01-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9012 ) Change subject: IMPALA-2397: Use atomics for metrics .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/9012/1/be/src/ut

[Impala-ASF-CR] IMPALA-2397: Use atomics for metrics

2018-01-17 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/9012 ) Change subject: IMPALA-2397: Use atomics for metrics .. Patch Set 2: (10 comments) http://gerrit.cloudera.org:8080/#/c/9012/1/be/src/util

[Impala-ASF-CR] IMPALA-2397: Use atomics for metrics

2018-01-16 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/9012 ) Change subject: IMPALA-2397: Use atomics for metrics .. IMPALA-2397: Use atomics for metrics This change removes the spinlock in IntGauge an

[Impala-ASF-CR] IMPALA-2397: Use atomics for metrics

2018-01-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9012 ) Change subject: IMPALA-2397: Use atomics for metrics .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/9012/1/be/src/util

[Impala-ASF-CR] IMPALA-2397: Use atomics for metrics

2018-01-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9012 ) Change subject: IMPALA-2397: Use atomics for metrics .. Patch Set 1: (6 comments) This should be a big improvement. http://gerrit.clo

[Impala-ASF-CR] IMPALA-2397: Use atomics for metrics

2018-01-11 Thread Michael Ho (Code Review)
Michael Ho has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9012 Change subject: IMPALA-2397: Use atomics for metrics .. IMPALA-2397: Use atomics for metrics This change removes the spinlock in IntGauge a