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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
