Github user mgaido91 commented on the issue:

    https://github.com/apache/spark/pull/20883
  
    please update the description moving
    
    > Fixes SPARK-23759 by moving connector.start() after connector.setHost()
    
    after the title (What changes...) and please remove the sentence:
    
    > This pull is to fix SPARK-23759 issue
    
    since you are referencing the JIRA in the commit message, thus it is 
obvious that it fixes that JIRA...
    
    Other than this, LGTM


---

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

Reply via email to