GitHub user caneGuy opened a pull request:

    https://github.com/apache/spark/pull/20845

    [SPARK-23708][CORE] Correct comment for function addShutDownHook in 
ShutdownHookManager

    ## What changes were proposed in this pull request?
    Comment below is not right.
    ```
    /**
       * Adds a shutdown hook with the given priority. Hooks with lower 
priority values run
       * first.
       *
       * @param hook The code to run during shutdown.
       * @return A handle that can be used to unregister the shutdown hook.
       */
      def addShutdownHook(priority: Int)(hook: () => Unit): AnyRef = {
        shutdownHooks.add(priority, hook)
      }
    ```
    
    ## How was this patch tested?
    
    UT


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/caneGuy/spark zhoukang/fix-shutdowncomment

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/20845.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #20845
    
----
commit 0199a30783dcef86c5e643fdbcf8b31d4675326b
Author: zhoukang <zhoukang199191@...>
Date:   2018-03-16T09:21:58Z

    Correct comment for function addShutDownHook in ShutdownHookManager

----


---

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

Reply via email to