[Impala-ASF-CR] Add FNV, Zobrist, and SIMD hash functions to the int hash benchmark.

2016-09-11 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add FNV, Zobrist, and SIMD hash functions to the int hash benchmark. .. Patch Set 3: Just commenting on an old patch to see if our new gerrit -> ASF mailing list s

[Impala-ASF-CR] IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs.

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#2). Change subject: IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs. .. IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs. While I'm here, fix as many of the warnings as possibl

[Impala-ASF-CR] IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs.

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs. .. Patch Set 1: (3 comments) > (5 comments) > > looks pretty reasonable. Might want to do a surface level

[Impala-ASF-CR] IMPALA-3918: Fix straggler Cloudera -> ASF license headers

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/4386 Change subject: IMPALA-3918: Fix straggler Cloudera -> ASF license headers .. IMPALA-3918: Fix straggler Cloudera -> ASF license headers Cha

[Impala-ASF-CR] IMPALA-3918: Fix straggler Cloudera -> ASF license headers

2016-09-12 Thread Jim Apple (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4386 to look at the new patch set (#2). Change subject: IMPALA-3918: Fix straggler Cloudera -> ASF license headers .. IMPAL

[Impala-ASF-CR] IMPALA-3918: Fix straggler Cloudera -> ASF license headers

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3918: Fix straggler Cloudera -> ASF license headers .. Patch Set 2: Code-Review+2 > lgtm, though it might be a good idea to document the reasoning that > the

[Impala-ASF-CR] IMPALA-4096: Allow clean.sh to work from snapshots

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4096: Allow clean.sh to work from snapshots .. IMPALA-4096: Allow clean.sh to work from snapshots buildall.sh calls bin/clean.sh, which fails when no

[Impala-ASF-CR] IMPALA-4096: Allow clean.sh to work from snapshots

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4096: Allow clean.sh to work from snapshots .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4336 To unsubscribe, visit http:/

[Impala-ASF-CR](asf-site) IMPALA-4068: Focus pages on Apache Impala, not Cloudera Impala.

2016-09-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4068: Focus pages on Apache Impala, not Cloudera Impala. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4318/2/community.html File community

[Impala-ASF-CR](asf-site) IMPALA-4068: Focus pages on Apache Impala, not Cloudera Impala.

2016-09-13 Thread Jim Apple (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4318 to look at the new patch set (#3). Change subject: IMPALA-4068: Focus pages on Apache Impala, not Cloudera Impala. ...

[Impala-ASF-CR](asf-site) IMPALA-4068: Focus pages on Apache Impala, not Cloudera Impala.

2016-09-13 Thread Jim Apple (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4318 to look at the new patch set (#4). Change subject: IMPALA-4068: Focus pages on Apache Impala, not Cloudera Impala. ...

[Impala-ASF-CR](asf-site) IMPALA-4068: Focus pages on Apache Impala, not Cloudera Impala.

2016-09-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4068: Focus pages on Apache Impala, not Cloudera Impala. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4318/2/community.html File community

[Impala-ASF-CR] IMPALA-4110: Apache RAT script on Impala tarballs.

2016-09-13 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/4405 Change subject: IMPALA-4110: Apache RAT script on Impala tarballs. .. IMPALA-4110: Apache RAT script on Impala tarballs. Apache RAT is a too

[Impala-ASF-CR] IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs.

2016-09-13 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#4). Change subject: IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs. .. IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs. While I'm here, fix as many of the warnings as possibl

[Impala-ASF-CR] IMPALA-4110: Clean up issues found by Apache RAT.

2016-09-13 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#5). Change subject: IMPALA-4110: Clean up issues found by Apache RAT. .. IMPALA-4110: Clean up issues found by Apache RAT. The script and support files are in another patch: https://gerr

[Impala-ASF-CR] IMPALA-4110: Apache RAT script on Impala tarballs.

2016-09-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110: Apache RAT script on Impala tarballs. .. Patch Set 1: Separated from https://gerrit.cloudera.org/#/c/4361/ following a request by Alex. -- To view, v

[Impala-ASF-CR] IMPALA-4110: Clean up issues found by Apache RAT.

2016-09-13 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#6). Change subject: IMPALA-4110: Clean up issues found by Apache RAT. .. IMPALA-4110: Clean up issues found by Apache RAT. The script and support files are in another patch: https://gerr

[Impala-ASF-CR] IMPALA-4110: Clean up issues found by Apache RAT.

2016-09-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110: Clean up issues found by Apache RAT. .. Patch Set 3: (3 comments) > (3 comments) > > Looks good to me. > > Can we please separate the tool integr

[Impala-ASF-CR] IMPALA-4110: Clean up issues found by Apache RAT.

2016-09-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110: Clean up issues found by Apache RAT. .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/4361/6/be/.impala.doxy File be/.impala.doxy: Line

[Impala-ASF-CR] IMPALA-4110: Apache RAT script on Impala tarballs.

2016-09-14 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4110: Apache RAT script on Impala tarballs. .. IMPALA-4110: Apache RAT script on Impala tarballs. Apache RAT is a tool for license auditing. It will

[Impala-ASF-CR] IMPALA-4110: Apache RAT script on Impala tarballs.

2016-09-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110: Apache RAT script on Impala tarballs. .. Patch Set 1: Verified+1 No GVO implications, so self-verifying -- To view, visit http://gerrit.cloudera.org:8

[Impala-ASF-CR] IMPALA-4110: Clean up issues found by Apache RAT.

2016-09-14 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#7). Change subject: IMPALA-4110: Clean up issues found by Apache RAT. .. IMPALA-4110: Clean up issues found by Apache RAT. The script and support files are in another patch: https://gerr

[Impala-ASF-CR] IMPALA-4110: Clean up issues found by Apache RAT.

2016-09-14 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#8). Change subject: IMPALA-4110: Clean up issues found by Apache RAT. .. IMPALA-4110: Clean up issues found by Apache RAT. Change-Id: I5bfe77f9a871018e7a67553ed270e2df53006962 --- M LICEN

[Impala-ASF-CR] IMPALA-4116: Remove 'cdh' from version string

2016-09-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4116: Remove 'cdh' from version string .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4421/1/bin/save-version.sh File bin/save-version.sh:

[Impala-ASF-CR] IMPALA-4116: Remove 'cdh' from version string

2016-09-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4116: Remove 'cdh' from version string .. Patch Set 2: Code-Review+1 I'd like another +1 on this, just for another pair of eyes. Once this is done, I'll cher

[Impala-ASF-CR] IMPALA-4116: Remove 'cdh' from version string

2016-09-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4116: Remove 'cdh' from version string .. Patch Set 2: Code-Review+2 I've rethought it and I'm convinced this has a very low probability of causing issues.

[Impala-ASF-CR] Make gen build version.py resilient to a failing git rev-parse

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Make gen_build_version.py resilient to a failing git rev-parse .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/4411/8/bin/save-version.sh File bin/s

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/4438 Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. The c

[Impala-ASF-CR] IMPALA-4116: Remove 'cdh' from version string again

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4116: Remove 'cdh' from version string again .. Patch Set 1: Code-Review+2 Since this already passed GVO, I think it's OK to commit without doing that again.

[Impala-ASF-CR] IMPALA-4116: Remove 'cdh' from version string again

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4116: Remove 'cdh' from version string again .. IMPALA-4116: Remove 'cdh' from version string again The rebase of another change went wrong and undid

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1) .. Patch Set 3: Any news on this? -- To view, visit http://gerrit.cloudera.org:8080/3936 To unsubscribe, vis

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4438/1/www/bootstrap/js/npm.js File www/bo

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4438/1/www/bootstrap/js/npm.js File www/bo

[Impala-ASF-CR] IMPALA-1654: General partition exprs in DDL operations.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-1654: General partition exprs in DDL operations. .. Patch Set 13: > (1 comment) > > Thanks for helping me fix the auditing bugs. I wish this wiki may > he

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4438 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3308: Get expr-test passing on PPC64LE

2016-09-19 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3308: Get expr-test passing on PPC64LE .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4186/3/be/src/exprs/expr-value.h File be/src/exprs/exp

[Impala-ASF-CR] IMPALA-3308: Get expr-test passing on PPC64LE

2016-09-19 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3308: Get expr-test passing on PPC64LE .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4186/3/be/src/exprs/expr-value.h File be/src/exprs/exp

[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-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-3411 [DOCS] Rework Impala governance topics to be generic.

2017-02-10 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 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5957/1//COMMIT_MSG Commit Messag

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

2017-02-10 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/5962/2/docs/topics/impala_ker

[Impala-ASF-CR] IMPALA-3973: optional 2nd and 3rd arguments for instr().

2017-02-12 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3973: optional 2nd and 3rd arguments for instr(). .. Patch Set 2: > (1 comment) Any updates? -- To view, visit http://gerrit.cloudera.org:8080/5589 To unsu

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

2017-02-13 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 1: (7 comments) http://gerrit.cloudera.org:8080/#/c/5969/1/be/src/exprs/expr-test.cc File be/src

[Impala-ASF-CR] IMPALA-4263: Fix wrong ommission of agg/analytic hash exchanges.

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4263: Fix wrong ommission of agg/analytic hash exchanges. .. Patch Set 1: Started pre-merge tests in dry-run mode so they will be ready ASAP but won't submit

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

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic .. Patch Set 3: Code-Review+2 Thanks, Ambreen! -- To view, visit http://gerrit.cloudera.org:8

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

2017-02-13 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 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5957/2/docs/topics/impala_lineage

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

2017-02-13 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 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5957/3/docs/topics/impala_lineage

[Impala-ASF-CR] IMPALA-4920: custom cluster tests: remove unnecessary escaped quote chars

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4920: custom cluster tests: remove unnecessary escaped quote chars .. Patch Set 1: Code-Review+2 (1 comment) Thanks for finding and fixing! FYI: pre-review

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

2017-02-13 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 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5957 To unsubscri

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

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

[Impala-ASF-CR] IMPALA-4920: custom cluster tests: remove unnecessary escaped quote chars

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4920: custom cluster tests: remove unnecessary escaped quote chars .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5978/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-4920: custom cluster tests: remove unnecessary escaped quote chars

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4920: custom cluster tests: remove unnecessary escaped quote chars .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5978/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-4263: Fix wrong ommission of agg/analytic hash exchanges.

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4263: Fix wrong ommission of agg/analytic hash exchanges. .. Patch Set 2: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/265/ finished successfully.

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

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic .. Patch Set 4: -Code-Review > I did a build and got a lot of errors and it doesn't look like t

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

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic .. Patch Set 4: > @Jim--Did you check the output? I got the same list of > "recoverable" error

[Impala-ASF-CR] Qualify min() in header

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Qualify min() in header .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5991 To unsubscribe, visit http://gerrit.cloudera.org:8080/s

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

2017-02-13 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: > Jim, if you haven't done so already, I resolved the merge conflicts > here ag

[Impala-ASF-CR](asf-site) A blog post about IMPALA-4916

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/5995 Change subject: A blog post about IMPALA-4916 .. A blog post about IMPALA-4916 While I'm here, fix IMPALA-4928 and allow the install instruc

[Impala-ASF-CR](asf-site) A blog post about IMPALA-4916

2017-02-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: A blog post about IMPALA-4916 .. Patch Set 1: I'm not planning on committing this until I talk to a few other contributors and see if they would be interested in bl

[Impala-ASF-CR] IMPALA-4920: custom cluster tests: fix generation of py.test options

2017-02-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4920: custom cluster tests: fix generation of py.test options .. Patch Set 2: Code-Review+2 Would you like me to kick off the Jenkins merge job? -- To view

[Impala-ASF-CR](asf-site) A blog post about IMPALA-4916

2017-02-14 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#2). Change subject: A blog post about IMPALA-4916 .. A blog post about IMPALA-4916 The work I authored is in nikola_site_generator/; blog/ contains the content that Nikola generated from

[Impala-ASF-CR](asf-site) A blog post about IMPALA-4916

2017-02-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: A blog post about IMPALA-4916 .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/5995/1//COMMIT_MSG Commit Message: PS1, Line 7: A blog post about IM

[Impala-ASF-CR](asf-site) A blog post about IMPALA-4916

2017-02-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: A blog post about IMPALA-4916 .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5995/1/nikola_site_generator/posts/where-did-i-leave-my-keys.md File n

[Impala-ASF-CR](asf-site) A blog post about IMPALA-4916

2017-02-14 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#3). Change subject: A blog post about IMPALA-4916 .. A blog post about IMPALA-4916 The work I authored is in nikola_site_generator/; blog/ contains the content that Nikola generated from

[Impala-ASF-CR](asf-site) A blog post about IMPALA-4916

2017-02-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: A blog post about IMPALA-4916 .. Patch Set 3: PS3 was because I forgot in PS1 and PS2 to include the actual blog post HTML. -- To view, visit http://gerrit.clouder

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

2017-02-14 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: (2 comments) http://gerrit.cloudera.org:8080/#/c/6003/1//COMMIT_MSG Commit Message: Line 16: Ph

[Impala-ASF-CR](asf-site) A blog post about IMPALA-4916

2017-02-15 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: A blog post about IMPALA-4916 .. Patch Set 3: > (1 comment) > > So, I believe that all of the pages still point to the old blog > site. This patch doesn't seem t

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

2017-02-15 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 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5902/4/be/src/util/decimal-util.cc File be/src/ut

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

2017-02-16 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 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/5969/2/be/src/exprs/expr-test.cc File be/src

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

2017-02-16 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: Laurel, I am satisfied if you are. -- To view, visit http://gerrit.cloudera.org:8080/5663 T

[Impala-ASF-CR] IMPALA-3403: [DOCS] Pare back irrelevant installation info

2017-02-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3403: [DOCS] Pare back irrelevant installation info .. Patch Set 1: (2 comments) Thanks for doing this! http://gerrit.cloudera.org:8080/#/c/6005/1/docs/topi

[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: (1 comment) http://gerrit.cloudera.org:8080/#/c/6003/1/docs/topics/impala_faq.xml File docs/topi

[Impala-ASF-CR] IMPALA-3405: [DOCS] Pare down upgrading section

2017-02-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3405: [DOCS] Pare down upgrading section .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/6008/1/docs/topics/impala_upgrading.xml File docs/t

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

2017-02-16 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 3: (3 comments) Thanks for your patience with these little nits. http://gerrit.cloudera.org:808

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

2017-02-16 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 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/5969/2/be/src/exprs/expr-test.cc File be/src

[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] IMPALA-4546: Fix Moscow timezone conversion after 2014

2017-02-17 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 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5969/2/be/src/exprs/expr-test.cc File be/src/

[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-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 http://gerrit.cloudera.

[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) http://gerrit.cloudera.org:8080/#/c/6002/3//COMMIT_

[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) http://gerrit.cloudera.org:8080/#/c/6002/3//COMMIT_

[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 unsub

[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: sel

[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-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-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 from

[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-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 unsubscri

[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 Fi

[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) http://gerrit.cloudera.org:8080/#/c/6002/6/tests/stress/t

[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 unsubscribe

[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 from

[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-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 unsubscribe

[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 t

[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 Fi

[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) http://gerrit.cloudera.org:8080/#/c/5969/5/be/src/exprs/timestamp-f

[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 File

[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 http:

  1   2   3   4   5   6   7   8   9   10   >