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 2022/04/24 11:08:38 UTC

[GitHub] [hudi] kazdy opened a new pull request, #5413: Document alter table unset tblproperties comand

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

   Hi team, 
   I wanted to add unset documentation for hudi spark sql, which I was searching for, so this might be useful for others.
   I tried it on my tables and worked perfectly fine as this is the syntax from spark sql documentation.
   
   Will this be merged to 0.11 docs or should I open a new PR?
   
   ## What is the purpose of the pull request
   
   This pull request modifies quick-start document
   
   ## Brief change log
   
   Add alter table unset tblproperties command documentation.
   
   ## Verify this pull request
   
   *(Please pick either of the following options)*
   
   This pull request is a trivial rework  of spark guide.
   
   ## Committer checklist
   
    - [ ] Has a corresponding JIRA in PR title & commit
    
    - [ ] Commit message is descriptive of the change
    
    - [ ] CI is green
   
    - [ ] Necessary doc changes done or have another open PR
          
    - [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
   


-- 
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] kazdy commented on pull request #5413: Document alter table unset tblproperties comand

Posted by GitBox <gi...@apache.org>.
kazdy commented on PR #5413:
URL: https://github.com/apache/hudi/pull/5413#issuecomment-1110730357

   Hi, 
   I see similar thing has been added to 0.11 docs:
   https://hudi.apache.org/docs/next/quick-start-guide#modifying-table-properties
   
   Not sure if this PR is needed anymore, maybe only for 0.10.1?


-- 
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] xushiyan merged pull request #5413: [MINOR][DOCS] Document alter table unset tblproperties comand

Posted by GitBox <gi...@apache.org>.
xushiyan merged PR #5413:
URL: https://github.com/apache/hudi/pull/5413


-- 
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] kazdy commented on pull request #5413: Document alter table unset tblproperties comand

Posted by GitBox <gi...@apache.org>.
kazdy commented on PR #5413:
URL: https://github.com/apache/hudi/pull/5413#issuecomment-1113969159

   Let's merge this PR to 0.10.1 docs, should bring some value :)
   I can't merge it myself, could You please merge it?


-- 
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] yihua commented on pull request #5413: Document alter table unset tblproperties comand

Posted by GitBox <gi...@apache.org>.
yihua commented on PR #5413:
URL: https://github.com/apache/hudi/pull/5413#issuecomment-1113861749

   > Not sure if this PR is needed anymore, maybe only for 0.10.1?
   
   I see you change the 0.10.1 docs only.  If you think they are still valuable, we can merge this PR.  You may also backport more example commands from the latest 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