steFaiz commented on code in PR #7889:
URL: https://github.com/apache/paimon/pull/7889#discussion_r3273138043


##########
paimon-common/src/main/java/org/apache/paimon/data/Blob.java:
##########
@@ -38,6 +38,31 @@
 @Public
 public interface Blob {
 
+    /**
+     * The placeholder blob, mainly for blob update in data-evolution. It 
should never be exposed to
+     * users.
+     */
+    Blob PLACE_HOLDER =

Review Comment:
   Thanks for your advise! But in https://github.com/apache/paimon/pull/7125 we 
supports storing nulls in blob file. I'm not clear how to distinguish 
placeholders and native NULLs if so. 
   
   From the semantics, NULLs are exposed to users, users know that they store 
some nulls. But placeholders are fully internal used, users should never be 
aware about them. If users set some rows as nulls, we may fallback those rows 
to earlier versions, this is not expected in our design.
   
   Could you please give me some advise?



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