You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2021/04/06 04:05:35 UTC

[GitHub] [iceberg] aokolnychyi opened a new issue #2424: Consider deprecating updating table state through ALTER TABLE ... SET TBLPROPERTIES

aokolnychyi opened a new issue #2424:
URL: https://github.com/apache/iceberg/issues/2424


   I'd consider deprecating and then removing support for changing the table location, current snapshot id, etc through ALTER TABLE ... SET TBLPROPERTIES. We have procedures/commands for most operations now. If missing, let's add more.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] szehon-ho commented on issue #2424: Consider deprecating updating table state through ALTER TABLE ... SET TBLPROPERTIES

Posted by GitBox <gi...@apache.org>.
szehon-ho commented on issue #2424:
URL: https://github.com/apache/iceberg/issues/2424#issuecomment-816565272


   Talked offline, i guess its more to do with snapshot, already supported by procedure.  location can be changed via ALTER TABLE SET LOCATION


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] szehon-ho commented on issue #2424: Consider deprecating updating table state through ALTER TABLE ... SET TBLPROPERTIES

Posted by GitBox <gi...@apache.org>.
szehon-ho commented on issue #2424:
URL: https://github.com/apache/iceberg/issues/2424#issuecomment-814787726


   Is there any procedure to change table location (pretty useful)?  Could not find it so far in spark-procedure.md doc: https://github.com/apache/iceberg/blob/90225d6c9413016d611e2ce5eff37db1bc1b4fc5/site/docs/spark-procedures.md
   
   
   
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] szehon-ho commented on issue #2424: Consider deprecating updating table state through ALTER TABLE ... SET TBLPROPERTIES

Posted by GitBox <gi...@apache.org>.
szehon-ho commented on issue #2424:
URL: https://github.com/apache/iceberg/issues/2424#issuecomment-815029911


   Looked through the latest code and didn't find it unless I missed it.  If nobody is working on it, I can take a look to add 'update_location' procedure (and maybe update_properties) to familiar myself with this code?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] szehon-ho edited a comment on issue #2424: Consider deprecating updating table state through ALTER TABLE ... SET TBLPROPERTIES

Posted by GitBox <gi...@apache.org>.
szehon-ho edited a comment on issue #2424:
URL: https://github.com/apache/iceberg/issues/2424#issuecomment-815029911


   Looked through the latest code and didn't find it unless I missed it.  If nobody is working on it, I can take a look to add 'update_location' procedure (and maybe update_properties) ?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org