Csaba Ringhofer has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24534 )

Change subject: POC IMPALA-15161: Add GEOMETRY type for WKB_EXPERIMENTAL mode
......................................................................


Patch Set 7:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/24534/5/be/src/runtime/types.cc
File be/src/runtime/types.cc:

http://gerrit.cloudera.org:8080/#/c/24534/5/be/src/runtime/types.cc@210
PS5, Line 210:    case TYPE_TIMESTAMP: return "timestamp";
             :     case TYPE_STRING:
             :       switch (col_type) {
             :         case TPrimitiveType::GEOMETRY: return "geometry";
             :         case TPrimitiveType::BINARY: return "binary";
             :         default: return "string";
             :       }
> Done
note that this chain of commits that changed type handling a bit: 
https://gerrit.cloudera.org/#/c/24567/



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0ff29cb15ab45f3899bf453041e605a32f195c27
Gerrit-Change-Number: 24534
Gerrit-PatchSet: 7
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Balazs Hevele <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Comment-Date: Fri, 10 Jul 2026 10:27:45 +0000
Gerrit-HasComments: Yes

Reply via email to