[GitHub] [spark] xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan

2019-11-09 Thread GitBox
xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan URL: https://github.com/apache/spark/pull/26167#issuecomment-552107036 Thanks @cloud-fan ! This

[GitHub] [spark] xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan

2019-11-05 Thread GitBox
xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan URL: https://github.com/apache/spark/pull/26167#issuecomment-550139637 @cloud-fan I updated the code. A notable change is the catalog resolve logic for `MergeInt

[GitHub] [spark] xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan

2019-11-05 Thread GitBox
xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan URL: https://github.com/apache/spark/pull/26167#issuecomment-549768500 @cloud-fan , I have some doubts. As the doc in `ParsedStatement`, a `ParsedStatement` is n

[GitHub] [spark] xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan

2019-10-31 Thread GitBox
xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan URL: https://github.com/apache/spark/pull/26167#issuecomment-548629211 retest this please. This

[GitHub] [spark] xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan

2019-10-24 Thread GitBox
xianyinxin commented on issue #26167: [SPARK-28893][SQL] Support MERGE INTO in the parser and add the corresponding logical plan URL: https://github.com/apache/spark/pull/26167#issuecomment-546184439 cc @cloud-fan @rdblue Th