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 2022/11/09 08:17:18 UTC

[GitHub] [iceberg] gaborkaszab opened a new pull request, #6154: Core: Rename HMS_TABLE_OWNER to follow naming convention

gaborkaszab opened a new pull request, #6154:
URL: https://github.com/apache/iceberg/pull/6154

   I introduced this property in #5763, however, I learned since that its value doesn't follow the naming conventions for a table property. Fortunately the change hasn't been released yet so we can safely change this property before we release 1.1.0. and won't hurt backward compatibility.


-- 
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: issues-unsubscribe@iceberg.apache.org

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 merged pull request #6154: Core: Rename HMS_TABLE_OWNER to follow naming convention

Posted by GitBox <gi...@apache.org>.
szehon-ho merged PR #6154:
URL: https://github.com/apache/iceberg/pull/6154


-- 
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: issues-unsubscribe@iceberg.apache.org

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 pull request #6154: Core: Rename HMS_TABLE_OWNER to follow naming convention

Posted by GitBox <gi...@apache.org>.
szehon-ho commented on PR #6154:
URL: https://github.com/apache/iceberg/pull/6154#issuecomment-1308490447

   Makes sense based on discussion: https://github.com/apache/iceberg/pull/6045#discussion_r1010730010 .  Thanks @gaborkaszab and @nastra for review


-- 
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: issues-unsubscribe@iceberg.apache.org

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