Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/7774#discussion_r36062013
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala ---
@@ -119,6 +119,9 @@ class DataFrame private[sql](
@transient val sqlContext: SQLContext,
@DeveloperApi @transient val queryExecution:
SQLContext#QueryExecution) extends Serializable {
+ // Note for Spark contributors: if adding or updating any action in
`DataFrame`, please make sure
--- End diff --
> @zsxwing This looks great. One high level question: why are there so few
dataframe operations with withNewExecutionId? Are you wrapping only those that
will trigger an action? If so you should clarify that in the java doc.
@andrewor14 I added the comment here because I think it should only be
visible to contributors.
---
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]
