You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "lvyanquan (via GitHub)" <gi...@apache.org> on 2023/02/21 09:42:11 UTC

[GitHub] [hudi] lvyanquan opened a new pull request, #8004: [MINOR][DOCS] Update SQL in procedures.md

lvyanquan opened a new pull request, #8004:
URL: https://github.com/apache/hudi/pull/8004

   ### Change Logs
   
   update doc of procedures about savepoint since this pr https://github.com/apache/hudi/pull/6186/files was merged.
   And add doc about usage of 'show_savepoints'.
   
   ### Impact
   
   only document, no code changed.
   
   ### Risk level (write none, low medium or high below)
   
   none
   
   ### Documentation Update
   
   Spark SQL [procedures](https://hudi.apache.org/docs/next/procedures#create_savepoints)
   
   ### Contributor's checklist
   
   - [ ] 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] bvaradar commented on pull request #8004: [HUDI-5856][DOCS] Update SQL in procedures.md

Posted by "bvaradar (via GitHub)" <gi...@apache.org>.
bvaradar commented on PR #8004:
URL: https://github.com/apache/hudi/pull/8004#issuecomment-1477218449

   @lvyanquan : Can you please fix the failing test. Also please rebase as the PR has been opened for a while. 


-- 
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] XuQianJin-Stars commented on pull request #8004: [HUDI-5856][DOCS] Update SQL in procedures.md

Posted by "XuQianJin-Stars (via GitHub)" <gi...@apache.org>.
XuQianJin-Stars commented on PR #8004:
URL: https://github.com/apache/hudi/pull/8004#issuecomment-1449907476

   > Resubmit the document to facilitate the addition of new content. Considering that there are more updated content, a new [jira tracking](https://issues.apache.org/jira/browse/HUDI-5856) is created, You may check it if you have time. @bvaradar @SteNicholas @XuQianJin-Stars
   > 
   > Currently, only unreleased documentation has been modified, and will copy to version documentation if needed.
   
   Good tracking.


-- 
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] bvaradar commented on pull request #8004: [MINOR][DOCS] Update SQL in procedures.md

Posted by "bvaradar (via GitHub)" <gi...@apache.org>.
bvaradar commented on PR #8004:
URL: https://github.com/apache/hudi/pull/8004#issuecomment-1439410852

   Will wait for procedures to be added before landing this.


-- 
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] lvyanquan commented on pull request #8004: [HUDI-5856][DOCS] Update SQL in procedures.md

Posted by "lvyanquan (via GitHub)" <gi...@apache.org>.
lvyanquan commented on PR #8004:
URL: https://github.com/apache/hudi/pull/8004#issuecomment-1445815942

   Resubmit the document to facilitate the addition of new content. Considering that there are more updated content, a new [jira tracking](https://issues.apache.org/jira/browse/HUDI-5856) is created, You may check it if you have time.
   @bvaradar @SteNicholas @XuQianJin-Stars 
   
   Currently, only unreleased documentation has been modified, and will copy to version documentation if needed.


-- 
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] bvaradar merged pull request #8004: [HUDI-5856][DOCS] Update SQL in procedures.md

Posted by "bvaradar (via GitHub)" <gi...@apache.org>.
bvaradar merged PR #8004:
URL: https://github.com/apache/hudi/pull/8004


-- 
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] SteNicholas commented on pull request #8004: [MINOR][DOCS] Update SQL in procedures.md

Posted by "SteNicholas (via GitHub)" <gi...@apache.org>.
SteNicholas commented on PR #8004:
URL: https://github.com/apache/hudi/pull/8004#issuecomment-1438502986

   @lvyanquan, could you complete the missing supported procedures in `procedures.md`? At present `procedures.md` doesn't list the all supported procedures. cc @XuQianJin-Stars


-- 
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] lvyanquan commented on pull request #8004: [MINOR][DOCS] Update SQL in procedures.md

Posted by "lvyanquan (via GitHub)" <gi...@apache.org>.
lvyanquan commented on PR #8004:
URL: https://github.com/apache/hudi/pull/8004#issuecomment-1439304237

   willing to do this and will try to complete it later.


-- 
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] lvyanquan commented on pull request #8004: [HUDI-5856][DOCS] Update SQL in procedures.md

Posted by "lvyanquan (via GitHub)" <gi...@apache.org>.
lvyanquan commented on PR #8004:
URL: https://github.com/apache/hudi/pull/8004#issuecomment-1477308314

   > @lvyanquan : Can you please fix the failing test. Also please rebase as the PR has been opened for a while.
   
   Done. This pr is rebased on [apache:asf-site](https://github.com/apache/hudi/tree/asf-site) branch and add another commit for https://github.com/apache/hudi/pull/8202


-- 
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