findepi commented on PR #16615: URL: https://github.com/apache/datafusion/pull/16615#issuecomment-3031617136
> nd cleans up the API semantics (like there's no difference between `None` and an empty `Vec`) Now there isn't and indeed that was the motivation for the change. Before the change, `None` and `Some(vec![])` would behave differently, basically Some(vec![])` was an erroneous value leading to potential bugs. > So I think that's a worthwhile change for the next major release. Do i understand correctly the next major is the next release going off `main` branch? I.e. we can merge as is now? -- 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
