Mihaly Szjatinya has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24590 )

Change subject: IMPALA-9821: Change DataSketches functions to return BINARY
......................................................................


Patch Set 1:

> Patch Set 1:
>
> (1 comment)
>
> lgtm at high level, but I would remove the STRING overloads

Just to clarify. The ticket says about the DS generating, i.e. returning  
functions, and those ARE breaking. But you mean also the consumer functions 
should be breaking?


--
To view, visit http://gerrit.cloudera.org:8080/24590
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id4a6b54089dd356e37257bc24adeb1eb98e82c25
Gerrit-Change-Number: 24590
Gerrit-PatchSet: 1
Gerrit-Owner: Mihaly Szjatinya <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Mihaly Szjatinya <[email protected]>
Gerrit-Comment-Date: Thu, 16 Jul 2026 09:51:22 +0000
Gerrit-HasComments: No

Reply via email to