nastra commented on code in PR #11620:
URL: https://github.com/apache/iceberg/pull/11620#discussion_r1860118523
##
core/src/main/java/org/apache/iceberg/TableUtil.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor
Fokko commented on code in PR #349:
URL: https://github.com/apache/iceberg-rust/pull/349#discussion_r1860112766
##
crates/iceberg/src/transaction.rs:
##
@@ -96,6 +109,60 @@ impl<'a> Transaction<'a> {
Ok(self)
}
+fn generate_unique_snapshot_id(&self) -> i64 {
Fokko commented on code in PR #349:
URL: https://github.com/apache/iceberg-rust/pull/349#discussion_r1860102313
##
crates/iceberg/src/spec/manifest_list.rs:
##
@@ -106,34 +106,38 @@ impl std::fmt::Debug for ManifestListWriter {
impl ManifestListWriter {
/// Construct a v
Fokko opened a new pull request, #1383:
URL: https://github.com/apache/iceberg-python/pull/1383
(no comment)
--
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,
Fokko commented on code in PR #349:
URL: https://github.com/apache/iceberg-rust/pull/349#discussion_r1860074706
##
crates/iceberg/src/spec/manifest_list.rs:
##
@@ -106,34 +106,38 @@ impl std::fmt::Debug for ManifestListWriter {
impl ManifestListWriter {
/// Construct a v
Fokko commented on code in PR #349:
URL: https://github.com/apache/iceberg-rust/pull/349#discussion_r1860068128
##
crates/e2e_test/Cargo.toml:
##
@@ -0,0 +1,37 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NO
liurenjie1024 commented on code in PR #704:
URL: https://github.com/apache/iceberg-rust/pull/704#discussion_r1860049342
##
crates/iceberg/src/writer/base_writer/position_delete_file_writer.rs:
##
@@ -0,0 +1,320 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+/
Fokko commented on code in PR #710:
URL: https://github.com/apache/iceberg-rust/pull/710#discussion_r1860062199
##
crates/iceberg/src/arrow/schema.rs:
##
@@ -382,12 +382,15 @@ impl ArrowSchemaVisitor for ArrowSchemaConverter {
DataType::Time64(unit) if unit == &Time
Fokko commented on issue #1284:
URL:
https://github.com/apache/iceberg-python/issues/1284#issuecomment-2503087394
Thanks for the context yesterday, I was still noodling on it overnight.
If I understand correctly (and please also share the video of you and
@adrianqin; I must have miss
Guosmilesmile commented on PR #11662:
URL: https://github.com/apache/iceberg/pull/11662#issuecomment-2503066336
@pvary Yes, you are right. We changed our approach to handle null values
instead of filtering them out. Before serialization, we added a flag for each
field, where true indicates
nastra merged PR #11652:
URL: https://github.com/apache/iceberg/pull/11652
--
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.ap
JE-Chen commented on PR #1377:
URL: https://github.com/apache/iceberg-python/pull/1377#issuecomment-2503058477
Same error.
```
error: the configured Python interpreter version (3.13) is newer than PyO3's
maximum supported version (3.12)
```
--
This is an automated message from th
ZENOTME commented on code in PR #703:
URL: https://github.com/apache/iceberg-rust/pull/703#discussion_r1860040274
##
crates/iceberg/src/writer/base_writer/equality_delete_writer.rs:
##
@@ -0,0 +1,502 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
Fokko commented on PR #682:
URL: https://github.com/apache/iceberg-python/pull/682#issuecomment-2503053345
https://github.com/dependabot rebase
--
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
ZENOTME commented on code in PR #703:
URL: https://github.com/apache/iceberg-rust/pull/703#discussion_r1860035033
##
crates/iceberg/src/writer/base_writer/equality_delete_writer.rs:
##
@@ -0,0 +1,502 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more c
Fokko merged PR #717:
URL: https://github.com/apache/iceberg-rust/pull/717
--
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.ap
Fokko commented on PR #717:
URL: https://github.com/apache/iceberg-rust/pull/717#issuecomment-2503040380
Thanks for cleaning this up @liurenjie1024 and thanks for the quick review
@ZENOTME
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
liurenjie1024 commented on code in PR #703:
URL: https://github.com/apache/iceberg-rust/pull/703#discussion_r1860022862
##
crates/iceberg/src/arrow/record_batch_projector.rs:
##
@@ -0,0 +1,288 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contribu
ZENOTME commented on PR #717:
URL: https://github.com/apache/iceberg-rust/pull/717#issuecomment-2502997254
Thanks @liurenjie1024
--
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 comme
liurenjie1024 commented on PR #715:
URL: https://github.com/apache/iceberg-rust/pull/715#issuecomment-2502994521
> Seems this PR caused clippy fail
>
> ```
> error: use of deprecated field
`catalog::TableUpdate::AddSchema::last_column_id`: This field is handled
internally, and sho
jeesou commented on PR #11040:
URL: https://github.com/apache/iceberg/pull/11040#issuecomment-2502877154
Hi @karuppayya , @amogh-jahagirdar kindly check the comment above.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
amogh-jahagirdar commented on code in PR #11369:
URL: https://github.com/apache/iceberg/pull/11369#discussion_r1859956563
##
core/src/test/java/org/apache/iceberg/rest/responses/TestFetchScanTasksResponseParser.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software F
jeesou commented on PR #11652:
URL: https://github.com/apache/iceberg/pull/11652#issuecomment-2502859962
Hi @karuppayya @nastra can you please check this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
amogh-jahagirdar commented on code in PR #11369:
URL: https://github.com/apache/iceberg/pull/11369#discussion_r1859959392
##
core/src/test/java/org/apache/iceberg/rest/requests/TestPlanTableScanRequestParser.java:
##
@@ -0,0 +1,151 @@
+/*
+ * Licensed to the Apache Software Foun
amogh-jahagirdar commented on code in PR #11369:
URL: https://github.com/apache/iceberg/pull/11369#discussion_r1859958795
##
core/src/test/java/org/apache/iceberg/rest/responses/TestPlanTableScanResponseParser.java:
##
@@ -0,0 +1,244 @@
+/*
+ * Licensed to the Apache Software Fo
amogh-jahagirdar commented on code in PR #11369:
URL: https://github.com/apache/iceberg/pull/11369#discussion_r1859957950
##
core/src/test/java/org/apache/iceberg/rest/responses/TestFetchScanTasksResponseParser.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software F
amogh-jahagirdar commented on code in PR #11369:
URL: https://github.com/apache/iceberg/pull/11369#discussion_r1859939105
##
core/src/test/java/org/apache/iceberg/rest/responses/TestFetchScanTasksResponseParser.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software F
amogh-jahagirdar commented on code in PR #11369:
URL: https://github.com/apache/iceberg/pull/11369#discussion_r1859932083
##
core/src/test/java/org/apache/iceberg/rest/responses/TestFetchScanTasksResponseParser.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software F
Guosmilesmile commented on PR #11662:
URL: https://github.com/apache/iceberg/pull/11662#issuecomment-2502780003
@ConeyLiu yes，I have added UT to cover the relevant changes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
amogh-jahagirdar commented on code in PR #11369:
URL: https://github.com/apache/iceberg/pull/11369#discussion_r1859909899
##
core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequestParser.java:
##
@@ -0,0 +1,126 @@
+/*
+ * Licensed to the Apache Software Foundati
amogh-jahagirdar commented on code in PR #11369:
URL: https://github.com/apache/iceberg/pull/11369#discussion_r1859891750
##
core/src/main/java/org/apache/iceberg/ContentFileParser.java:
##
@@ -48,6 +48,97 @@ class ContentFileParser {
private ContentFileParser() {}
+ pub
ConeyLiu commented on PR #11662:
URL: https://github.com/apache/iceberg/pull/11662#issuecomment-2502609743
Thanks for the contributions. Should we better handle the null value instead
of skipping it? Could you also add the UT to cover these changes?
cc @stevenzwu @pvary
--
This is an
manuzhang commented on code in PR #11654:
URL: https://github.com/apache/iceberg/pull/11654#discussion_r1859767751
##
core/src/test/java/org/apache/iceberg/hadoop/TestHadoopCommits.java:
##
@@ -446,7 +446,7 @@ public void testConcurrentFastAppends(@TempDir File dir)
throws Exce
Guosmilesmile opened a new pull request, #11662:
URL: https://github.com/apache/iceberg/pull/11662
When configuring the distribution mode to RANGE, if the partition field in
the data contains null values, it will cause the SortKey serialization to fail,
resulting in the job continuously res
wypoon commented on PR #11661:
URL: https://github.com/apache/iceberg/pull/11661#issuecomment-2502493531
@nastra can you please review this?
My impetus for the refactor is that I want to implement Parquet
page-skipping, and with the refactor, a core change in logic can be applied in
one
wypoon commented on code in PR #11661:
URL: https://github.com/apache/iceberg/pull/11661#discussion_r1859480431
##
arrow/src/main/java/org/apache/iceberg/arrow/vectorized/parquet/VectorizedParquetDefinitionLevelReader.java:
##
@@ -46,122 +47,217 @@ public VectorizedParquetDefini
wypoon commented on code in PR #11661:
URL: https://github.com/apache/iceberg/pull/11661#discussion_r1859480431
##
arrow/src/main/java/org/apache/iceberg/arrow/vectorized/parquet/VectorizedParquetDefinitionLevelReader.java:
##
@@ -46,122 +47,217 @@ public VectorizedParquetDefini
aokolnychyi commented on code in PR #11657:
URL: https://github.com/apache/iceberg/pull/11657#discussion_r1859474395
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/source/DVIterable.java:
##
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
aokolnychyi commented on code in PR #11657:
URL: https://github.com/apache/iceberg/pull/11657#discussion_r1859473563
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/source/DVIterable.java:
##
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
github-actions[bot] commented on issue #10401:
URL: https://github.com/apache/iceberg/issues/10401#issuecomment-2502320652
This issue has been automatically marked as stale because it has been open
for 180 days with no activity. It will be closed in next 14 days if no further
activity occur
github-actions[bot] commented on issue #10003:
URL: https://github.com/apache/iceberg/issues/10003#issuecomment-2502320515
This issue has been automatically marked as stale because it has been open
for 180 days with no activity. It will be closed in next 14 days if no further
activity occur
aokolnychyi commented on code in PR #11620:
URL: https://github.com/apache/iceberg/pull/11620#discussion_r1859414335
##
core/src/main/java/org/apache/iceberg/SerializableTable.java:
##
@@ -143,6 +143,10 @@ protected Table newTable(TableOperations ops, String
tableName) {
r
guykhazma commented on PR #11615:
URL: https://github.com/apache/iceberg/pull/11615#issuecomment-2502221330
@huaxingao yes, it is possible to reuse the logic from the aggregate
pushdown by reusing the AggregateEvaluator instead of the current code to
aggregate from the manifests. Something
aokolnychyi commented on PR #11587:
URL: https://github.com/apache/iceberg/pull/11587#issuecomment-2502200150
I think I'd prefer #11620 for now, not a strong opinion, though.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
aokolnychyi commented on code in PR #11620:
URL: https://github.com/apache/iceberg/pull/11620#discussion_r1859414335
##
core/src/main/java/org/apache/iceberg/SerializableTable.java:
##
@@ -143,6 +143,10 @@ protected Table newTable(TableOperations ops, String
tableName) {
r
aokolnychyi commented on code in PR #11620:
URL: https://github.com/apache/iceberg/pull/11620#discussion_r1859417454
##
core/src/main/java/org/apache/iceberg/TableUtil.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contri
aokolnychyi commented on code in PR #11620:
URL: https://github.com/apache/iceberg/pull/11620#discussion_r1859414335
##
core/src/main/java/org/apache/iceberg/SerializableTable.java:
##
@@ -143,6 +143,10 @@ protected Table newTable(TableOperations ops, String
tableName) {
r
rdblue commented on code in PR #10962:
URL: https://github.com/apache/iceberg/pull/10962#discussion_r1859381182
##
core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java:
##
@@ -833,7 +833,17 @@ public List apply(TableMetadata base,
Snapshot snapshot) {
fil
rdblue commented on code in PR #10962:
URL: https://github.com/apache/iceberg/pull/10962#discussion_r1859372481
##
core/src/test/java/org/apache/iceberg/TestRewriteFiles.java:
##
@@ -384,6 +386,116 @@ public void testRewriteDataAndAssignOldSequenceNumber() {
assertThat(list
rdblue commented on code in PR #10962:
URL: https://github.com/apache/iceberg/pull/10962#discussion_r1859371399
##
core/src/test/java/org/apache/iceberg/TestRewriteFiles.java:
##
@@ -384,6 +386,116 @@ public void testRewriteDataAndAssignOldSequenceNumber() {
assertThat(list
rdblue commented on code in PR #10962:
URL: https://github.com/apache/iceberg/pull/10962#discussion_r1859362907
##
core/src/test/java/org/apache/iceberg/TestRewriteFiles.java:
##
@@ -384,6 +386,116 @@ public void testRewriteDataAndAssignOldSequenceNumber() {
assertThat(list
aokolnychyi commented on code in PR #11561:
URL: https://github.com/apache/iceberg/pull/11561#discussion_r1859282133
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/source/SparkBatchQueryScan.java:
##
@@ -168,7 +168,7 @@ protected Map rewritableDeletes() {
for (S
HonahX opened a new pull request, #11660:
URL: https://github.com/apache/iceberg/pull/11660
This PR introduces a new section, "Snapshot Summary", in the table spec
under [Snapshots](https://iceberg.apache.org/spec/#snapshots) to document
optional fields in the snapshot summary, including me
HonahX opened a new issue, #11659:
URL: https://github.com/apache/iceberg/issues/11659
### Proposed Change
The proposal introduces a new section in the table spec under
[Snapshots](https://iceberg.apache.org/spec/#snapshots) to document optional
fields in the snapshot summary, includ
dependabot[bot] opened a new pull request, #1382:
URL: https://github.com/apache/iceberg-python/pull/1382
Bumps [pydantic](https://github.com/pydantic/pydantic) from 2.10.1 to 2.10.2.
Release notes
Sourced from https://github.com/pydantic/pydantic/releases";>pydantic's
releases.
dependabot[bot] opened a new pull request, #1381:
URL: https://github.com/apache/iceberg-python/pull/1381
Bumps [pyarrow](https://github.com/apache/arrow) from 18.0.0 to 18.1.0.
Release notes
Sourced from https://github.com/apache/arrow/releases";>pyarrow's releases.
Apache A
stevenzwu commented on code in PR #11301:
URL: https://github.com/apache/iceberg/pull/11301#discussion_r1859281089
##
flink/v1.20/flink/src/test/java/org/apache/iceberg/flink/source/TestIcebergSourceBoundedRow.java:
##
@@ -0,0 +1,96 @@
+/*
+ * Licensed to the Apache Software Fou
rdblue commented on code in PR #10962:
URL: https://github.com/apache/iceberg/pull/10962#discussion_r1859261982
##
core/src/test/java/org/apache/iceberg/TestBase.java:
##
@@ -108,6 +108,14 @@ public class TestBase {
.withPartitionPath("data_bucket=1") // easy way to s
rdblue commented on code in PR #10962:
URL: https://github.com/apache/iceberg/pull/10962#discussion_r1859257408
##
core/src/test/java/org/apache/iceberg/TestRewriteFiles.java:
##
@@ -384,6 +386,116 @@ public void testRewriteDataAndAssignOldSequenceNumber() {
assertThat(list
rdblue commented on code in PR #10962:
URL: https://github.com/apache/iceberg/pull/10962#discussion_r1859256224
##
core/src/test/java/org/apache/iceberg/TestRewriteFiles.java:
##
@@ -384,6 +386,116 @@ public void testRewriteDataAndAssignOldSequenceNumber() {
assertThat(list
rdblue commented on code in PR #10962:
URL: https://github.com/apache/iceberg/pull/10962#discussion_r1859255414
##
core/src/test/java/org/apache/iceberg/TestRewriteFiles.java:
##
@@ -384,6 +386,116 @@ public void testRewriteDataAndAssignOldSequenceNumber() {
assertThat(list
Fokko commented on issue #1223:
URL:
https://github.com/apache/iceberg-python/issues/1223#issuecomment-2501946144
@tusharchou Thanks. I was noodling on this, and instead of having a
`.to_arrow()`, we could also have a `.count()` that will return the number of
rows that match the predicate.
kevinjqliu commented on PR #1380:
URL: https://github.com/apache/iceberg-python/pull/1380#issuecomment-2501935264
👍
--
This is an automated message from the Apac
kevinjqliu merged PR #1380:
URL: https://github.com/apache/iceberg-python/pull/1380
--
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...@i
kevinjqliu merged PR #1371:
URL: https://github.com/apache/iceberg-python/pull/1371
--
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...@i
pedorro closed issue #11650: Renamed column returns null values from 'appended'
Parquet file not originally created by Iceberg
URL: https://github.com/apache/iceberg/issues/11650
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
pedorro commented on issue #11650:
URL: https://github.com/apache/iceberg/issues/11650#issuecomment-2501897179
Closing this ticket as apparently not an issue, but rather
ignorant-user-error.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
Fokko opened a new pull request, #1380:
URL: https://github.com/apache/iceberg-python/pull/1380
I like this part from the Iceberg pull-request 👍
--
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 t
Fokko commented on PR #1377:
URL: https://github.com/apache/iceberg-python/pull/1377#issuecomment-2501858847
@JE-Chen Thanks for checking. It looks like Py3O is already at 3.13:
https://github.com/PyO3/pyo3/issues/4636 I was hoping that maybe bumping the
Poetry build might fix it https://gi
Fokko merged PR #1379:
URL: https://github.com/apache/iceberg-python/pull/1379
--
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...@iceber
Fokko commented on PR #1371:
URL: https://github.com/apache/iceberg-python/pull/1371#issuecomment-2501851589
@kevinjqliu Ah I see, I don't think that we have _fully_ reproducible
builds. For example, if a timestamp in some wheel is different, then it would
yield another hash. I think it is
Fokko commented on issue #1366:
URL:
https://github.com/apache/iceberg-python/issues/1366#issuecomment-2501843565
Here's the logic:
https://github.com/apache/iceberg-python/blob/1e9bdc21d95d2702f0777b21c61409262d1e3052/pyiceberg/table/update/snapshot.py#L271-L279
We always set
pedorro commented on issue #11650:
URL: https://github.com/apache/iceberg/issues/11650#issuecomment-2501840264
Well dad-gum! Setting that table property appears to have resolved it. I'm
quite new to Iceberg in general, and so was unaware of this property. I really
appreciate you taking a m
Fokko commented on issue #1366:
URL:
https://github.com/apache/iceberg-python/issues/1366#issuecomment-2501830533
As a side note, we don't have a proper retry strategy in Iceberg yet. To
appends should not cause a conflict right away because they don't interfere.
Instead what should happen
jbonofre commented on PR #11658:
URL: https://github.com/apache/iceberg/pull/11658#issuecomment-2501796971
@pvary I started some cleanups in Flink tests.
--
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
jbonofre opened a new pull request, #11658:
URL: https://github.com/apache/iceberg/pull/11658
This PR replaces use of deprecated method in Flink (1.20) tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
tusharchou commented on issue #1223:
URL:
https://github.com/apache/iceberg-python/issues/1223#issuecomment-2501709107
# RCA
Hi @Visorgood,
The behavior expected here is a simple partition push-down implementation in
duck db which this pr solves for-
https://github.com/duckdb/duckd
kevinjqliu commented on code in PR #11597:
URL: https://github.com/apache/iceberg/pull/11597#discussion_r1859082300
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -412,6 +412,28 @@ private void validateTableIsIcebergTableOrView(
}
}
+
kevinjqliu commented on code in PR #11597:
URL: https://github.com/apache/iceberg/pull/11597#discussion_r1859082627
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -412,6 +412,28 @@ private void validateTableIsIcebergTableOrView(
}
}
+
kevinjqliu commented on issue #851:
URL: https://github.com/apache/iceberg-python/issues/851#issuecomment-2501696638
@willcollins10 assigned to you
--
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
kevinjqliu commented on PR #1371:
URL: https://github.com/apache/iceberg-python/pull/1371#issuecomment-2501640410
I was comparing between running the action on
[main](https://github.com/kevinjqliu/iceberg-python/actions/runs/12036359688)
versus [this
branch](https://github.com/kevinjqliu/i
kevinjqliu commented on issue #1366:
URL:
https://github.com/apache/iceberg-python/issues/1366#issuecomment-2501614001
@HaraldVanWoerkom if you have a reproducible process, can you check if this
occurs with another catalog? For example, this one that we use for integration
tests
https:
emkornfield commented on PR #9502:
URL: https://github.com/apache/iceberg/pull/9502#issuecomment-2501606427
Just wanted to ask does this only apply to Avro or does adding it to
AvroGenericReader also cover other file types ORC and Parquet?
--
This is an automated message from the Apache G
emkornfield commented on code in PR #10831:
URL: https://github.com/apache/iceberg/pull/10831#discussion_r1858978402
##
format/spec.md:
##
@@ -182,6 +182,21 @@ A **`list`** is a collection of values with some element
type. The element field
A **`map`** is a collection of key
emkornfield commented on code in PR #10831:
URL: https://github.com/apache/iceberg/pull/10831#discussion_r1858976684
##
format/spec.md:
##
@@ -178,6 +178,21 @@ A **`list`** is a collection of values with some element
type. The element field
A **`map`** is a collection of key
kevinjqliu commented on issue #1378:
URL:
https://github.com/apache/iceberg-python/issues/1378#issuecomment-2501517247
hi @dongsupkim-onepredict thanks for raising this issue. can you also
include the python code that cause this error?
--
This is an automated message from the Apache Git
singhpk234 commented on issue #11648:
URL: https://github.com/apache/iceberg/issues/11648#issuecomment-2501506598
@davseitsev can you please also try setting to true ?
```
spark.sql.iceberg.executor-cache.locality.enabled
```
please ref : https://github.com/apache/iceberg/pu
kevinjqliu commented on PR #1369:
URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2501494494
thanks @fokko cherry-picked #1373 as well
--
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
singhpk234 commented on issue #11650:
URL: https://github.com/apache/iceberg/issues/11650#issuecomment-2501447829
This is an interesting observation can you please help with the following :
[1] is this prop being set : schema.name-mapping.default in tbl properties
if not can you set it ?
nastra commented on code in PR #11654:
URL: https://github.com/apache/iceberg/pull/11654#discussion_r1858896301
##
core/src/test/java/org/apache/iceberg/hadoop/TestHadoopCommits.java:
##
@@ -446,7 +446,7 @@ public void testConcurrentFastAppends(@TempDir File dir)
throws Excepti
ajantha-bhat commented on issue #11649:
URL: https://github.com/apache/iceberg/issues/11649#issuecomment-2501250605
Duplicate of https://github.com/apache/iceberg/issues/11655
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
ajantha-bhat closed issue #11649: How an application can communicate with the
Iceberg REST catalog like Dremio Arctic (Nessie), Snowflake’s Polaris ,
Gravitino
URL: https://github.com/apache/iceberg/issues/11649
--
This is an automated message from the Apache Git Service.
To respond to the m
ajantha-bhat commented on issue #11655:
URL: https://github.com/apache/iceberg/issues/11655#issuecomment-2501247204
Since you are using spark engine, spark session can be configured to use
Iceberg catalogs.
Just make sure catalog type is `rest`.
https://iceberg.apache.org/docs/nightl
ajantha-bhat closed issue #11655: Interaction with the Iceberg REST catalog
like Dremio Arctic (Nessie), Snowflake’s Polaris , Gravitino
URL: https://github.com/apache/iceberg/issues/11655
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
pvary commented on PR #11430:
URL: https://github.com/apache/iceberg/pull/11430#issuecomment-2501235249
Thanks @jbonofre for the PR and @nastra for the review.
@jbonofre: Please port the changes to Flink 1.19/1.18 too.
Thanks,
Peter
--
This is an automated message from the Apa
pvary merged PR #11430:
URL: https://github.com/apache/iceberg/pull/11430
--
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.apa
Fokko commented on PR #11656:
URL: https://github.com/apache/iceberg/pull/11656#issuecomment-2501216005
Seems that JDK22 is not yet supported :(
--
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 th
Fokko commented on PR #11656:
URL: https://github.com/apache/iceberg/pull/11656#issuecomment-2501152732
Yes, looks like we need to drop JDK 11 first before we can upgrade to the
newer baseline version:
https://github.com/palantir/gradle-baseline/releases/tag/6.0.0
--
This is an automated
Fokko commented on PR #11656:
URL: https://github.com/apache/iceberg/pull/11656#issuecomment-2501142678
@jbonofre It looks like the palantir baseline plugin doesn't know how to
handle Java 22 multi-source packages. Let me take a look first.
--
This is an automated message from the Apache
jbonofre commented on PR #11656:
URL: https://github.com/apache/iceberg/pull/11656#issuecomment-2501078210
@Fokko it seems that jackson is shaded two times with Parquet 1.15. Do you
want me to investigate ?
--
This is an automated message from the Apache Git Service.
To respond to the mes
1 - 100 of 145 matches
Mail list logo
