You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2021/05/05 06:32:36 UTC

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #6879: Allow empty string in MV column

Jackie-Jiang opened a new pull request #6879:
URL: https://github.com/apache/incubator-pinot/pull/6879


   ## Description
   Currently we allow empty string as SV string value, but not allow it in the MV string value. This inconsistent behavior can cause confusion and unexpected behavior, especially when people sets empty string as the column default value.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #6879: Allow empty string in MV column

Posted by GitBox <gi...@apache.org>.
Jackie-Jiang merged pull request #6879:
URL: https://github.com/apache/incubator-pinot/pull/6879


   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org