You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2023/01/19 18:38:03 UTC
[GitHub] [hudi] vectorijk opened a new pull request, #7710: remove duplicated match clause in MergeInto syntax
vectorijk opened a new pull request, #7710:
URL: https://github.com/apache/hudi/pull/7710
### Documentation Update
remove duplicated match clause in MergeInto syntax
### Contributor's checklist
- [x] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute)
- [ ] Change Logs and Impact were stated clearly
- [ ] Adequate tests were added if applicable
- [ ] CI passed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [hudi] danny0405 commented on a diff in pull request #7710: [Doc] [minor] remove duplicated match clause in MergeInto syntax
Posted by GitBox <gi...@apache.org>.
danny0405 commented on code in PR #7710:
URL: https://github.com/apache/hudi/pull/7710#discussion_r1082141639
##########
website/versioned_docs/version-0.12.2/quick-start-guide.md:
##########
@@ -761,7 +761,6 @@ MERGE INTO tableIdentifier AS target_alias
USING (sub_query | tableIdentifier) AS source_alias
ON <merge_condition>
[ WHEN MATCHED [ AND <condition> ] THEN <matched_action> ]
-[ WHEN MATCHED [ AND <condition> ] THEN <matched_action> ]
[ WHEN NOT MATCHED [ AND <condition> ] THEN <not_matched_action> ]
Review Comment:
Can we fix all the versions that support MERGE INTO?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [hudi] nsivabalan merged pull request #7710: [DOC] Remove duplicated match clause in MergeInto syntax
Posted by "nsivabalan (via GitHub)" <gi...@apache.org>.
nsivabalan merged PR #7710:
URL: https://github.com/apache/hudi/pull/7710
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [hudi] vectorijk commented on a diff in pull request #7710: [Doc] [minor] remove duplicated match clause in MergeInto syntax
Posted by "vectorijk (via GitHub)" <gi...@apache.org>.
vectorijk commented on code in PR #7710:
URL: https://github.com/apache/hudi/pull/7710#discussion_r1089267892
##########
website/versioned_docs/version-0.12.2/quick-start-guide.md:
##########
@@ -761,7 +761,6 @@ MERGE INTO tableIdentifier AS target_alias
USING (sub_query | tableIdentifier) AS source_alias
ON <merge_condition>
[ WHEN MATCHED [ AND <condition> ] THEN <matched_action> ]
-[ WHEN MATCHED [ AND <condition> ] THEN <matched_action> ]
[ WHEN NOT MATCHED [ AND <condition> ] THEN <not_matched_action> ]
Review Comment:
remove this line in all version of docs
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org