Hisoka-X commented on code in PR #7060:
URL: https://github.com/apache/seatunnel/pull/7060#discussion_r1666726117


##########
seatunnel-api/src/main/java/org/apache/seatunnel/api/table/type/SqlType.java:
##########
@@ -36,5 +36,10 @@ public enum SqlType {
     TIME,
     TIMESTAMP,
     ROW,
+    BINARY_VECTOR,
+    FLOAT_VECTOR,
+    FLOAT16_VECTOR,
+    BFLOAT16_VECTOR,
+    SPARSE_FLOAT_VECTOR,

Review Comment:
   What's different bwtween `FLOAT_VECTOR` and `FLOAT` array in SeaTunnel? 
Since we use arrays to store data in memory, why not continue to use arrays 
instead of adding a new type? In the current SeaTunnel, vector data does not 
bring any reason to replace arrays.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to