a-agmon commented on PR #664:
URL: https://github.com/apache/iceberg-rust/pull/664#issuecomment-2495833710
LGTM
--
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 unsub
aihuaxu commented on code in PR #10831:
URL: https://github.com/apache/iceberg/pull/10831#discussion_r1855366984
##
format/spec.md:
##
@@ -1154,6 +1169,7 @@ Maps with non-string keys must use an array
representation with the `map` logica
|**`struct`**|`record`||
|**`list`**|`
dependabot[bot] opened a new pull request, #11641:
URL: https://github.com/apache/iceberg/pull/11641
Bumps [mkdocs-material](https://github.com/squidfunk/mkdocs-material) from
9.5.44 to 9.5.45.
Release notes
Sourced from https://github.com/squidfunk/mkdocs-material/releases";>mkdoc
dependabot[bot] opened a new pull request, #11640:
URL: https://github.com/apache/iceberg/pull/11640
Bumps `testcontainers` from 1.20.3 to 1.20.4.
Updates `org.testcontainers:testcontainers` from 1.20.3 to 1.20.4
Release notes
Sourced from https://github.com/testcontainers/testco
dependabot[bot] opened a new pull request, #11638:
URL: https://github.com/apache/iceberg/pull/11638
Bumps
[com.google.errorprone:error_prone_annotations](https://github.com/google/error-prone)
from 2.35.1 to 2.36.0.
Release notes
Sourced from https://github.com/google/error-prone
dependabot[bot] opened a new pull request, #11639:
URL: https://github.com/apache/iceberg/pull/11639
Bumps software.amazon.awssdk:bom from 2.29.15 to 2.29.20.
[ -> None:
def test_assert_ref_snapshot_id(table_v2: Table) -> No
vinjai commented on code in PR #941:
URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1855299774
##
pyiceberg/table/update/__init__.py:
##
@@ -609,11 +609,14 @@ class AssertRefSnapshotId(ValidatableTableRequirement):
type: Literal["assert-ref-snapshot-id
vinjai commented on code in PR #941:
URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1855299357
##
pyiceberg/table/__init__.py:
##
@@ -1003,22 +1015,27 @@ def overwrite(
overwrite_filter: ALWAYS_TRUE when you overwrite all the data,
sungwy commented on code in PR #1304:
URL: https://github.com/apache/iceberg-python/pull/1304#discussion_r1855272561
##
pyiceberg/table/metadata.py:
##
@@ -517,12 +517,15 @@ def new_table_metadata(
location: str,
properties: Properties = EMPTY_DICT,
table_uuid: Op
kevinjqliu commented on issue #1200:
URL:
https://github.com/apache/iceberg-python/issues/1200#issuecomment-2495641285
That looks generally correct to me. There are a few caveats though. This
assumes that the entire iceberg table (metadata and data files) is in a single
location and that n
kevinjqliu commented on code in PR #1304:
URL: https://github.com/apache/iceberg-python/pull/1304#discussion_r1855257218
##
pyiceberg/table/metadata.py:
##
@@ -517,12 +517,15 @@ def new_table_metadata(
location: str,
properties: Properties = EMPTY_DICT,
table_uuid
kevinjqliu commented on code in PR #1354:
URL: https://github.com/apache/iceberg-python/pull/1354#discussion_r1855235419
##
tests/io/test_pyarrow_stats.py:
##
@@ -681,6 +685,39 @@ def test_stats_types(table_schema_nested: Schema) -> None:
]
+def test_read_missing_statis
twuebi commented on code in PR #11317:
URL: https://github.com/apache/iceberg/pull/11317#discussion_r1855212216
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/sql/TestRestDropPurgeTable.java:
##
@@ -0,0 +1,138 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
kevinjqliu commented on issue #1363:
URL:
https://github.com/apache/iceberg-python/issues/1363#issuecomment-2495556035
Thanks @JasperHG90 for the breakdown. The server spec above describes what
the server should send and what the client should expect.
According to the spec, Polaris shou
sullis closed pull request #11594: add assertions in TestRowDelta
URL: https://github.com/apache/iceberg/pull/11594
--
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 unsubscri
sullis commented on code in PR #11594:
URL: https://github.com/apache/iceberg/pull/11594#discussion_r1855223900
##
core/src/test/java/org/apache/iceberg/TestRowDelta.java:
##
@@ -74,6 +74,9 @@ public void addOnlyDeleteFilesProducesDeleteOperation() {
assertThat(snap.sequenc
twuebi commented on code in PR #11317:
URL: https://github.com/apache/iceberg/pull/11317#discussion_r1855212216
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/sql/TestRestDropPurgeTable.java:
##
@@ -0,0 +1,138 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
manuzhang commented on code in PR #11606:
URL: https://github.com/apache/iceberg/pull/11606#discussion_r1855205570
##
docs/docs/spark-procedures.md:
##
@@ -936,3 +936,40 @@ as an `UPDATE_AFTER` image, resulting in the following
pre/post update images:
|-||-
43 matches
Mail list logo