Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-19 Thread via GitHub
dramaticlly commented on PR #12115: URL: https://github.com/apache/iceberg/pull/12115#issuecomment-2669456273 Special shoutout to @szehon-ho for iterating and refining the documentation, also appreciate @RussellSpitzer @flyrain @singhpk234 @manuzhang for additional input and review -- T

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-18 Thread via GitHub
szehon-ho merged PR #12115: URL: https://github.com/apache/iceberg/pull/12115 -- 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-unsubscr...@iceberg

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-18 Thread via GitHub
szehon-ho commented on PR #12115: URL: https://github.com/apache/iceberg/pull/12115#issuecomment-2667739875 Merged, thanks again ! -- 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 com

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-18 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1960797057 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,94 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-18 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1960777677 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,97 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-18 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1960779365 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,97 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-18 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1960776871 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,97 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-16 Thread via GitHub
dramaticlly commented on PR #12115: URL: https://github.com/apache/iceberg/pull/12115#issuecomment-2662241789 > I think it almost looks ready, I just had one last comment, sorry about not putting it correctly before ! > > I think when I looked last night at for [#12115 (comment)](htt

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-16 Thread via GitHub
szehon-ho commented on PR #12115: URL: https://github.com/apache/iceberg/pull/12115#issuecomment-2662217136 I think it almost looks ready, I just had one last comment, sorry about that! I think when I looked last night at for https://github.com/apache/iceberg/pull/12115#discussion_r19

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957072281 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,101 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957074174 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,101 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957074174 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,101 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957073932 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,101 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957073691 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,101 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957073492 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,98 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957073109 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,101 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957071171 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,98 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-15 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957066734 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,98 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-14 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1956628329 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-12 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1953619836 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics o

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-12 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1953617634 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics o

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-10 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1950358255 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-10 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1950358255 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-10 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1950356735 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-10 Thread via GitHub
dramaticlly commented on PR #12115: URL: https://github.com/apache/iceberg/pull/12115#issuecomment-2648793414 @RussellSpitzer @flyrain @szehon-ho appreciate another look on the docs. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-10 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1949601619 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics o

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-10 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1949600678 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics o

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-10 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1949595771 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics o

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1947552020 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1947552684 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1947552684 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1947552020 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1947551705 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
manuzhang commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1947551167 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1947547180 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics o

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1947546783 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-08 Thread via GitHub
dramaticlly commented on PR #12115: URL: https://github.com/apache/iceberg/pull/12115#issuecomment-2644728950 With #11929 merged, I can remove the warning in the documentation for table with statistics files -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-06 Thread via GitHub
flyrain commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1945855015 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,100 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of th

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-06 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1945680164 ## docs/docs/spark-procedures.md: ## @@ -976,18 +976,18 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => ## Table Replicati

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-06 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1945679456 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,101 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics o

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-06 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1945624171 ## docs/docs/spark-procedures.md: ## @@ -976,18 +976,18 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => ## Table Replication

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-06 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1945547732 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-02 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1938207738 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-02 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1938841961 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-29 Thread via GitHub
dramaticlly commented on PR #12115: URL: https://github.com/apache/iceberg/pull/12115#issuecomment-2622983101 thank you Russell for the detailed review! This is getting closer and I did a local check to ensure the info and warning section rendered as expected like in following screenshot. h

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-29 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1934566388 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-29 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1934550850 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-29 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1934549172 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-29 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1934547493 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-29 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1934547087 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932694495 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932694495 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932694495 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932576282 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932573350 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932567338 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932565068 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932541292 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-28 Thread via GitHub
singhpk234 commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1932470513 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
dramaticlly commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931319129 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
dramaticlly commented on PR #12115: URL: https://github.com/apache/iceberg/pull/12115#issuecomment-2617131258 thank you @flyrain @RussellSpitzer for the promptly review, updated per your suggestion! -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931221140 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931220126 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931219808 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931213481 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931212445 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931210421 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931203849 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931203205 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931202570 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
flyrain commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931199301 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of the

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931198658 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931198111 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931195406 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931192700 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
RussellSpitzer commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931192700 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
flyrain commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1931184314 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,86 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of the

[PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-01-27 Thread via GitHub
dramaticlly opened a new pull request, #12115: URL: https://github.com/apache/iceberg/pull/12115 Documentation for #11931 #11555 I took wording and docs with original authro of @flyrain and @szehon-ho Also @manuzhang is interested in this and wondering if @amogh-jahagirdar is rea