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

    https://github.com/apache/spark/pull/16609#discussion_r97210569
  
    --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
    @@ -322,6 +333,9 @@ class Dataset[T] private[sql](
       }
     
       override def toString: String = {
    +    if (name != null) {
    +      return name
    +    }
    --- End diff --
    
    The storage tab gets it from the cache manager (which gets it via 
Dataset.persist()). I think this is what will be displayed when the Dataset is 
printed in the repl, I'm not sure if it's also used elsewhere.
    Given that there's also a .name method I agree that it would be nice to 
keep the schema in here. I'll add it in, but will need to update the tests to 
take that into account.


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