Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 12: Verified+1
--
To view, visit
Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 12:
Build started:
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 12: Code-Review+2
rebase
--
To view, visit
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 11:
Thanks for the quick and thorough review Jim! Much
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 9:
(1 comment)
Jim Apple has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 9:
(1 comment)
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 9:
(2 comments)
Hello Bharath Vissapragada, Jim Apple, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8569
to look at the new patch set (#9).
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
Hello Bharath Vissapragada, Jim Apple, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8569
to look at the new patch set (#7).
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
Hello Bharath Vissapragada, Jim Apple, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8569
to look at the new patch set (#8).
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 7: Code-Review+2
--
To view, visit
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 6: Code-Review+2
Rebase and trivial size fix. Tests passed.
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 5:
Before merging, I'll run the tests one more time and give
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 5: Code-Review+2
That's clearer, thanks.
--
To view, visit
Hello Bharath Vissapragada, Jim Apple, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8569
to look at the new patch set (#5).
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 3:
(1 comment)
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 3:
(14 comments)
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 3:
(14 comments)
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 3:
(1 comment)
Jim Apple has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 3:
(1 comment)
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 3:
(1 comment)
Jim Apple has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 3:
(1 comment)
Bharath Vissapragada has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 3: Code-Review+1
(3 comments)
LGTM.
Hello Bharath Vissapragada,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8569
to look at the new patch set (#2).
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Alex Behm has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 1:
(7 comments)
Bharath Vissapragada has posted comments on this change. (
http://gerrit.cloudera.org:8080/8569 )
Change subject: IMPALA-5310: Part 2: Add SAMPLED_NDV() function.
..
Patch Set 1:
(7 comments)
I have a bunch of nits. Although
27 matches
Mail list logo
