Thomas-HuWei commented on code in PR #7060:
URL: https://github.com/apache/seatunnel/pull/7060#discussion_r1667868356
##########
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:
Because vectors are just display as arrays, but the management and
processing methods are different from arrays.
For example, arrays have no dimension restrictions, while the data in
vectors has fixed dimensions.
If the array type is used, in the future, if functions are expanded or data
is verified, special processing must be performed for the vector situation,
which will make the code look very strange and not decoupled, so the vector
type is added.
In addition, many vector databases on the market have separate types for
vector columns, and arrays and vectors do not share the same type in terms of
external SQL types.
--
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]
