jayzhan211 commented on PR #14046: URL: https://github.com/apache/datafusion/pull/14046#issuecomment-2579134148
> > I think we should fix it on the display/formatting side. For example, we still cannot distinguish: > > ``` > > DataFusion CLI v44.0.0 > > > > > select array[], array[null]; > > +--------------+------------------+ > > | make_array() | make_array(NULL) | > > +--------------+------------------+ > > | [] | [] | > > +--------------+------------------+ > > 1 row(s) fetched. > > Elapsed 0.007 seconds. > > ``` > > > > > > > > > > > > > > > > > > > > > > > > See [#13872 (comment)](https://github.com/apache/datafusion/issues/13872#issuecomment-2579050717) > > makes sense, should we start showing NULL for the null values ? It will make things easy for us. Even the postgres, apache spark and others shows, NULL for null values. > > ``` > > select array[], array[null]; > +--------------+------------------+ > | make_array() | make_array(NULL) | > +--------------+------------------+ > | [] | [NULL] | > +--------------+------------------+ > 1 row(s) fetched. > Elapsed 0.007 seconds. > ``` > > This will make changes everywhere. I think we should start showing `NULL` for null values. > > @jayzhan211 @jonahgao This looks fine to me -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
