Add register_table procedure usage instructions#25784
Conversation
| default. The procedure is enabled only when | ||
| `iceberg.register-table-procedure.enabled` is set to `true`. | ||
|
|
||
| Avoid registering multiple tables with the same location. |
There was a problem hiding this comment.
Reasoning behind the recommandation is missing
There was a problem hiding this comment.
@findinpath I follow ebyhr's suggestion and remove it before. #25536 (comment)
now I have re-described it, please help review again, thanks~
There was a problem hiding this comment.
I wasn't suggesting removing the reason - the removed part isn't the "reason". It's just a protection rule.
XavieLee
force-pushed
the
update/doc/register_table
branch
from
b6abfff to
a38189f
Compare
| default. The procedure is enabled only when | ||
| `iceberg.register-table-procedure.enabled` is set to `true`. | ||
|
|
||
| Ensure tables are registered using their prior storage path, and avoid associating multiple tables with identical locations. |
There was a problem hiding this comment.
"Ensure tables are registered using their prior storage path"
Users don't have to ensure that. The procedure blocks such execution either way.
|
This pull request has gone a while without any activity. Ask for help on #core-dev on Trino slack. |
|
Closing this pull request, as it has been stale for six weeks. Feel free to re-open at any time. |
3 participants
Description
People try to register tables in a different location and face query failures later, add description for this function.
Additional context and related issues
#16105
Release notes
(x) This is not user-visible or is docs only, and no release notes are required.