[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-04-02 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-608218137 @HyukjinKwon You can merge this PR to branch-3.0 first.

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-04-02 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-608210409 @HyukjinKwon I know you mean. It is too difficult to solve in a PR. I can make one PR for large files

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-21 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-589899591 @HyukjinKwon @maropu @cloud-fan @Ngone51 @dongjoon-hyun @gatorsmile @jiangxb1987 @HeartSaVioR Thanks

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-21 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-589611133 @HyukjinKwon @cloud-fan I count the number of entries displayed with `set -v` is 121.

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-21 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-589593887 > @beliefer, why don't you just write down the versions in this PR per the concerns about master vs

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-19 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-588185025 retest this please This is an

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-18 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-587979039 retest this please This is an

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-18 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-587398581 retest this please This is an

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-18 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-587333560 retest this please This is an

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-16 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-586793691 @HyukjinKwon I need a lot of time to sort out their version numbers, not immediately, so I want to

[GitHub] [spark] beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder

2020-02-16 Thread GitBox
beliefer commented on issue #27592: [SPARK-30840][CORE][SQL] Add version property for ConfigEntry and ConfigBuilder URL: https://github.com/apache/spark/pull/27592#issuecomment-586700066 > For reviewers, can you add a screenshot of a html document generated by `gen-sql-config-docs.py` in