corgy-w commented on code in PR #7401:
URL: https://github.com/apache/seatunnel/pull/7401#discussion_r1722629010


##########
seatunnel-api/src/main/java/org/apache/seatunnel/api/table/type/VectorType.java:
##########
@@ -30,8 +30,8 @@ public class VectorType<T> implements SeaTunnelDataType<T> {
     public static final VectorType<Map> VECTOR_SPARSE_FLOAT_TYPE =
             new VectorType<>(Map.class, SqlType.SPARSE_FLOAT_VECTOR);
 
-    public static final VectorType<Byte> VECTOR_BINARY_TYPE =
-            new VectorType<>(Byte.class, SqlType.BINARY_VECTOR);
+    public static final VectorType<ByteBuffer> VECTOR_BINARY_TYPE =

Review Comment:
   Currently first connector of binary vectors is implemented in this pr with 
no historical version problem



-- 
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