Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-17 Thread via GitHub
Fokko commented on code in PR #602: URL: https://github.com/apache/iceberg-python/pull/602#discussion_r1568666707 ## tests/integration/test_inspect_table.py: ## @@ -274,6 +274,64 @@ def test_inspect_entries_partitioned(spark: SparkSession, session_catalog: Catal

Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-17 Thread via GitHub
Fokko commented on PR #602: URL: https://github.com/apache/iceberg-python/pull/602#issuecomment-2061026458 Thanks @geruh for working on this  -- 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

Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-17 Thread via GitHub
Fokko merged PR #602: URL: https://github.com/apache/iceberg-python/pull/602 -- 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:

Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-16 Thread via GitHub
Fokko commented on PR #602: URL: https://github.com/apache/iceberg-python/pull/602#issuecomment-2059988471 @geruh Thanks for creating this PR! Can you resolve the conflicts so we can get this in? Thanks! -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-16 Thread via GitHub
Fokko commented on code in PR #602: URL: https://github.com/apache/iceberg-python/pull/602#discussion_r1567985291 ## pyiceberg/table/__init__.py: ## @@ -3410,6 +3410,32 @@ def _readable_metrics_struct(bound_type: PrimitiveType) -> pa.StructType:

Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-15 Thread via GitHub
geruh commented on code in PR #602: URL: https://github.com/apache/iceberg-python/pull/602#discussion_r1565252853 ## tests/integration/test_inspect_table.py: ## @@ -266,3 +266,56 @@ def test_inspect_entries_partitioned(spark: SparkSession, session_catalog: Catal assert

Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-15 Thread via GitHub
geruh commented on code in PR #602: URL: https://github.com/apache/iceberg-python/pull/602#discussion_r1565238973 ## pyiceberg/table/__init__.py: ## @@ -3410,6 +3410,32 @@ def _readable_metrics_struct(bound_type: PrimitiveType) -> pa.StructType:

Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-13 Thread via GitHub
amogh-jahagirdar commented on code in PR #602: URL: https://github.com/apache/iceberg-python/pull/602#discussion_r1564265956 ## tests/integration/test_inspect_table.py: ## @@ -266,3 +266,56 @@ def test_inspect_entries_partitioned(spark: SparkSession, session_catalog: Catal

Re: [PR] Add Refs metadata table [iceberg-python]

2024-04-13 Thread via GitHub
Fokko commented on code in PR #602: URL: https://github.com/apache/iceberg-python/pull/602#discussion_r1564223181 ## pyiceberg/table/__init__.py: ## @@ -3410,6 +3410,32 @@ def _readable_metrics_struct(bound_type: PrimitiveType) -> pa.StructType:

[PR] Add Refs metadata table [iceberg-python]

2024-04-12 Thread via GitHub
geruh opened a new pull request, #602: URL: https://github.com/apache/iceberg-python/pull/602 This PR adds the Refs metadata table the existing inspect logic for Iceberg tables as listed in #511. The refs metadata table in Iceberg stores the table's known snapshot references including