Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9711 )
Change subject: IMPALA-6695: Fix PyPi regex, update setuptools version
..
Patch Set 1: Code-Review+2
I think this is LGTM for now.
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/7241 )
Change subject: IMPALA-2782: Allow impala-shell to connect directly to impalad
when configured with load balancer and kerberos.
..
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/7241 )
Change subject: IMPALA-2782: Allow impala-shell to connect directly to impalad
when configured with load balancer and kerberos.
..
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9600 )
Change subject: Consistently use Java 1.7 compiler.
..
Patch Set 2:
Pretty superficial inspection suggested this was stuck at the
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9600 )
Change subject: Consistently use Java 1.7 compiler.
..
Patch Set 2: Code-Review+2
Carry after fixing spelling nit in commit message.
Hello Michael Brown, Alex Behm, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9600
to look at the new patch set (#2).
Change subject: Consistently use Java 1.7 compiler.
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9588 )
Change subject: Removing (broken) retries from split-hbase.sh.
..
Patch Set 2: Code-Review+2
Carry +2 after rebase.
--
To view,
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9614 )
Change subject: DRAFT IMPALA-5973: Provide query plan in JSON format.
..
Patch Set 1:
> into JSON format, so it can be displayed in
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9614 )
Change subject: DRAFT IMPALA-5973: Provide query plan in JSON format.
..
Patch Set 1:
I’m excited to see this being worked on!
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9600 )
Change subject: Consistently use Java 1.7 compiler.
..
Patch Set 1:
https://jenkins.impala.io/job/gerrit-verify-dryrun-external/93/
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9600
Change subject: Consistently use Java 1.7 compiler.
..
Consistently use Java 1.7 compiler.
We use Java 1.7 in fe/pom.xml, where most
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9588 )
Change subject: Removing (broken) retries from split-hbase.sh.
..
Patch Set 1:
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9588
Change subject: Removing (broken) retries from split-hbase.sh.
..
Removing (broken) retries from split-hbase.sh.
The retries in
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9395 )
Change subject: IMPALA-6341, IMPALA-5917: Reduce mem-limit for
start-impala-cluster.
..
Patch Set 2:
(1 comment)
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9395 )
Change subject: IMPALA-6341, IMPALA-5917: Reduce mem-limit for
start-impala-cluster.
..
Patch Set 2:
Tim,
Please take a look at
Hello Tim Armstrong, Alex Behm,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9395
to look at the new patch set (#2).
Change subject: IMPALA-6341, IMPALA-5917: Reduce mem-limit for
start-impala-cluster.
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9531 )
Change subject: IMPALA-6405: Ignore commit for automatic cherry picking
..
Patch Set 1: Verified+1 Code-Review+2
I'm +1'ing the
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9508 )
Change subject: IMPALA-6606: date_trunc() misinterprets MILLENNIUM/CENTURY
precision
..
Patch Set 1:
(1 comment)
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/8434 )
Change subject: IMPALA-5564: Release lock during planning. (wip)
..
Patch Set 1:
Took me a long time to get back to this, but I'm
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9431 )
Change subject: IMPALA-6570: Support KUDU on SLES12.
..
Patch Set 2:
(1 comment)
Hello Thomas Tauber-Marshall, David Knupp,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9431
to look at the new patch set (#3).
Change subject: IMPALA-6570: Support KUDU on SLES12.
..
Hello Thomas Tauber-Marshall,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9431
to look at the new patch set (#2).
Change subject: IMPALA-6570: Support KUDU on SLES12.
..
IMPALA-6570:
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9431
Change subject: IMPALA-6570: Support KUDU on SLES12.
..
IMPALA-6570: Support KUDU on SLES12.
Different versions (service packs) of
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9395 )
Change subject: IMPALA-6341, IMPALA-5917: Reduce mem-limit for
start-impala-cluster.
..
Patch Set 1:
I see this happening in
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9395
Change subject: IMPALA-6341, IMPALA-5917: Reduce mem-limit for
start-impala-cluster.
..
IMPALA-6341, IMPALA-5917: Reduce mem-limit
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9310 )
Change subject: IMPALA-6517: bootstrap_toolchain.py fails to recognize
lsb_release output from RHEL OS
..
Patch Set 4: Code-Review+2
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/8958 )
Change subject: IMPALA-5152: Introduce metadata loading phase
..
Patch Set 5: Code-Review+1
Thanks for addressing my comments!
--
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9227 )
Change subject: IMPALA-6482: add QUERY_TIME_LIMIT_S option
..
Patch Set 5:
(2 comments)
I just looked at how these two were
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9349 )
Change subject: Bump Kudu Java version to 1.7.0
..
Patch Set 1: Code-Review+2
--
To view, visit
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9286 )
Change subject: IMPALA-6509: [docs] Note for haproxy for Kerberized clusters
..
Patch Set 5:
> Uploaded patch set 5: Patch Set 4 was
Philip Zeyliger has abandoned this change. (
http://gerrit.cloudera.org:8080/9192 )
Change subject: IMPALA-6204: Remove external DataSource
..
Abandoned
Per conversation on user@, folks are interested in keeping this around.
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9264 )
Change subject: Remove Yarn from minicluster.
..
Patch Set 1:
> Patch Set 1:
>
> > > Patch Set 1:
> > >
> > > Did you also test
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9264 )
Change subject: Remove Yarn from minicluster.
..
Patch Set 1:
> Patch Set 1:
>
> Did you also test data load and snapshot creation?
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9268 )
Change subject: IMPALA-6495: fix targeted-perf for new column alias syntax
..
Patch Set 1: Code-Review+2
Change looks fine. Should
Philip Zeyliger has uploaded a new patch set (#3). (
http://gerrit.cloudera.org:8080/9085 )
Change subject: IMPALA-6070: Expose using Docker to run tests faster.
..
IMPALA-6070: Expose using Docker to run tests faster.
Allows
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9264 )
Change subject: Remove Yarn from minicluster.
..
Patch Set 1:
https://jenkins.impala.io/job/gerrit-verify-dryrun-external/78/ is
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9264
Change subject: Remove Yarn from minicluster.
..
Remove Yarn from minicluster.
Removes vestiges of Yarn from the minicluster.
Turns
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/8894 )
Change subject: IMPALA-6372: Go parallel for Hive dataload
..
Patch Set 7:
I ran tests with this change and observed data loading go
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9168 )
Change subject: IMPALA-5690: Upgrade thrift to 0.9.3-p3
..
Patch Set 4:
Thomas: could you give this a review?
Sailesh: please
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9168 )
Change subject: IMPALA-5690: Upgrade thrift to 0.9.3-p3
..
Patch Set 4:
(1 comment)
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9251 )
Change subject: IMPALA-4475: part 1, reduce size of TExecQueryFInstancesParams
..
Patch Set 1:
The hammer here is to compress all
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9173 )
Change subject: IMPALA-3271: organise and warn on removed startup flags
..
Patch Set 3: Code-Review+2
LGTM.
--
To view, visit
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9192 )
Change subject: IMPALA-6204: Remove external DataSource
..
Patch Set 7:
Rebased. Running
Hello Jim Apple, Dimitris Tsirogiannis, Alex Behm, Zach Amsden, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9192
to look at the new patch set (#7).
Change subject: IMPALA-6204: Remove external DataSource
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9192 )
Change subject: IMPALA-6204: Remove external DataSource
..
Patch Set 5:
(2 comments)
Addressed the comments. Removed the remaining
Hello Jim Apple, Dimitris Tsirogiannis, Alex Behm, Zach Amsden, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9192
to look at the new patch set (#6).
Change subject: IMPALA-6204: Remove external DataSource
Hello Dimitris Tsirogiannis, Alex Behm, Zach Amsden, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9192
to look at the new patch set (#5).
Change subject: IMPALA-6204: Remove external DataSource
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9192 )
Change subject: IMPALA-6204: Remove external DataSource
..
Patch Set 4:
(1 comment)
I've been generally hesitant of removing
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9192 )
Change subject: IMPALA-6204: Remove external DataSource
..
Patch Set 4:
Hello Dimitris Tsirogiannis,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9192
to look at the new patch set (#4).
Change subject: IMPALA-6204: Remove external DataSource
..
IMPALA-6204:
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9192 )
Change subject: IMPALA-6204: Remove external DataSource
..
Patch Set 3:
I think this is now reviewable. I ran core tests
Hello Dimitris Tsirogiannis,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9192
to look at the new patch set (#3).
Change subject: IMPALA-6204: Remove external DataSource
..
IMPALA-6204:
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9198 )
Change subject: IMPALA-3916: Reserve SQL:2016 reserved words
..
Patch Set 1: Code-Review+2
The conflict was boring, yes?
I'd prefer
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9192 )
Change subject: IMPALA-6204: Remove external DataSource
..
Patch Set 2: Code-Review-1
Sorry, hold off on this please. I rebased and
Hello Dimitris Tsirogiannis,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9192
to look at the new patch set (#2).
Change subject: IMPALA-6204: Remove external DataSource
..
IMPALA-6204:
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9192
Change subject: IMPALA-6204: Remove external DataSource
..
IMPALA-6204: Remove external DataSource
Removes DataSourceScanNode,
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9096 )
Change subject: IMPALA-3916: Reserve SQL:2016 reserved words
..
Patch Set 10:
> Patch Set 10:
>
> > Patch Set 10: Code-Review+2
>
>
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9096 )
Change subject: IMPALA-3916: Reserve SQL:2016 reserved words
..
Patch Set 10: Code-Review+2
--
To view, visit
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9096 )
Change subject: IMPALA-3916: Reserve SQL:2016 reserved words
..
Patch Set 7:
(1 comment)
> Patch Set 7:
>
> (1 comment)
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9168 )
Change subject: IMPALA-5690: Upgrade thrift to 0.9.3-p3
..
Patch Set 1:
(5 comments)
I'm not really qualified to review this
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9165 )
Change subject: IMPALA-6455: unique tmpdirs for
test_partition_metadata_compatibility
..
Patch Set 2: Code-Review+2
Looks good to
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9165 )
Change subject: IMPALA-6455: unique tmpdirs for
test_partition_metadata_compatibility
..
Patch Set 1:
(2 comments)
Thanks; this
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9145 )
Change subject: Bump Kudu version to c6beta-impala-toolchain-tag1
..
Patch Set 1: Code-Review+2
--
To view, visit
Hello Taras Bobrovytsky, Lars Volker, Jim Apple,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9135
to look at the new patch set (#3).
Change subject: IMPALA-6410: compare_branches: use looser expression
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9135 )
Change subject: IMPALA-6410: compare_branches: use looser expression
..
Patch Set 2:
(1 comment)
Hello Taras Bobrovytsky, Lars Volker, Jim Apple,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9135
to look at the new patch set (#2).
Change subject: IMPALA-6410: compare_branches: use looser expression
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9135
Change subject: IMPALA-6410: compare_branches: use looser expression
..
IMPALA-6410: compare_branches: use looser expression
We've
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9109 )
Change subject: Patch thrift-0.9.3 for python 2.6.
..
Patch Set 1: Code-Review+2
Looks fine to me.
--
To view, visit
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9124 )
Change subject: IMPALA-6440: Backwards compatibility for HBase metadata
..
Patch Set 2:
(1 comment)
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9124 )
Change subject: IMPALA-6440: Backwards compatibility for HBase metadata
..
Patch Set 1:
(4 comments)
Minor feedback. (Code logic
Hello Gabor Kaszab, Tim Armstrong, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9100
to look at the new patch set (#2).
Change subject: IMPALA-6318: Revert "Adjustment for hanging query cancellation
test"
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9102 )
Change subject: IMPALA-6435: Disable codegen for CHAR literals.
..
Patch Set 3: Code-Review+1
Looks better to me. Not an area of my
Hello Lars Volker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9101
to look at the new patch set (#3).
Change subject: Expose refs/heads/2.x and refs/sandbox//*.
..
Expose
Philip Zeyliger has abandoned this change. (
http://gerrit.cloudera.org:8080/9104 )
Change subject: Modify access rules
..
Abandoned
--
To view, visit http://gerrit.cloudera.org:8080/9104
To unsubscribe, visit
Hello Lars Volker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9101
to look at the new patch set (#2).
Change subject: Expose refs/heads/2.x and refs/sandbox//*.
..
Expose
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9104
Change subject: Modify access rules
..
Modify access rules
Change-Id: I0ad3cec33990043480decb3b1bb37893d5f64b6e
---
M project.config
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9102 )
Change subject: IMPALA-6435: Disable codegen for CHAR literals.
..
Patch Set 1:
(1 comment)
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9100 )
Change subject: IMPALA-6318: Revert "Adjustment for hanging query cancellation
test"
..
Patch Set 1:
GVO failed, but we know this
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9101
Change subject: Expose refs/heads/2.x and refs/scratch/2.x-staging to
impala-public-jenkins.
..
Expose refs/heads/2.x and
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9096 )
Change subject: IMPALA-3916: Reserve SQL:2016 reserved words
..
Patch Set 1:
(3 comments)
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/8900 )
Change subject: IMPALA-3282: Adds regexp_escape built-in function
..
Patch Set 7: Code-Review+1
--
To view, visit
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9100 )
Change subject: IMPALA-6318: Revert "Adjustment for hanging query cancellation
test"
..
Patch Set 1: Code-Review+2
I'm testing GVO
Hello Tim Armstrong, Impala Public Jenkins,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/9100
to review the following change.
Change subject: IMPALA-6318: Revert "Adjustment for hanging query cancellation
test"
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9082 )
Change subject: IMPALA-6394: Enable HDFS debug logging
..
Patch Set 1:
I don't really have a sense of how frequent this is. I would
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9085
Change subject: IMPALA-6070: Expose using Docker to run tests faster.
..
IMPALA-6070: Expose using Docker to run tests faster.
Allows
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9044 )
Change subject: Bumping version to 3.0.
..
Patch Set 3:
Are folks comfortable +2'ing this?
Once this goes in, I'll start
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/8894 )
Change subject: IMPALA-6372: Go parallel for Hive dataload
..
Patch Set 7:
(16 comments)
I did one pass through. This looks like a
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9074 )
Change subject: IMPALA-6420: Fix TestCharFormats for local filesystem tests
..
Patch Set 2:
Do you think we have this elsewhere?
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9060 )
Change subject: IMPALA-6418: Find a reliable way to detect supported TLS
versions
..
Patch Set 1:
> ... returns an internal
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9062 )
Change subject: IMPALA-4924: Enable Decimal V2 by default
..
Patch Set 1:
(1 comment)
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9060 )
Change subject: IMPALA-6418: Find a reliable way to detect supported TLS
versions
..
Patch Set 1:
Is there a convenient place where
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9045 )
Change subject: IMPALA-6410: Tool to cherrypick changes across branches.
..
Patch Set 2:
(25 comments)
Thanks for the detailed
Hello Taras Bobrovytsky,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9044
to look at the new patch set (#3).
Change subject: Bumping version to 3.0.
..
Bumping version to 3.0.
This
Hello Taras Bobrovytsky, Jim Apple,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9045
to look at the new patch set (#4).
Change subject: IMPALA-6410: Tool to cherrypick changes across branches.
Hello Taras Bobrovytsky,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9045
to look at the new patch set (#3).
Change subject: IMPALA-6410: Tool to cherrypick changes across branches.
..
Philip Zeyliger has uploaded a new patch set (#2). (
http://gerrit.cloudera.org:8080/9044 )
Change subject: Bumping version to 3.0.
..
Bumping version to 3.0.
This changes the version that Impala presents as to 3.0. We are
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9044
Change subject: Bumping version to 3.0.
..
Bumping version to 3.0.
This changes the version that Impala presents as to 3.0. We are
Philip Zeyliger has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9045
Change subject: IMPALA-6410: Tool to cherrypick changes across branches.
..
IMPALA-6410: Tool to cherrypick changes across branches.
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/8900 )
Change subject: IMPALA-3282: Adds regexp_escape built-in function
..
Patch Set 5:
(2 comments)
Thanks for the iteration!
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9035 )
Change subject: Bump thrift to 0.9.3-p2; Add bison 3.0.4
..
Patch Set 2: Code-Review+2
Looks fine to me. Thanks!
--
To view, visit
701 - 800 of 855 matches
Mail list logo