[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. IMPALA-4735: Upgrade pytest in python env to version 2.9.2. The current version of pytest in

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5640 To unsubscribe,

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Fix typo in link text .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5881 To unsubscribe, visit

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Fix typo in link text .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5881 To unsubscribe, visit

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

2017-02-03 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Release note updates for Impala 2.8 .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5668/6/docs/topics/impala_incompatible_changes.xml File

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-01-31 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5829/1/.pep8rc File .pep8rc: PS1, Line 5: # E251 - Remove

[Impala-ASF-CR] Release note updates for Impala 2.8

2017-01-31 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Release note updates for Impala 2.8 .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/5668/7/docs/shared/impala_common.xml File

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-01-31 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5829/1/.pep8rc File .pep8rc: PS1, Line 5: # E251 - Remove

[Impala-ASF-CR] Release note updates for Impala 2.8

2017-01-31 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Release note updates for Impala 2.8 .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5668/6/docs/topics/impala_incompatible_changes.xml File

[Impala-ASF-CR] Add a build flag for the undefined behavior sanitizer, aka "ubsan".

2017-02-04 Thread Jim Apple (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5082 to look at the new patch set (#5). Change subject: Add a build flag for the undefined behavior sanitizer, aka "ubsan".

[Impala-ASF-CR] Add a build flag for the undefined behavior sanitizer, aka "ubsan".

2017-02-04 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add a build flag for the undefined behavior sanitizer, aka "ubsan". .. Patch Set 5: Code-Review+1 (18 comments) Carry Tim's +1.

[Impala-ASF-CR] IMPALA-3909 (follow-up): Properly qualify min() and max() in header

2017-02-07 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3909 (follow-up): Properly qualify min() and max() in header .. Patch Set 2: Code-Review+2 (1 comment) Thank you for keeping the code tidy, Henry!

[Impala-ASF-CR] IMPALA-3909 (follow-up): Properly qualify min() and max() in header

2017-02-07 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3909 (follow-up): Properly qualify min() and max() in header .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5916/2//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala security topics to be generic

2017-02-08 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala security topics to be generic .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5931 To unsubscribe,

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4876: Remove _test suffix from test names .. Patch Set 1: Do you want me to start the pre-merge Jenkins test? -- To view, visit

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4876: Remove _test suffix from test names .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5886 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3909 (follow-up): Properly qualify min() and max() in header

2017-02-06 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3909 (follow-up): Properly qualify min() and max() in header .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5916/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-3524: Don't process spilled partitions with 0 probe rows

2017-02-06 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3524: Don't process spilled partitions with 0 probe rows .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5389 To

[Impala-ASF-CR] IMPALA-2020: Make it easy to work with big numbers

2017-02-06 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-2020: Make it easy to work with big numbers .. Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/5902/2//COMMIT_MSG Commit Message: PS2, Line

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala security topics to be generic

2017-02-07 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala security topics to be generic .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5931/1//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-4887: Skip parquet stats writer test for unsupported filesystems

2017-02-05 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4887: Skip parquet stats writer test for unsupported filesystems .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5910 To

[Impala-ASF-CR] IMPALA-4887: Skip parquet stats writer test for unsupported filesystems

2017-02-05 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4887: Skip parquet stats writer test for unsupported filesystems .. Patch Set 1: Would you like me to run the Jenkins merge job? -- To view, visit

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 8: David, do you want me to rebase and commit this? -- To view, visit

[Impala-ASF-CR] Patch references to Cloudera and CDH in Impala tutorial

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Patch references to Cloudera and CDH in Impala tutorial .. Patch Set 2: Laurel, if this addresses your -1, I am ready to +2. -- To view, visit

[Impala-ASF-CR](asf-site) Remove "Git repository" from navigation bar.

2017-01-31 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Remove "Git repository" from navigation bar. .. Patch Set 2: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5785 To unsubscribe, visit

[Impala-ASF-CR](asf-site) Remove "Git repository" from navigation bar.

2017-01-31 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: Remove "Git repository" from navigation bar. .. Remove "Git repository" from navigation bar. It is not in the navigation bar of any other page; it remained

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-01 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5640 To unsubscribe,

[Impala-ASF-CR] IMPALA-1861: Simplify conditionals with constant conditions

2017-01-23 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-1861: Simplify conditionals with constant conditions .. Patch Set 12: Code-Review+2 carry -- To view, visit http://gerrit.cloudera.org:8080/5585 To

[Impala-ASF-CR] Add a build flag for the undefined behavior sanitizer, aka "ubsan".

2017-01-26 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add a build flag for the undefined behavior sanitizer, aka "ubsan". .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-4750: Rename test infra classes so they don't mimic test classes.

2017-01-26 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4750: Rename test infra classes so they don't mimic test classes. .. Patch Set 3: > > (1 comment) > > > > I did my own survey for this renaming and feel

[Impala-ASF-CR] IMPALA-4055: Speed up to date() with custom implementation.

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4055: Speed up to_date() with custom implementation. .. Patch Set 4: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-4750: Rename test infra classes so they don't mimic test classes.

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4750: Rename test infra classes so they don't mimic test classes. .. Patch Set 2: > (1 comment) > > I did my own survey for this renaming and feel you

[Impala-ASF-CR] IMPALA-4750: Rename test infra classes so they don't mimic test classes.

2017-01-26 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4750: Rename test infra classes so they don't mimic test classes. .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5794 To

[Impala-ASF-CR] Release note updates for Impala 2.8

2017-01-30 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Release note updates for Impala 2.8 .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5668/6/docs/topics/impala_incompatible_changes.xml File

[Impala-ASF-CR] IMPALA-4818: Ensure the same number of tests are run every time

2017-01-30 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4818: Ensure the same number of tests are run every time .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-4818: Ensure the same number of tests are run every time

2017-01-28 Thread Jim Apple (Code Review)
Hello David Knupp, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5784 to look at the new patch set (#2). Change subject: IMPALA-4818: Ensure the same number of tests are run every time

[Impala-ASF-CR] IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1"

2017-01-28 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1" .. Patch Set 1: hewenting, are you interested in continuing to work on

[Impala-ASF-CR] IMPALA-4818: Ensure the same number of tests are run every time

2017-01-28 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4818: Ensure the same number of tests are run every time .. Patch Set 2: rebase only, carry David's +1 -- To view, visit

[Impala-ASF-CR] IMPALA-889: Add support for an ISO-SQL compliant trim() function.

2017-01-28 Thread Jim Apple (Code Review)
Jim Apple has abandoned this change. Change subject: IMPALA-889: Add support for an ISO-SQL compliant trim() function. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4474 To unsubscribe, visit

[Impala-ASF-CR] Release note updates for Impala 2.8

2017-01-28 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Release note updates for Impala 2.8 .. Patch Set 6: (4 comments) http://gerrit.cloudera.org:8080/#/c/5668/3/docs/topics/impala_incompatible_changes.xml File

[Impala-ASF-CR] IMPALA-4055: Speed up to date() with custom implementation.

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4055: Speed up to_date() with custom implementation. .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/5791/3//COMMIT_MSG Commit Message:

[Impala-ASF-CR](asf-site) Initial commit of the blog section of the Impala ASF website.

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Initial commit of the blog section of the Impala ASF website. .. Patch Set 8: (3 comments) http://gerrit.cloudera.org:8080/#/c/5667/8//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-4055: Speed up to date() with custom implementation.

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4055: Speed up to_date() with custom implementation. .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-4818: Ensure the same number of tests are run every time

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4818: Ensure the same number of tests are run every time .. Patch Set 1: > Dry-run: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/208/ Failed due

[Impala-ASF-CR] IMPALA-3202: implement spill-to-disk in new buffer pool

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3202: implement spill-to-disk in new buffer pool .. Patch Set 16: (1 comment) clang-tidy found this, but the Jenkins jobs that runs clang-tidy has confusing

[Impala-ASF-CR] IMPALA-3202: implement spill-to-disk in new buffer pool

2017-01-26 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3202: implement spill-to-disk in new buffer pool .. Patch Set 17: (1 comment)

[Impala-ASF-CR](asf-site) Download and signature links for 2.8.0 release

2017-01-20 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Download and signature links for 2.8.0 release .. Patch Set 1: This can be submitted in 24 hours, since the IPMC vote passed and the release artifacts are in SVN

[Impala-ASF-CR] Update copyright year to 2017

2017-01-20 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/5754 Change subject: Update copyright year to 2017 .. Update copyright year to 2017 Change-Id: I6a815c67176fc2a08ed693b581fac4c6919f297c --- M

[Impala-ASF-CR] Update copyright year to 2017

2017-01-20 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Update copyright year to 2017 .. Patch Set 1: This was found by one of our IPMC release voters. -- To view, visit http://gerrit.cloudera.org:8080/5754 To

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-01-20 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 3: (10 comments) Thank you for making this patch

[Impala-ASF-CR] Prepare for official 2.8 release by icnrementing version number

2017-01-20 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/5763 Change subject: Prepare for official 2.8 release by icnrementing version number .. Prepare for official 2.8 release by icnrementing version

[Impala-ASF-CR] Prepare for official 2.8 release by incrementing version number

2017-01-20 Thread Jim Apple (Code Review)
Hello Marcel Kornacker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5763 to look at the new patch set (#2). Change subject: Prepare for official 2.8 release by incrementing version number

[Impala-ASF-CR] Prepare for official 2.8 release by incrementing version number

2017-01-20 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Prepare for official 2.8 release by incrementing version number .. Patch Set 2: Code-Review+2 (1 comment) carry +2

[Impala-ASF-CR](asf-site) Initial commit of the blog section of the Impala ASF website.

2017-01-26 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Initial commit of the blog section of the Impala ASF website. .. Patch Set 9: A reminder to us both: the first blog post should change the Blog header link to

[Impala-ASF-CR](asf-site) Initial commit of the blog section of the Impala ASF website.

2017-01-26 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Initial commit of the blog section of the Impala ASF website. .. Patch Set 9: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5667 To

[Impala-ASF-CR] MPALA-4750: Rename test infra classes so they don't mimic test classes.

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: MPALA-4750: Rename test infra classes so they don't mimic test classes. .. Patch Set 1: (2 comments) You checked that the same tests are selected to be run both

[Impala-ASF-CR] MPALA-4750: Rename test infra classes so they don't mimic test classes.

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: MPALA-4750: Rename test infra classes so they don't mimic test classes. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5794/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-01-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/5640/3//COMMIT_MSG Commit Message:

[Impala-ASF-CR](asf-site) Initial commit of the blog section of the Impala ASF website.

2017-01-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Initial commit of the blog section of the Impala ASF website. .. Patch Set 3: > There's probably no reason not to. I went ahead and pushed a > version with

[Impala-ASF-CR] IMPALA-4818: Ensure the same number of tests are run every time

2017-01-24 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/5784 Change subject: IMPALA-4818: Ensure the same number of tests are run every time .. IMPALA-4818: Ensure the same number of tests are run

[Impala-ASF-CR](asf-site) Initial commit of the blog section of the Impala ASF website.

2017-01-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Initial commit of the blog section of the Impala ASF website. .. Patch Set 4: (5 comments)

[Impala-ASF-CR](asf-site) Initial commit of the blog section of the Impala ASF website.

2017-01-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Initial commit of the blog section of the Impala ASF website. .. Patch Set 3: > > There's probably no reason not to. I went ahead and pushed a > > version with

[Impala-ASF-CR](asf-site) Remove "Git repository" from navigation bar.

2017-01-24 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/5785 Change subject: Remove "Git repository" from navigation bar. .. Remove "Git repository" from navigation bar. It is not in the navigation

[Impala-ASF-CR] IMPALA-4750: Rename test infra classes so they don't mimic test classes.

2017-01-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4750: Rename test infra classes so they don't mimic test classes. .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-4818: Ensure the same number of tests are run every time

2017-01-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4818: Ensure the same number of tests are run every time .. Patch Set 1: Dry-run: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/208/ -- To view,

[Impala-ASF-CR] Update copyright year to 2017

2017-01-28 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: Update copyright year to 2017 .. Update copyright year to 2017 Change-Id: I6a815c67176fc2a08ed693b581fac4c6919f297c Reviewed-on:

[Impala-ASF-CR] Update copyright year to 2017

2017-01-28 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Update copyright year to 2017 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5754 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4936 and IMPALA-4915: Fix decimal overflow test

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4936 and IMPALA-4915: Fix decimal overflow test .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6068/2/be/src/runtime/decimal-value.inline.h

[Impala-ASF-CR] IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests .. Patch Set 6: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6002 To

[Impala-ASF-CR] IMPALA-3401 [DOCS] Phase 1 of removing 'Cloudera Manager'

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3401 [DOCS] Phase 1 of removing 'Cloudera Manager' .. Patch Set 2: Code-Review+2 Thank you! -- To view, visit http://gerrit.cloudera.org:8080/6049 To

[Impala-ASF-CR] IMPALA-4904,IMPALA-4914: whitelist workload "targeted-stress" for exhaustive tests

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4904,IMPALA-4914: whitelist workload "targeted-stress" for exhaustive tests .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-4926: Upgrade LZ4 to 1.7.5

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4926: Upgrade LZ4 to 1.7.5 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6030 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4904,IMPALA-4914: whitelist workload "targeted-stress" for exhaustive tests

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4904,IMPALA-4914: whitelist workload "targeted-stress" for exhaustive tests .. Patch Set 2: (9 comments)

[Impala-ASF-CR] IMPALA-3406: [DOCS] Empty the original Cloudera FAQ

2017-02-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3406: [DOCS] Empty the original Cloudera FAQ .. Patch Set 1: Code-Review-1 > (2 comments) > > I made these changes but now when I push to gerrit I get: >

[Impala-ASF-CR](asf-site) publishing Impala 2.8 rendered docs

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: publishing Impala 2.8 rendered docs .. Patch Set 1: Code-Review-2 Verified-1 Practicing with Laurel -- To view, visit http://gerrit.cloudera.org:8080/6062 To

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6025/5/be/src/exprs/aggregate-functions-ir.cc File

[Impala-ASF-CR] IMPALA-4936 and IMPALA-4915: Fix decimal overflow test

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4936 and IMPALA-4915: Fix decimal overflow test .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6068/1//COMMIT_MSG Commit Message: Line 22:

[Impala-ASF-CR] IMPALA-2020: Inline big number strings

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-2020: Inline big number strings .. Patch Set 5: (8 comments) http://gerrit.cloudera.org:8080/#/c/5902/5//COMMIT_MSG Commit Message: PS5, Line 27: Query:

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6025/5/be/src/exprs/aggregate-functions-ir.cc File

[Impala-ASF-CR] IMPALA-3401 [DOCS] Phase 5 of "Cloudera Manager" removal. Most of these fixes entailed hiding the paragraph where there are mentions of CM and writing a replacement paragraph for the u

2017-02-18 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3401 [DOCS] Phase 5 of "Cloudera Manager" removal. Most of these fixes entailed hiding the paragraph where there are mentions of CM and writing a replacement paragraph for the upstream docs. This removes the CM references

[Impala-ASF-CR] IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1"

2017-02-22 Thread Jim Apple (Code Review)
Jim Apple has abandoned this change. Change subject: IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1" .. Abandoned No contact from author in over 2 months -- To view, visit

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-02-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5829/1//COMMIT_MSG Commit Message: PS1, Line 7: Add .pep8rc

[Impala-ASF-CR] Add a build flag for the undefined behavior sanitizer, aka "ubsan".

2017-02-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add a build flag for the undefined behavior sanitizer, aka "ubsan". .. Patch Set 5: If it would be easier, I can break this up into N commits, with the first one

[Impala-ASF-CR] [DOCS] Remove reference to CDH 4 from DECIMAL page

2017-02-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Remove reference to CDH 4 from DECIMAL page .. Patch Set 1: > ??? Can't see +1/+2 buttons when I attempt to post. Happens at > multiple screen resolutions,

[Impala-ASF-CR] [DOCS] Remove reference to CDH 4 from DECIMAL page

2017-02-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Remove reference to CDH 4 from DECIMAL page .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6146/2/docs/topics/impala_decimal.xml File

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-02-19 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6025/5/be/src/exprs/aggregate-functions-ir.cc File

[Impala-ASF-CR] IMPALA-3401 [DOCS] Phase 1 of removing 'Cloudera Manager' Rewrote sections to eliminate 'Cloudera Manager' from topics. Look for subsequent phases to remove remaining instances of CM.

2017-02-17 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3401 [DOCS] Phase 1 of removing 'Cloudera Manager' Rewrote sections to eliminate 'Cloudera Manager' from topics. Look for subsequent phases to remove remaining instances of CM.

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/6025/5/be/src/exprs/aggregate-functions-ir.cc

[Impala-ASF-CR] IMPALA-3406: [DOCS] Empty the original Cloudera FAQ

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3406: [DOCS] Empty the original Cloudera FAQ .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6003 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3401 [DOCS] Phase 5 of "Cloudera Manager" removal. Most of these fixes entailed hiding the paragraph where there are mentions of CM and writing a replacement paragraph for the u

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3401 [DOCS] Phase 5 of "Cloudera Manager" removal. Most of these fixes entailed hiding the paragraph where there are mentions of CM and writing a replacement paragraph for the upstream docs. This removes the CM references

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Release note updates for Impala 2.8 .. Patch Set 9: 2.8.0 shipped a month ago. The delay on this patch is going to start preventing developers from updating

[Impala-ASF-CR] IMPALA-4546: Fix Moscow timezone conversion after 2014

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4546: Fix Moscow timezone conversion after 2014 .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5969/5/be/src/exprs/timestamp-functions.cc

[Impala-ASF-CR] IMPALA-4546: Fix Moscow timezone conversion after 2014

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4546: Fix Moscow timezone conversion after 2014 .. Patch Set 6: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests .. IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests This patch allows any

[Impala-ASF-CR] IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4904,IMPALA-4914: add targeted-stress to exhaustive tests .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6002 To

[Impala-ASF-CR] IMPALA-4546: Fix Moscow timezone conversion after 2014

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4546: Fix Moscow timezone conversion after 2014 .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5969/5/be/src/exprs/timestamp-functions.cc

[Impala-ASF-CR] IMPALA-3411 [DOCS] Rework Impala governance topics to be generic.

2017-02-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3411 [DOCS] Rework Impala governance topics to be generic. .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5957 To

[Impala-ASF-CR] Patch references to Cloudera and CDH in Impala tutorial

2017-02-22 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Patch references to Cloudera and CDH in Impala tutorial .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5663 To unsubscribe, visit

  1   2   3   4   5   6   7   8   9   10   >