Fokko commented on PR #1869:
URL: https://github.com/apache/iceberg-python/pull/1869#issuecomment-2777694368
@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 specific commen
zratkai commented on code in PR #12461:
URL: https://github.com/apache/iceberg/pull/12461#discussion_r2028197038
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HMSTablePropertyHelper.java:
##
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
zratkai commented on code in PR #12461:
URL: https://github.com/apache/iceberg/pull/12461#discussion_r2028193995
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HMSTablePropertyHelper.java:
##
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
Fokko merged PR #1881:
URL: https://github.com/apache/iceberg-python/pull/1881
--
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 code in PR #12644:
URL: https://github.com/apache/iceberg/pull/12644#discussion_r2028181000
##
format/spec.md:
##
@@ -1453,13 +1457,15 @@ Each sort field in the fields list is stored as an
object with the following pro
| V1 | V2 | V3 | Fi
nastra commented on PR #12720:
URL: https://github.com/apache/iceberg/pull/12720#issuecomment-2777678813
merging this, since test failures are unrelated
--
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
nastra merged PR #12720:
URL: https://github.com/apache/iceberg/pull/12720
--
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
wypoon commented on PR #12681:
URL: https://github.com/apache/iceberg/pull/12681#issuecomment-2777665104
@pvary see https://github.com/apache/iceberg/pull/12721.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
wypoon opened a new pull request, #12721:
URL: https://github.com/apache/iceberg/pull/12721
... where we attempt to have a single version of TestHiveMetastore.java.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
wypoon commented on PR #12681:
URL: https://github.com/apache/iceberg/pull/12681#issuecomment-2777659880
@pvary I replied on the dev list.
> I really would like to see HiveCatalog testing against Hive4, but Spark
has constraints around testing newer Hive versions which we need to solv
amitgilad3 commented on code in PR #12257:
URL: https://github.com/apache/iceberg/pull/12257#discussion_r2028156549
##
core/src/main/java/org/apache/iceberg/actions/RewriteDataFilesCommitManager.java:
##
@@ -51,7 +53,12 @@ public RewriteDataFilesCommitManager(Table table, long
nastra opened a new pull request, #12720:
URL: https://github.com/apache/iceberg/pull/12720
(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, e-m
nastra commented on code in PR #12671:
URL: https://github.com/apache/iceberg/pull/12671#discussion_r2028127620
##
build.gradle:
##
@@ -530,6 +530,7 @@ project(':iceberg-aws') {
recommend.set(true)
}
check.dependsOn('validateS3SignerSpec')
+ check.dependsOn('integrat
Souldiv opened a new issue, #12719:
URL: https://github.com/apache/iceberg/issues/12719
### Query engine
trino
### Question
I have been playing around with kafka connect and iceberg with hms as
catalog. I followed the 1.8.1 iceberg kafka-connect documentation with the
s
lliangyu-lin opened a new pull request, #12718:
URL: https://github.com/apache/iceberg/pull/12718
### Description
* Fix integration test failures in aws dynamodb and glue
* Some fixes are directly copied from this [closed
PR](https://github.com/apache/iceberg/pull/7234) from @jackye199
ForeverAngry commented on code in PR #1880:
URL: https://github.com/apache/iceberg-python/pull/1880#discussion_r2027638423
##
pyproject.toml:
##
@@ -83,6 +83,7 @@ cachetools = "^5.5.0"
pyiceberg-core = { version = "^0.4.0", optional = true }
polars = { version = "^1.21.0", opt
wypoon commented on code in PR #12461:
URL: https://github.com/apache/iceberg/pull/12461#discussion_r2027937054
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveTableOperations.java:
##
@@ -230,12 +190,14 @@ protected void doCommit(TableMetadata base, TableMetadata
m
jdockerty commented on code in PR #1116:
URL: https://github.com/apache/iceberg-rust/pull/1116#discussion_r2025155007
##
crates/iceberg/src/spec/name_mapping.rs:
##
@@ -33,12 +48,38 @@ pub struct NameMapping {
#[serde(rename_all = "kebab-case")]
pub struct MappedField {
#
gh-yzou commented on code in PR #12655:
URL: https://github.com/apache/iceberg/pull/12655#discussion_r2027459032
##
core/src/main/java/org/apache/iceberg/rest/RESTCatalog.java:
##
@@ -65,7 +68,14 @@ public RESTCatalog(Function, RESTClient>
clientBuilder) {
public RESTCatalog
Jordano-Dremio opened a new pull request, #12716:
URL: https://github.com/apache/iceberg/pull/12716
The InputFile Interface/Object is often used directly in error messages.
[Example](https://github.com/apache/iceberg/blob/main/core/src/main/java/org/apache/iceberg/avro/AvroIterable.java#L104
jackylee-ch commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2027914368
##
docs/docs/hive.md:
##
@@ -126,9 +90,6 @@ To enable Hive support globally for an application, set
`iceberg.engine.hive.ena
For example, setting this in the `h
jackylee-ch commented on PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#issuecomment-2777263902
Done. I have checked the document again and didn't find any other problems.
Please take another look at it.
--
This is an automated message from the Apache Git Service.
To respond
lliangyu-lin commented on code in PR #12718:
URL: https://github.com/apache/iceberg/pull/12718#discussion_r2027881602
##
aws/src/integration/java/org/apache/iceberg/aws/glue/TestGlueCatalogTable.java:
##
@@ -333,28 +332,33 @@ public void testRenameTableFailsToCreateNewTable() {
b-rick opened a new pull request, #1881:
URL: https://github.com/apache/iceberg-python/pull/1881
When using pydantic>=2.11.0, we get an error when creating bucket
transforms. In this version, it's illegal to access self before calling super.
To fix this, we just need to ensure we call `supe
lliangyu-lin commented on code in PR #12718:
URL: https://github.com/apache/iceberg/pull/12718#discussion_r2027881602
##
aws/src/integration/java/org/apache/iceberg/aws/glue/TestGlueCatalogTable.java:
##
@@ -333,28 +332,33 @@ public void testRenameTableFailsToCreateNewTable() {
arnaudbriche commented on issue #367:
URL: https://github.com/apache/iceberg-go/issues/367#issuecomment-2776562353
After digging a bit deeper, I noticed that most of the features I need have
already been implemented.
I can see this in the code:
```go
MetadataDeleteAfterCommitEna
RussellSpitzer commented on issue #12717:
URL: https://github.com/apache/iceberg/issues/12717#issuecomment-2777194037
I'm removing the Specification tag, because this is a Reference IMPL change
and shouldn't effect the spec
--
This is an automated message from the Apache Git Service.
To r
RussellSpitzer commented on issue #12717:
URL: https://github.com/apache/iceberg/issues/12717#issuecomment-2777192668
Thanks @sn2479 . I think this is a reasonable thing to do, I believe our
original implementation is just set up that way because of Spark's limitations
but I think widening
Fokko commented on PR #1747:
URL: https://github.com/apache/iceberg-python/pull/1747#issuecomment-2776740059
Gentle ping @kevinjqliu. Any thoughts on the `cached_property` suggested by
@hussein-awala?
--
This is an automated message from the Apache Git Service.
To respond to the message,
amogh-jahagirdar commented on PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#issuecomment-2777098589
Thanks @ricardopereira33 and thanks @nastra for reviewing!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
abstractdog commented on code in PR #12427:
URL: https://github.com/apache/iceberg/pull/12427#discussion_r2026705407
##
core/src/main/java/org/apache/iceberg/RemoveSnapshots.java:
##
@@ -153,6 +153,13 @@ public ExpireSnapshots planWith(ExecutorService
executorService) {
re
amogh-jahagirdar merged PR #12580:
URL: https://github.com/apache/iceberg/pull/12580
--
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...@
dttung2905 opened a new pull request, #376:
URL: https://github.com/apache/iceberg-go/pull/376
(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,
sn2479 opened a new issue, #12717:
URL: https://github.com/apache/iceberg/issues/12717
### Proposed Change
Some engines are able to not only produce new data file during row-level
change to a table, but can also eliminate entire data files files in the same
pending update. Alternativ
amogh-jahagirdar commented on PR #12580:
URL: https://github.com/apache/iceberg/pull/12580#issuecomment-2777032298
Thanks @danielcweeks , and @RussellSpitzer @rdblue @aokolnychyi for
reviewing, and [all others in the
community](https://lists.apache.org/thread/61w6lj3tmnyv1gtvmz08twbp6lb8nzs
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2026825325
##
core/src/main/java/org/apache/iceberg/MetadataUpdateParser.java:
##
@@ -417,11 +426,9 @@ private static void
writeAddSnapshot(MetadataUpdate.AddSnapshot update, Js
ricardopereira33 commented on PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#issuecomment-2776402107
@nastra @amogh-jahagirdar Pushed the change you requested. Thank you guys
for the fast reviews and feedback! 💪🏽
--
This is an automated message from the Apache Git Service.
relentless-leader commented on issue #7272:
URL: https://github.com/apache/iceberg/issues/7272#issuecomment-2776801561
Latest PR tracked by: [#12257 ](https://github.com/apache/iceberg/pull/12257)
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
relentless-leader commented on PR #12257:
URL: https://github.com/apache/iceberg/pull/12257#issuecomment-2776797664
Are we good to merge this PR ? Is any work pending on this ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
ForeverAngry commented on code in PR #1880:
URL: https://github.com/apache/iceberg-python/pull/1880#discussion_r2027645539
##
pyiceberg/table/update/snapshot.py:
##
@@ -745,6 +763,8 @@ class
ManageSnapshots(UpdateTableMetadata["ManageSnapshots"]):
def _commit(self) -> Up
lliangyu-lin commented on code in PR #12671:
URL: https://github.com/apache/iceberg/pull/12671#discussion_r2027343151
##
aws/src/integration/java/org/apache/iceberg/aws/TestAssumeRoleAwsClientFactory.java:
##
@@ -47,6 +48,16 @@
import software.amazon.awssdk.services.iam.model.P
ForeverAngry commented on code in PR #1880:
URL: https://github.com/apache/iceberg-python/pull/1880#discussion_r2027643598
##
pyiceberg/table/update/snapshot.py:
##
@@ -239,7 +257,7 @@ def _summary(self, snapshot_properties: Dict[str, str] =
EMPTY_DICT) -> Summary:
Fokko commented on code in PR #1880:
URL: https://github.com/apache/iceberg-python/pull/1880#discussion_r2027605325
##
pyiceberg/table/update/snapshot.py:
##
@@ -745,6 +763,8 @@ class
ManageSnapshots(UpdateTableMetadata["ManageSnapshots"]):
def _commit(self) -> UpdatesAn
pvary commented on PR #12713:
URL: https://github.com/apache/iceberg/pull/12713#issuecomment-2776533157
@Guosmilesmile: Please keep separate PRs for every backport. It is easier on
developers maintaining their own release.
Could you please create 2 different PRs?
Thanks,
Peter
pvary commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2027452101
##
docs/docs/hive.md:
##
@@ -602,7 +567,6 @@ Here are the features highlights for Iceberg Hive read
support:
1. **Predicate pushdown**: Pushdown of the Hive SQL `WHER
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2026838728
##
core/src/main/java/org/apache/iceberg/MetadataUpdateParser.java:
##
@@ -150,7 +150,8 @@ private MetadataUpdateParser() {}
.put(MetadataUpdate.SetPartitio
Fokko commented on code in PR #1879:
URL: https://github.com/apache/iceberg-python/pull/1879#discussion_r2027379456
##
pyiceberg/table/update/snapshot.py:
##
@@ -236,7 +236,6 @@ def _summary(self, snapshot_properties: Dict[str, str] =
EMPTY_DICT) -> Summary:
return upd
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2026831937
##
core/src/main/java/org/apache/iceberg/MetadataUpdateParser.java:
##
@@ -557,11 +564,18 @@ private static MetadataUpdate readAddSnapshot(JsonNode
node) {
private
lliangyu-lin commented on code in PR #12671:
URL: https://github.com/apache/iceberg/pull/12671#discussion_r2027542721
##
aws/src/integration/java/org/apache/iceberg/aws/TestAssumeRoleAwsClientFactory.java:
##
@@ -47,6 +49,14 @@
import software.amazon.awssdk.services.iam.model.P
lliangyu-lin opened a new issue, #12715:
URL: https://github.com/apache/iceberg/issues/12715
### Feature Request / Improvement
The current [AWS integration
tests](https://github.com/apache/iceberg/tree/main/aws/src/integration/java/org/apache/iceberg/aws)
are not well maintained beca
Fokko commented on code in PR #1879:
URL: https://github.com/apache/iceberg-python/pull/1879#discussion_r2027521950
##
pyiceberg/table/update/snapshot.py:
##
@@ -236,7 +236,6 @@ def _summary(self, snapshot_properties: Dict[str, str] =
EMPTY_DICT) -> Summary:
return upd
Fokko commented on code in PR #1879:
URL: https://github.com/apache/iceberg-python/pull/1879#discussion_r2027518680
##
pyiceberg/table/update/snapshot.py:
##
@@ -236,7 +236,6 @@ def _summary(self, snapshot_properties: Dict[str, str] =
EMPTY_DICT) -> Summary:
return upd
Fokko commented on code in PR #1879:
URL: https://github.com/apache/iceberg-python/pull/1879#discussion_r2027518680
##
pyiceberg/table/update/snapshot.py:
##
@@ -236,7 +236,6 @@ def _summary(self, snapshot_properties: Dict[str, str] =
EMPTY_DICT) -> Summary:
return upd
Fokko commented on PR #1840:
URL: https://github.com/apache/iceberg-python/pull/1840#issuecomment-2775312549
Ahh thanks @kevinjqliu I don't think we've really implemented this properly.
Ja
danielcweeks commented on PR #12555:
URL: https://github.com/apache/iceberg/pull/12555#issuecomment-2776494155
I'm more than a little hesitant to start changing this now. Pivoting the
way we think about sessions/scopes doesn't really add a lot at this point and
we've just completed a major
pvary commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2027454952
##
docs/docs/hive.md:
##
@@ -26,28 +26,18 @@ a
[StorageHandler](https://cwiki.apache.org/confluence/display/Hive/StorageHandl
## Feature support
The following featur
pvary commented on code in PR #12461:
URL: https://github.com/apache/iceberg/pull/12461#discussion_r2027439420
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HMSTablePropertyHelper.java:
##
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
rdblue commented on code in PR #12644:
URL: https://github.com/apache/iceberg/pull/12644#discussion_r2027429855
##
format/spec.md:
##
@@ -1414,12 +1414,16 @@ Each partition field in `fields` is stored as a JSON
object with the following p
| V1 | V2 | V3 | F
rdblue commented on code in PR #12644:
URL: https://github.com/apache/iceberg/pull/12644#discussion_r2027415792
##
format/spec.md:
##
@@ -1453,13 +1457,15 @@ Each sort field in the fields list is stored as an
object with the following pro
| V1 | V2 | V3 | F
kevinjqliu commented on code in PR #1879:
URL: https://github.com/apache/iceberg-python/pull/1879#discussion_r2027386623
##
tests/integration/test_writes/test_writes.py:
##
@@ -262,6 +262,100 @@ def test_summaries(spark: SparkSession, session_catalog:
Catalog, arrow_table_wi
jackylee-ch opened a new pull request, #12712:
URL: https://github.com/apache/iceberg/pull/12712
This PR introduces configurable HMSClient customization to address specific
use cases where enterprises require custom Hive Metastore client
implementations.
--
This is an automated message f
xiaoxuandev commented on code in PR #12671:
URL: https://github.com/apache/iceberg/pull/12671#discussion_r2027383951
##
aws/src/integration/java/org/apache/iceberg/aws/TestAssumeRoleAwsClientFactory.java:
##
@@ -47,6 +49,14 @@
import software.amazon.awssdk.services.iam.model.Pu
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2026824248
##
core/src/main/java/org/apache/iceberg/MetadataUpdateParser.java:
##
@@ -229,7 +230,15 @@ public static void toJson(MetadataUpdate metadataUpdate,
JsonGenerator gen
danielcweeks commented on PR #12658:
URL: https://github.com/apache/iceberg/pull/12658#issuecomment-2776344693
@aihuaxu and @rdblue is there a reason we need to explicitly restrict the
lower/upper bounds to shredded fields? I would think that the stats pruning
would be useful for any field
kevinjqliu commented on code in PR #1879:
URL: https://github.com/apache/iceberg-python/pull/1879#discussion_r2027360226
##
tests/integration/test_writes/test_writes.py:
##
@@ -262,6 +262,100 @@ def test_summaries(spark: SparkSession, session_catalog:
Catalog, arrow_table_wi
manuzhang commented on PR #12707:
URL: https://github.com/apache/iceberg/pull/12707#issuecomment-2776330349
Let me try with https://github.com/apache/iceberg/pull/12714
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
lliangyu-lin commented on code in PR #12671:
URL: https://github.com/apache/iceberg/pull/12671#discussion_r2027343151
##
aws/src/integration/java/org/apache/iceberg/aws/TestAssumeRoleAwsClientFactory.java:
##
@@ -47,6 +48,16 @@
import software.amazon.awssdk.services.iam.model.P
ajantha-bhat commented on PR #12655:
URL: https://github.com/apache/iceberg/pull/12655#issuecomment-2776096706
Moving this out of 1.9.0 milestone as of now as we don't have conclusion on
this and this is not really a release blocker.
--
This is an automated message from the Apache Git Se
Guosmilesmile commented on PR #12691:
URL: https://github.com/apache/iceberg/pull/12691#issuecomment-2776038624
@pvary As you mentioned, some configs are about to expire, and some are
mistakes. I’ve summarized all the configs:
**Source:**
table.exec.iceberg.infer-source-paral
Guosmilesmile commented on PR #12713:
URL: https://github.com/apache/iceberg/pull/12713#issuecomment-2776042921
@pvary This is a clearn backport. Please take some time to review it ,
Thanks !
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2027131882
##
core/src/main/java/org/apache/iceberg/MetadataUpdate.java:
##
@@ -328,6 +328,11 @@ public void applyTo(TableMetadata.Builder metadataBuilder)
{
}
}
+ /*
Guosmilesmile commented on PR #12703:
URL: https://github.com/apache/iceberg/pull/12703#issuecomment-2775797791
@pvary The array made by this method is used in SketchUtil.partition to
find a SortKey in which part. So, I think a shorter array in this case is ok .
https://github.com/
zratkai commented on PR #12461:
URL: https://github.com/apache/iceberg/pull/12461#issuecomment-2775788409
@pvary thanks for the reivew. Fixed your comments.
--
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
zratkai commented on code in PR #12461:
URL: https://github.com/apache/iceberg/pull/12461#discussion_r2026993865
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveViewOperations.java:
##
@@ -175,11 +172,14 @@ public void doCommit(ViewMetadata base, ViewMetadata
metada
zratkai commented on code in PR #12461:
URL: https://github.com/apache/iceberg/pull/12461#discussion_r2026993015
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HMSTablePropertyHelper.java:
##
@@ -0,0 +1,261 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
deniskuzZ commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2026970779
##
docs/docs/hive.md:
##
@@ -145,17 +130,14 @@ The table level configuration overrides the global Hadoop
configuration.
# Hive on Tez configuration
-To us
deniskuzZ commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2026952361
##
docs/docs/hive.md:
##
@@ -145,17 +130,14 @@ The table level configuration overrides the global Hadoop
configuration.
# Hive on Tez configuration
-To us
deniskuzZ commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2026959066
##
docs/docs/hive.md:
##
@@ -89,29 +89,14 @@ Hive supports the following additional features with Hive
version 4.0.0 and abov
## Enabling Iceberg support in Hiv
pvary merged PR #12080:
URL: https://github.com/apache/iceberg/pull/12080
--
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
pvary commented on PR #12703:
URL: https://github.com/apache/iceberg/pull/12703#issuecomment-2775654775
@Guosmilesmile: Based on my understanding, if we fix the issue in the way
you proposed then we will have a shorter array than expected. This might cause
issues later when some algorithm e
pvary commented on code in PR #12427:
URL: https://github.com/apache/iceberg/pull/12427#discussion_r2026883731
##
core/src/main/java/org/apache/iceberg/SnapshotProducer.java:
##
@@ -197,7 +198,7 @@ protected String targetBranch() {
}
protected ExecutorService workerPool(
pvary commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2026879916
##
docs/docs/hive.md:
##
@@ -213,7 +195,7 @@ SET iceberg.catalog.glue.lock.table=myGlueLockTable;
## DDL Commands
-Not all the features below are supported with Hi
pvary commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2026876312
##
docs/docs/hive.md:
##
@@ -145,17 +130,14 @@ The table level configuration overrides the global Hadoop
configuration.
# Hive on Tez configuration
-To use th
pvary commented on code in PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#discussion_r2026877554
##
docs/docs/hive.md:
##
@@ -145,17 +130,14 @@ The table level configuration overrides the global Hadoop
configuration.
# Hive on Tez configuration
-To use th
pvary commented on PR #12707:
URL: https://github.com/apache/iceberg/pull/12707#issuecomment-2775527627
@manuzhang: Is it flaky because the waiting time is too short? Shall we just
decrease the number of threads, or commits, or increase the timeout?
I usually prefer to avoid retries a
Guosmilesmile commented on PR #12703:
URL: https://github.com/apache/iceberg/pull/12703#issuecomment-2775513562
@pvary Yes, I just removed the null values from the array, and the order of
the array stayed the same. This modification did not change the original
behavior of the method.
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2026833933
##
core/src/test/java/org/apache/iceberg/TestMetadataUpdateParser.java:
##
@@ -1018,9 +1041,16 @@ public void assertEquals(
(MetadataUpdate.AddSnapshot) e
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2026832528
##
core/src/main/java/org/apache/iceberg/TableMetadata.java:
##
@@ -1864,8 +1870,13 @@ private static List updateSnapshotLog(
List changes) {
Set inter
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2026828023
##
core/src/main/java/org/apache/iceberg/MetadataUpdateParser.java:
##
@@ -557,11 +564,18 @@ private static MetadataUpdate readAddSnapshot(JsonNode
node) {
private
drnta opened a new issue, #12711:
URL: https://github.com/apache/iceberg/issues/12711
### Query engine
_No response_
### Question
Hi everyone.
It seems that in
https://github.com/apache/iceberg/blob/main/open-api/rest-catalog-open-api.yaml
the `ETag` header is re
nastra commented on code in PR #12670:
URL: https://github.com/apache/iceberg/pull/12670#discussion_r2026821208
##
core/src/main/java/org/apache/iceberg/MetadataUpdate.java:
##
@@ -328,6 +328,11 @@ public void applyTo(TableMetadata.Builder metadataBuilder)
{
}
}
+ /*
Fokko commented on PR #12644:
URL: https://github.com/apache/iceberg/pull/12644#issuecomment-2775432019
@szehon-ho I think that's a good idea. Let me rework the PR 👍
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
dolcino-li commented on code in PR #12706:
URL: https://github.com/apache/iceberg/pull/12706#discussion_r2026070423
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestReplaceWhere.java:
##
@@ -0,0 +1,60 @@
+/*
+ * Licensed to the Apache Software
Fokko opened a new pull request, #1879:
URL: https://github.com/apache/iceberg-python/pull/1879
# Rationale for this change
@kevinjqliu PTAL. I took the liberty of providing a fix for this since I was
curious where this was coming from, hope you don't mind! I've cherry-picked
your co
pvary commented on PR #12703:
URL: https://github.com/apache/iceberg/pull/12703#issuecomment-2775191541
@Guosmilesmile: Did we check that the sorter array will not cause later
issues when the statistics is used?
--
This is an automated message from the Apache Git Service.
To respond to th
pvary commented on PR #11826:
URL: https://github.com/apache/iceberg/pull/11826#issuecomment-2775170197
Thanks @Guosmilesmile!
Please create the backport 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 ab
jackylee-ch commented on PR #12700:
URL: https://github.com/apache/iceberg/pull/12700#issuecomment-2774933815
@manuzhang @pvary I have updated the PR. PTAL, thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
pvary commented on code in PR #12691:
URL: https://github.com/apache/iceberg/pull/12691#discussion_r2026600147
##
docs/docs/flink-configuration.md:
##
@@ -198,4 +198,42 @@ they are.
This is only applicable to {@link StatisticsType#Map} for low-cardinality
scenario. For
{@link
pvary commented on code in PR #12691:
URL: https://github.com/apache/iceberg/pull/12691#discussion_r2026602432
##
docs/docs/flink-configuration.md:
##
@@ -198,4 +198,42 @@ they are.
This is only applicable to {@link StatisticsType#Map} for low-cardinality
scenario. For
{@link
pvary commented on code in PR #12691:
URL: https://github.com/apache/iceberg/pull/12691#discussion_r2026604541
##
docs/docs/flink-configuration.md:
##
@@ -198,4 +198,42 @@ they are.
This is only applicable to {@link StatisticsType#Map} for low-cardinality
scenario. For
{@link
1 - 100 of 131 matches
Mail list logo
