[Impala-ASF-CR] IMPALA-7963: Don't underflow uint64 t when time goes backwards

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12091 ) Change subject: IMPALA-7963: Don't underflow uint64_t when time goes backwards .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7864: Deflake test replan limit

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12105 ) Change subject: IMPALA-7864: Deflake test_replan_limit .. Patch Set 1: Build Successful

[Impala-ASF-CR] IMPALA-7990: make test lifecycle failures more permissive

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12103 ) Change subject: IMPALA-7990: make test_lifecycle_failures more permissive .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7990: make test lifecycle failures more permissive

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12103 ) Change subject: IMPALA-7990: make test_lifecycle_failures more permissive .. IMPALA-7990: make test_lifecycle_failures

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 9: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. IMPALA-7948: part 1: initial docker container build

[Impala-ASF-CR] IMPALA-7864: Deflake test replan limit

2018-12-17 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12105 Change subject: IMPALA-7864: Deflake test_replan_limit .. IMPALA-7864: Deflake test_replan_limit - Doubles the number of

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 9: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7990: make test lifecycle failures more permissive

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12103 ) Change subject: IMPALA-7990: make test_lifecycle_failures more permissive .. Patch Set 1: Build Successful

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12100 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Remove Python 2.4 workarounds in

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12100 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7963: Don't underflow uint64 t when time goes backwards

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12091 ) Change subject: IMPALA-7963: Don't underflow uint64_t when time goes backwards .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 9: Build Successful

[Impala-ASF-CR] IMPALA-7990: make test lifecycle failures more permissive

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12103 ) Change subject: IMPALA-7990: make test_lifecycle_failures more permissive .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-7990: make test lifecycle failures more permissive

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12103 ) Change subject: IMPALA-7990: make test_lifecycle_failures more permissive .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7990: make test lifecycle failures more permissive

2018-12-17 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12103 Change subject: IMPALA-7990: make test_lifecycle_failures more permissive .. IMPALA-7990: make test_lifecycle_failures more

[Impala-ASF-CR] IMPALA-4555: Make QueryState's status reporting more robust

2018-12-17 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12049 ) Change subject: IMPALA-4555: Make QueryState's status reporting more robust .. Patch Set 2: (8 comments) Some trivial comments for now.

[Impala-ASF-CR] IMPALA-7183: Include remote host when logging unknown execution status.

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12102 ) Change subject: IMPALA-7183: Include remote host when logging unknown execution status. .. Patch Set 1: Build Successful

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 9: Build started:

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12074 to look at the new patch set (#9). Change subject: IMPALA-7948: part 1: initial docker container build

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Found a bug in the build scripts - forgot to

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Build failed:

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Build failed:

[Impala-ASF-CR] IMPALA-7183: Include remote host when logging unknown execution status.

2018-12-17 Thread Andrew Sherman (Code Review)
Andrew Sherman has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12102 Change subject: IMPALA-7183: Include remote host when logging unknown execution status. .. IMPALA-7183: Include remote host when

[Impala-ASF-CR] IMPALA-7980: Fix spinning threads because of buggy handling of num unqueued files .

2018-12-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12097 ) Change subject: IMPALA-7980: Fix spinning threads because of buggy handling of num_unqueued_files_. .. Patch Set 1: Would be nice to

[Impala-ASF-CR] IMPALA-7694: Add host resource usage metrics to profile

2018-12-17 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12069 ) Change subject: IMPALA-7694: Add host resource usage metrics to profile .. Patch Set 7: (3 comments)

[Impala-ASF-CR] IMPALA-7980: Fix spinning threads because of buggy handling of num unqueued files .

2018-12-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12097 ) Change subject: IMPALA-7980: Fix spinning threads because of buggy handling of num_unqueued_files_. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7980: Fix spinning threads because of buggy handling of num unqueued files .

2018-12-17 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12097 ) Change subject: IMPALA-7980: Fix spinning threads because of buggy handling of num_unqueued_files_. .. Patch Set 1: (3 comments)

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Build started:

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Build started:

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Hit IMPALA-7990 and another unrelated failure

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Verified-1 Build failed:

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12100 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 1: Build started:

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-17 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/12100 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12100 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 1: Build Successful

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12100 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 1: I reran and confirmed it worked this time

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12100 Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Remove Python 2.4 workarounds in start-impala-cluster.py

[Impala-ASF-CR] IMPALA-7963: Don't underflow uint64 t when time goes backwards

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12091 ) Change subject: IMPALA-7963: Don't underflow uint64_t when time goes backwards .. Patch Set 2: Build Successful

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 7: Build Successful

[Impala-ASF-CR] IMPALA-7931: fix executor shutdown races

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12082 ) Change subject: IMPALA-7931: fix executor shutdown races .. Patch Set 7: Build Successful

[Impala-ASF-CR] IMPALA-7931: fix executor shutdown races

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12082 ) Change subject: IMPALA-7931: fix executor shutdown races .. Patch Set 6: Build Successful

[Impala-ASF-CR] IMPALA-7980: Fix spinning threads because of buggy handling of num unqueued files .

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12097 ) Change subject: IMPALA-7980: Fix spinning threads because of buggy handling of num_unqueued_files_. .. Patch Set 1: (5 comments) No

[Impala-ASF-CR] IMPALA-7963: Don't underflow uint64 t when time goes backwards

2018-12-17 Thread Joe McDonnell (Code Review)
Hello Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12091 to look at the new patch set (#2). Change subject: IMPALA-7963: Don't underflow uint64_t when time goes backwards

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Build started:

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 8: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12074 to look at the new patch set (#7). Change subject: IMPALA-7948: part 1: initial docker container build

[Impala-ASF-CR] IMPALA-7931: fix executor shutdown races

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/12082 ) Change subject: IMPALA-7931: fix executor shutdown races .. IMPALA-7931: fix executor shutdown races There were two races: * queries

[Impala-ASF-CR] IMPALA-7931: fix executor shutdown races

2018-12-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12082 ) Change subject: IMPALA-7931: fix executor shutdown races .. Patch Set 6: (2 comments)

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 6: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 6: Any more comment? -- To view, visit

[Impala-ASF-CR] IMPALA-7931: fix executor shutdown races

2018-12-17 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12082 to look at the new patch set (#7). Change subject: IMPALA-7931: fix executor shutdown races ..

[Impala-ASF-CR] Added timeout to run-all-tests

2018-12-17 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/12086 ) Change subject: Added timeout to run-all-tests .. Patch Set 1: (2 comments)

[Impala-ASF-CR] WIP: IMPALA-5843: Use page index in Parquet files to skip pages

2018-12-17 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/12065 ) Change subject: WIP: IMPALA-5843: Use page index in Parquet files to skip pages .. Patch Set 1: (15 comments)

[Impala-ASF-CR] IMPALA-7963: Don't underflow uint64 t when time goes backwards

2018-12-17 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12091 ) Change subject: IMPALA-7963: Don't underflow uint64_t when time goes backwards .. Patch Set 1: (1 comment)