You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudberry.apache.org by ma...@apache.org on 2025/02/25 06:15:03 UTC
(cloudberry) branch main updated: Fix gp_hyperloglog cstring type hadnling (#953)
This is an automated email from the ASF dual-hosted git repository.
maxyang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudberry.git
The following commit(s) were added to refs/heads/main by this push:
new 00da831842 Fix gp_hyperloglog cstring type hadnling (#953)
00da831842 is described below
commit 00da831842077cc170f12293a1aadc4c49ff00dc
Author: reshke <re...@double.cloud>
AuthorDate: Tue Feb 25 11:14:59 2025 +0500
Fix gp_hyperloglog cstring type hadnling (#953)
fixes issue https://github.com/apache/cloudberry/issues/952
---
src/backend/utils/hyperloglog/gp_hyperloglog.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/src/backend/utils/hyperloglog/gp_hyperloglog.c b/src/backend/utils/hyperloglog/gp_hyperloglog.c
index f9ce8e3324..5567eb0cd5 100644
--- a/src/backend/utils/hyperloglog/gp_hyperloglog.c
+++ b/src/backend/utils/hyperloglog/gp_hyperloglog.c
@@ -786,6 +786,11 @@ gp_hyperloglog_add_item(GpHLLCounter hllcounter, Datum element, int16 typlen, bo
/* varlena */
hyperloglog = gp_hll_add_element(hyperloglog, VARDATA_ANY(element), VARSIZE_ANY_EXHDR(element));
}
+ else if (typlen == -2)
+ {
+ /* cstring */
+ hyperloglog = gp_hll_add_element(hyperloglog, DatumGetCString(element), strlen(DatumGetCString(element)));
+ }
else if (typbyval)
{
/* fixed-length, passed by value */
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cloudberry.apache.org
For additional commands, e-mail: commits-help@cloudberry.apache.org