tustvold commented on code in PR #4750:
URL: https://github.com/apache/arrow-datafusion/pull/4750#discussion_r1057845030


##########
datafusion/core/src/dataframe.rs:
##########
@@ -88,12 +88,6 @@ impl DataFrame {
 
     /// Create a physical plan
     pub async fn create_physical_plan(mut self) -> Result<Arc<dyn 
ExecutionPlan>> {
-        self.create_physical_plan_impl().await
-    }
-
-    /// Temporary pending #4626

Review Comment:
   I no longer intend to do this, as it creates far too much churn to justify 
it, this PR instead opts to workaround the issue by punting responsibility for 
setting the execution start time onto the caller if they're using the 
lower-level APIs.



-- 
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]

Reply via email to