Doc: Update Instructions for rewrite_table_path. #12628
Conversation
|
@manuzhang Could you please help review this PR? Thank you very much! It's a small change. |
|
@manuzhang I also noticed a small change. Since #11147 has been fixed and the latest version of Iceberg (apache-iceberg-1.8.1) has been released, which includes the fix for #11157, I believe we can remove the warning. Given that this change is minor, I don't think we need to submit a separate PR for the fix. |
|
@manuzhang Gentle ping! could you helping to merge this PR? Thank you very much! |
|
This looks fine to me, but actually would you mind putting the revert of warning for #11157 in separate pr for book-keeping purpose and ease of revert / cherry pick (if necessary) |
@szehon-ho Thank you for your message! I will split the pr and submit it according to your suggestion. |
|
Merged, thanks @slfan1989 for pr and extra work splitting it, and also @manuzhang @ebyhr for review! |
Doc: Update Instructions for rewrite_table_path.
We have recently been using rewrite_table_path and noticed an issue with the use of quotes in the examples provided in the documentation. While other procedures use single quotes for variables, this stored procedure uses double quotes.