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

Reply via email to