GitHub user vanzin opened a pull request:

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

    [SPARK-22779][sql] Resolve default values for fallback configs.

    SQLConf allows some callers to define a custom default value for
    configs, and that complicates a little bit the handling of fallback
    config entries, since most of the default value resolution is
    hidden by the config code.
    
    This change peaks into the internals of these fallback configs
    to figure out the correct default value, and also returns the
    current human-readable default when showing the default value
    (e.g. through "set -v").

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

    $ git pull https://github.com/vanzin/spark SPARK-22779

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

    https://github.com/apache/spark/pull/19974.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 #19974
    
----
commit 4402fc7ae33cf3bd1d5c2c8f2dcf380a90615f7c
Author: Marcelo Vanzin <[email protected]>
Date:   2017-12-14T02:49:28Z

    [SPARK-22779][sql] Resolve default values for fallback configs.
    
    SQLConf allows some callers to define a custom default value for
    configs, and that complicates a little bit the handling of fallback
    config entries, since most of the default value resolution is
    hidden by the config code.
    
    This change peaks into the internals of these fallback configs
    to figure out the correct default value, and also returns the
    current human-readable default when showing the default value
    (e.g. through "set -v").

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to