alamb commented on code in PR #15016: URL: https://github.com/apache/datafusion/pull/15016#discussion_r1981151334
########## datafusion/common/src/scalar/mod.rs: ########## @@ -2764,15 +2764,6 @@ impl ScalarValue { Ok(scalars) } - // TODO: Support more types after other ScalarValue is wrapped with ArrayRef - /// Get raw data (inner array) inside ScalarValue - pub fn raw_data(&self) -> Result<ArrayRef> { Review Comment: Thank you very much @qazxcdswe123 -- this is a nice find. Per https://datafusion.apache.org/contributor-guide/api-health.html instead of removing this function, can you please mark it as #[deprecated] instead? Maybe it would be a good time to go remove any deprecated functions from 41 or earlier 🤔 -- 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