nastra commented on code in PR #13222:
URL: https://github.com/apache/iceberg/pull/13222#discussion_r2184567240
##
core/src/main/java/org/apache/iceberg/ManifestFilterManager.java:
##
@@ -224,7 +235,9 @@ List filterManifests(Schema tableSchema,
List manife
private boolean ca
nastra commented on code in PR #13222:
URL: https://github.com/apache/iceberg/pull/13222#discussion_r2184551512
##
core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java:
##
@@ -1130,6 +1132,11 @@ protected ManifestReader
newManifestReader(ManifestFile manifest) {
nastra merged PR #13463:
URL: https://github.com/apache/iceberg/pull/13463
--
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
nastra commented on code in PR #12264:
URL: https://github.com/apache/iceberg/pull/12264#discussion_r2184495084
##
aws/src/main/java/org/apache/iceberg/aws/s3/DefaultS3FileIOAwsClientFactory.java:
##
@@ -46,6 +46,7 @@ public void initialize(Map properties) {
public S3Client s
nastra commented on code in PR #13425:
URL: https://github.com/apache/iceberg/pull/13425#discussion_r2184490432
##
core/src/main/java/org/apache/iceberg/PartitionStats.java:
##
@@ -109,7 +114,12 @@ public void liveEntry(ContentFile file, Snapshot
snapshot) {
break;
arul-cc commented on issue #2120:
URL:
https://github.com/apache/iceberg-python/issues/2120#issuecomment-3034456297
Thanks @kevinjqliu
--
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 specif
arul-cc closed issue #2120: Table metadata corruption during parallel upsert
operations
URL: https://github.com/apache/iceberg-python/issues/2120
--
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
pan3793 commented on PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#issuecomment-3034287282
@RussellSpitzer @amogh-jahagirdar thanks for the review.
- I have tuned the test cases you pointed to make them meaningful, please
take a look at each inline reply for details.
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2184185956
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/procedures/AncestorsOfProcedure.java:
##
@@ -62,18 +67,23 @@ protected AncestorsOfProcedure doBuild() {
ForeverAngry closed pull request #2170: Implement snapshot expiration and
related unit tests
URL: https://github.com/apache/iceberg-python/pull/2170
--
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
ForeverAngry opened a new pull request, #2170:
URL: https://github.com/apache/iceberg-python/pull/2170
# Rationale for this change
Removing unused member variable.
# Are these changes tested?
yes.
# Are there any user-facing changes?
No.
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2184173549
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestFastForwardBranchProcedure.java:
##
@@ -167,18 +167,21 @@ public void testInvalid
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2184173549
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestFastForwardBranchProcedure.java:
##
@@ -167,18 +167,21 @@ public void testInvalid
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2184170197
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRollbackToSnapshotProcedure.java:
##
@@ -250,27 +250,33 @@ public void testInvali
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2184169715
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestCherrypickSnapshotProcedure.java:
##
@@ -168,22 +168,26 @@ public void testInvali
ajantha-bhat commented on code in PR #13425:
URL: https://github.com/apache/iceberg/pull/13425#discussion_r2184121767
##
core/src/main/java/org/apache/iceberg/PartitionStats.java:
##
@@ -109,7 +114,12 @@ public void liveEntry(ContentFile file, Snapshot
snapshot) {
brea
ccmao1130 opened a new pull request, #2169:
URL: https://github.com/apache/iceberg-python/pull/2169
# Rationale for this change
We recently changed our site domain so want to update all Daft documentation
links. And noticed that our package should be updated from `
gty404 commented on code in PR #112:
URL: https://github.com/apache/iceberg-cpp/pull/112#discussion_r2183983587
##
src/iceberg/table_scan.cc:
##
@@ -0,0 +1,286 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See
github-actions[bot] commented on PR #13224:
URL: https://github.com/apache/iceberg/pull/13224#issuecomment-3033994276
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pul
github-actions[bot] commented on PR #13220:
URL: https://github.com/apache/iceberg/pull/13220#issuecomment-3033994222
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pul
github-actions[bot] commented on PR #13066:
URL: https://github.com/apache/iceberg/pull/13066#issuecomment-3033994122
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pul
github-actions[bot] commented on PR #12968:
URL: https://github.com/apache/iceberg/pull/12968#issuecomment-3033994081
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pul
github-actions[bot] commented on issue #11822:
URL: https://github.com/apache/iceberg/issues/11822#issuecomment-3033993853
This issue has been closed because it has not received any activity in the
last 14 days since being marked as 'stale'
--
This is an automated message from the Apache
github-actions[bot] closed issue #11822: 在使用flink执行任务的时候报错，可以帮我看一下吗
URL: https://github.com/apache/iceberg/issues/11822
--
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
ccmao1130 opened a new pull request, #13463:
URL: https://github.com/apache/iceberg/pull/13463
We recently changed our site domain so want to update all Daft documentation
links
(I think my last PR https://github.com/apache/iceberg/pull/12860 didn't make
it into release so all the li
RussellSpitzer commented on PR #13461:
URL: https://github.com/apache/iceberg/pull/13461#issuecomment-3033899350
Merged, Thanks @singhpk234 !
--
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 s
RussellSpitzer merged PR #13461:
URL: https://github.com/apache/iceberg/pull/13461
--
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...@ic
stevenzwu commented on code in PR #13189:
URL: https://github.com/apache/iceberg/pull/13189#discussion_r2183702710
##
format/spec.md:
##
@@ -101,10 +101,10 @@ Inheriting the sequence number from manifest metadata
allows writing a new manif
Row-level deletes are stored in del
singhpk234 commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183858089
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,24 @@ private String rebuildMetadata() {
singhpk234 commented on PR #13461:
URL: https://github.com/apache/iceberg/pull/13461#issuecomment-3033835294
cc @amogh-jahagirdar @RussellSpitzer @stevenzwu
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
NikitaMatskevich commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183847443
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,24 @@ private String rebuildMetadata(
amogh-jahagirdar commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183832620
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestWriteAborts.java:
##
@@ -121,7 +121,7 @@ public void testBatchAppend() t
szehon-ho commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183827290
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/procedures/BaseProcedure.java:
##
@@ -44,19 +46,37 @@
import org.apache.spark.sql.connector.catalog.Iden
RussellSpitzer commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183806637
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/procedures/BaseProcedure.java:
##
@@ -44,19 +46,37 @@
import org.apache.spark.sql.connector.catalog
RussellSpitzer commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183806637
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/procedures/BaseProcedure.java:
##
@@ -44,19 +46,37 @@
import org.apache.spark.sql.connector.catalog
RussellSpitzer commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183804631
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/procedures/AncestorsOfProcedure.java:
##
@@ -62,18 +67,23 @@ protected AncestorsOfProcedure doBuild(
RussellSpitzer commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183801608
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteManifestsProcedure.java:
##
@@ -265,7 +266,7 @@ public void testRew
RussellSpitzer commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183798986
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestFastForwardBranchProcedure.java:
##
@@ -167,18 +167,21 @@ public void test
RussellSpitzer commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183797309
##
spark/v4.0/spark-extensions/src/main/scala/org/apache/spark/sql/execution/datasources/v2/ExtendedDataSourceV2Strategy.scala:
##
@@ -76,7 +71,7 @@ case class
RussellSpitzer commented on issue #13460:
URL: https://github.com/apache/iceberg/issues/13460#issuecomment-3033757352
I'm not sure this is related to Iceberg, but the code you are using assumes
that the Spark instance is setup with the default spark_catalog which uses a
Hive based implement
ebyhr commented on issue #13456:
URL: https://github.com/apache/iceberg/issues/13456#issuecomment-3033752679
I'm not sure why this is marked as a bug. Iceberg 1.9.1 doesn't contain
`BigQueryMetastoreCatalog` as far as I know. You will need to wait for 1.10.0
release.
--
This is an autom
amogh-jahagirdar merged PR #12450:
URL: https://github.com/apache/iceberg/pull/12450
--
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...@
singhpk234 commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183779315
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,24 @@ private String rebuildMetadata() {
aihuaxu commented on issue #13462:
URL: https://github.com/apache/iceberg/issues/13462#issuecomment-3033728508
cc @rdblue @danielcweeks, @RussellSpitzer. Let me know if we have a solution
to address this without making such spec change.
--
This is an automated message from the Apache Gi
norton120 commented on PR #1862:
URL: https://github.com/apache/iceberg-python/pull/1862#issuecomment-3033724018
> Thanks @norton120 I think this is a great addition to the docs 🙌 I left
some small suggestions, let me know what you think of it 👍
@Fokko sorry, somehow this slipped th
aihuaxu opened a new issue, #13462:
URL: https://github.com/apache/iceberg/issues/13462
### Proposed Change
https://github.com/apache/iceberg/pull/12658 defines the spec for variant
metadata.
For a Variant column with the schema as follows:
```
{
"event_type": "l
amogh-jahagirdar commented on code in PR #13222:
URL: https://github.com/apache/iceberg/pull/13222#discussion_r2183765078
##
core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java:
##
@@ -1130,6 +1132,11 @@ protected ManifestReader
newManifestReader(ManifestFile man
singhpk234 opened a new pull request, #13461:
URL: https://github.com/apache/iceberg/pull/13461
### About the change
This change cherry-picks 502 / 504 not being retried to 1.9 branch so that
we can proceed with 1.9.2 release
--
This is an automated message from the Apach
Fokko commented on code in PR #2168:
URL: https://github.com/apache/iceberg-python/pull/2168#discussion_r2183728459
##
mkdocs/docs/expression-dsl.md:
##
@@ -151,20 +151,6 @@ age_in_range = Not(
)
```
-### Type Safety
Review Comment:
I think this is confusing, since this
Fokko opened a new pull request, #2168:
URL: https://github.com/apache/iceberg-python/pull/2168
# Rationale for this change
# Are these changes tested?
# Are there any user-facing changes?
--
This is an automated message from the Apache Git Ser
Fokko commented on PR #1862:
URL: https://github.com/apache/iceberg-python/pull/1862#issuecomment-3033589429
I think this is very valuable, and let's merge this and follow up with a PR
to clean up the two nits
--
This is an automated message from the Apache Git Service.
To respond to the
Fokko merged PR #1862:
URL: https://github.com/apache/iceberg-python/pull/1862
--
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
brunolnetto opened a new issue, #13460:
URL: https://github.com/apache/iceberg/issues/13460
### Query engine
Spark
```
> pip show pyspark
Name: pyspark
Version: 4.0.0
Summary: Apache Spark Python API
Home-page: https://github.com/apache/spark/tree/master/python
stevenzwu commented on code in PR #13425:
URL: https://github.com/apache/iceberg/pull/13425#discussion_r2183690496
##
core/src/main/java/org/apache/iceberg/PartitionStats.java:
##
@@ -109,7 +114,12 @@ public void liveEntry(ContentFile file, Snapshot
snapshot) {
break;
stevenzwu commented on code in PR #13189:
URL: https://github.com/apache/iceberg/pull/13189#discussion_r2183702710
##
format/spec.md:
##
@@ -101,10 +101,10 @@ Inheriting the sequence number from manifest metadata
allows writing a new manif
Row-level deletes are stored in del
stevenzwu commented on code in PR #13425:
URL: https://github.com/apache/iceberg/pull/13425#discussion_r2183690496
##
core/src/main/java/org/apache/iceberg/PartitionStats.java:
##
@@ -109,7 +114,12 @@ public void liveEntry(ContentFile file, Snapshot
snapshot) {
break;
szehon-ho commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183684054
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/procedures/BaseProcedure.java:
##
@@ -178,6 +184,29 @@ protected InternalRow newInternalRow(Object... val
szehon-ho commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183678138
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAncestorsOfProcedure.java:
##
@@ -147,14 +147,16 @@ public void testAncestorOfU
szehon-ho commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183677105
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAncestorsOfProcedure.java:
##
@@ -147,14 +147,16 @@ public void testAncestorOfU
szehon-ho commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183677105
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAncestorsOfProcedure.java:
##
@@ -147,14 +147,16 @@ public void testAncestorOfU
szehon-ho commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183670641
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,24 @@ private String rebuildMetadata() {
ehsantn opened a new pull request, #2167:
URL: https://github.com/apache/iceberg-python/pull/2167
# Rationale for this change
Adds support for Bodo DataFrame library, which is a drop in replacement for
Pandas that accelerates and scales Python code automaticall
Fokko commented on code in PR #13398:
URL: https://github.com/apache/iceberg/pull/13398#discussion_r2183634318
##
api/src/main/java/org/apache/iceberg/expressions/StrictMetricsEvaluator.java:
##
@@ -69,13 +71,26 @@ public StrictMetricsEvaluator(Schema schema, Expression
unbound
NikitaMatskevich commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183601029
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,42 @@ private String rebuildMetadata(
NikitaMatskevich commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183601029
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,42 @@ private String rebuildMetadata(
NikitaMatskevich commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183601029
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,42 @@ private String rebuildMetadata(
NikitaMatskevich commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183594589
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,42 @@ private String rebuildMetadata(
jayceslesar commented on issue #2159:
URL:
https://github.com/apache/iceberg-python/issues/2159#issuecomment-3033347997
@Fokko @kevinjqliu do you think its worth setting up a roadmap for what
should be candidates for rolling wheels from rust? Would really help focus
efforts on lacking part
koenvo commented on issue #2159:
URL:
https://github.com/apache/iceberg-python/issues/2159#issuecomment-309213
Totally agree. Lets start exploring the iceberg-rust codebase
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183558595
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAncestorsOfProcedure.java:
##
@@ -147,14 +147,16 @@ public void testAncestorOfUsi
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183558595
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAncestorsOfProcedure.java:
##
@@ -147,14 +147,16 @@ public void testAncestorOfUsi
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183558595
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAncestorsOfProcedure.java:
##
@@ -147,14 +147,16 @@ public void testAncestorOfUsi
pan3793 commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183558595
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAncestorsOfProcedure.java:
##
@@ -147,14 +147,16 @@ public void testAncestorOfUsi
szehon-ho commented on code in PR #13459:
URL: https://github.com/apache/iceberg/pull/13459#discussion_r2183529878
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java:
##
@@ -312,22 +316,42 @@ private String rebuildMetadata() {
jayceslesar commented on issue #2159:
URL:
https://github.com/apache/iceberg-python/issues/2159#issuecomment-3033171165
> Honestly, I think it would be a better use of community resources to
invest more in the iceberg-rust/datafusion path so that the bulk of this logic
can be moved out
manirajv06 commented on code in PR #13398:
URL: https://github.com/apache/iceberg/pull/13398#discussion_r2183470286
##
api/src/main/java/org/apache/iceberg/expressions/StrictMetricsEvaluator.java:
##
@@ -69,13 +71,26 @@ public StrictMetricsEvaluator(Schema schema, Expression
un
szehon-ho commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183446598
##
spark/v4.0/spark-extensions/src/main/scala/org/apache/spark/sql/execution/datasources/v2/ExtendedDataSourceV2Strategy.scala:
##
@@ -76,7 +71,7 @@ case class Exte
szehon-ho commented on code in PR #13106:
URL: https://github.com/apache/iceberg/pull/13106#discussion_r2183443385
##
spark/v4.0/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAncestorsOfProcedure.java:
##
@@ -147,14 +147,16 @@ public void testAncestorOfU
stevenzwu commented on code in PR #13222:
URL: https://github.com/apache/iceberg/pull/13222#discussion_r2183335091
##
core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java:
##
@@ -920,6 +920,8 @@ protected Map summary() {
@Override
public List apply(TableMet
stevenzwu commented on code in PR #13222:
URL: https://github.com/apache/iceberg/pull/13222#discussion_r2183335091
##
core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java:
##
@@ -920,6 +920,8 @@ protected Map summary() {
@Override
public List apply(TableMet
stevenzwu commented on code in PR #13222:
URL: https://github.com/apache/iceberg/pull/13222#discussion_r2183335091
##
core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java:
##
@@ -920,6 +920,8 @@ protected Map summary() {
@Override
public List apply(TableMet
stevenzwu commented on code in PR #13222:
URL: https://github.com/apache/iceberg/pull/13222#discussion_r2183335091
##
core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java:
##
@@ -920,6 +920,8 @@ protected Map summary() {
@Override
public List apply(TableMet
NikitaMatskevich commented on issue #13458:
URL: https://github.com/apache/iceberg/issues/13458#issuecomment-3033030561
PR was opened here: https://github.com/apache/iceberg/pull/13459
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
stevenzwu commented on code in PR #13222:
URL: https://github.com/apache/iceberg/pull/13222#discussion_r2183270318
##
core/src/test/java/org/apache/iceberg/TestRewriteFiles.java:
##
@@ -777,4 +778,40 @@ public void testNewDeleteFile() {
.rewriteFiles(Sets.newSet(FIL
NikitaMatskevich opened a new pull request, #13459:
URL: https://github.com/apache/iceberg/pull/13459
RewriteTablePath leverages iceberg native IO for everything except for
writing a "file list" file. For this task it currently leverages a Spark
writer, which in turn uses Hadoop. This might
NikitaMatskevich opened a new issue, #13458:
URL: https://github.com/apache/iceberg/issues/13458
### Feature Request / Improvement
RewriteTablePath leverages iceberg native IO for everything except for
writing a "file list" file. For this task it currently leverages a Spark
writer, w
SanjayMarreddi commented on PR #13347:
URL: https://github.com/apache/iceberg/pull/13347#issuecomment-3033021243
Hi @geruh, I have updated the PR with a new commit that enables running all
the relevant integration tests with and without AAL as discussed above. Please
have a look at it. Than
CTTY commented on PR #1484:
URL: https://github.com/apache/iceberg-rust/pull/1484#issuecomment-3033003159
cc @liurenjie1024 @Xuanwo this PR is ready for review, PTAL, thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
sriharshaj commented on PR #12634:
URL: https://github.com/apache/iceberg/pull/12634#issuecomment-3032971075
Commenting to keep it alive.
--
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 speci
corleyma commented on issue #2159:
URL:
https://github.com/apache/iceberg-python/issues/2159#issuecomment-3032961475
I think @Anton-Tarazi's original point -- creating a bunch of (Python
object) filter expressions for every row in a large dataframe is going to be
slow, and we do that befor
rmoff commented on issue #13457:
URL: https://github.com/apache/iceberg/issues/13457#issuecomment-3032947176
The problem here is the SMT. If I use dynamic routing and just rely on an
existing field in the data, it works fine:
```
echo '{"target":"tmp.scratch","order_id": "001", "cu
rmoff opened a new issue, #13457:
URL: https://github.com/apache/iceberg/issues/13457
### Apache Iceberg version
None
### Query engine
None
### Please describe the bug 🐞
## Summary
If I use
```
"iceberg.tables":"tmp.static_orders_json",
huaxingao commented on PR #13410:
URL: https://github.com/apache/iceberg/pull/13410#issuecomment-3032927845
We have more
[discussion](https://github.com/apache/datafusion-comet/issues/1934) in Comet
community and decided to keep the current arrow shading in iceberg. I will
close this PR fo
huaxingao closed pull request #13410: Exclude org.apache.arrow.c.** from shading
URL: https://github.com/apache/iceberg/pull/13410
--
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
xiaoxuandev commented on PR #13451:
URL: https://github.com/apache/iceberg/pull/13451#issuecomment-3032835154
@manuzhang That makes sense. I’ve updated the PR to target 4.0 only. We
could backporting to 3.4 as well.
--
This is an automated message from the Apache Git Service.
To respond t
igorvoltaic commented on PR #11623:
URL: https://github.com/apache/iceberg/pull/11623#issuecomment-3032806210
BTW, faced an issue with current type of routing. We are using datahub as
metastore and there is no way to correctly map table to topic to produce linage
in the metastore.
--
Th
ZENOTME commented on PR #1485:
URL: https://github.com/apache/iceberg-rust/pull/1485#issuecomment-3032757551
cc @liurenjie1024 @Xuanwo @Fokko @sdd
--
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
ZENOTME opened a new pull request, #1485:
URL: https://github.com/apache/iceberg-rust/pull/1485
## Which issue does this PR close?
I think we don't need data_file_content in FileScanTask seems it's always be
`Data` and the delete file will be stored in `deletes`.
## Wha
mun1r0b0t commented on PR #11623:
URL: https://github.com/apache/iceberg/pull/11623#issuecomment-3032681544
Still here, still waiting.
--
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
danielcweeks commented on code in PR #12562:
URL: https://github.com/apache/iceberg/pull/12562#discussion_r2183055209
##
core/src/main/java/org/apache/iceberg/rest/auth/OAuth2Manager.java:
##
@@ -61,7 +61,7 @@ public class OAuth2Manager extends RefreshingAuthManager {
private
1 - 100 of 209 matches
Mail list logo
