[GitHub] [spark] cloud-fan edited a comment on pull request #29146: [SPARK-32257][SQL] Reports explicit errors for invalid usage of SET command

2020-07-24 Thread GitBox
cloud-fan edited a comment on pull request #29146: URL: https://github.com/apache/spark/pull/29146#issuecomment-663569282 If we don't allow space in the config name by default(requires quoting), I think we can do that for other special chars as well. Then the parser rule can be very simple

[GitHub] [spark] cloud-fan edited a comment on pull request #29146: [SPARK-32257][SQL] Reports explicit errors for invalid usage of SET command

2020-07-24 Thread GitBox
cloud-fan edited a comment on pull request #29146: URL: https://github.com/apache/spark/pull/29146#issuecomment-663569282 If we don't allow space in the config name by default(requires quoting), I think we can do that for other special chars as well. Then the parser rule can be very simple

[GitHub] [spark] cloud-fan edited a comment on pull request #29146: [SPARK-32257][SQL] Reports explicit errors for invalid usage of SET command

2020-07-24 Thread GitBox
cloud-fan edited a comment on pull request #29146: URL: https://github.com/apache/spark/pull/29146#issuecomment-663569282 If we don't allow space in the config name by default(requires quoting), I think we can do that for other special chars as well. Then the parser rule can be very simple