Github user marmbrus commented on a diff in the pull request:

    https://github.com/apache/spark/pull/6356#discussion_r41087599
  
    --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala ---
    @@ -114,7 +114,7 @@ private[sql] object DataFrame {
     @Experimental
     class DataFrame private[sql](
         @transient val sqlContext: SQLContext,
    -    @DeveloperApi @transient val queryExecution: 
SQLContext#QueryExecution) extends Serializable {
    +    @DeveloperApi @transient val queryExecution: QueryExecution) extends 
Serializable {
    --- End diff --
    
    The signature of the function to retrieve it has changed though.  I think 
that we should rename this parameter and add a method that constructs a new 
instance of the inner subclass to return with the name `queryExecution`.  I 
think its worth rerunning the optimizer to not have to recompile all our perf 
tools.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to