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

    https://github.com/apache/spark/pull/15943#discussion_r88796208
  
    --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
    @@ -705,12 +701,10 @@ class Dataset[T] private[sql](
        *
        * @param right Right side of the join operation.
        * @param usingColumn Name of the column to join on. This column must 
exist on both sides.
    -   *
    -   * @note If you perform a self-join using this function without aliasing 
the input
    -   * [[DataFrame]]s, you will NOT be able to reference any columns after 
the join, since
    -   * there is no way to disambiguate which side of the join you would like 
to reference.
    -   *
    -   * @group untypedrel
    +    * @note If you perform a self-join using this function without 
aliasing the input
    +   *        [[DataFrame]]s, you will NOT be able to reference any columns 
after the join, since
    +   *        there is no way to disambiguate which side of the join you 
would like to reference.
    +    * @group untypedrel
    --- End diff --
    
    It seems my IntelliJ does not. Whether it is a problem or not, reformatting 
seems not the goal of this PR and I guess it seems reformatting is arguable. 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to