bitsondatadev opened a new issue, #8965:
URL: https://github.com/apache/iceberg/issues/8965
There's an issue with how mkdocs renders.
https://github.com/squidfunk/mkdocs-material/issues/868
The solution is to provide an
[override](https://squidfunk.github.io/mkdocs-material/cus
waltczhang opened a new issue, #8964:
URL: https://github.com/apache/iceberg/issues/8964
### Query engine
_No response_
### Question
Theoretically, it is possible for updateSchema and UpdatePartitionSpec to
fail during the commit process. I want to know why retries have
coded9 opened a new pull request, #8963:
URL: https://github.com/apache/iceberg/pull/8963
(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-mai
wypoon commented on code in PR #8805:
URL: https://github.com/apache/iceberg/pull/8805#discussion_r1378364590
##
core/src/main/java/org/apache/iceberg/deletes/Deletes.java:
##
@@ -127,14 +130,25 @@ public static StructLikeSet toEqualitySet(
public static PositionDeleteInde
liurenjie1024 commented on code in PR #89:
URL: https://github.com/apache/iceberg-rust/pull/89#discussion_r1378337379
##
crates/iceberg/src/spec/table_metadata.rs:
##
@@ -109,50 +110,139 @@ pub struct TableMetadata {
}
impl TableMetadata {
+/// Returns format version of
liurenjie1024 commented on code in PR #89:
URL: https://github.com/apache/iceberg-rust/pull/89#discussion_r1378335610
##
crates/iceberg/src/spec/table_metadata.rs:
##
@@ -109,50 +110,139 @@ pub struct TableMetadata {
}
impl TableMetadata {
+/// Returns format version of
liurenjie1024 commented on code in PR #89:
URL: https://github.com/apache/iceberg-rust/pull/89#discussion_r1378335162
##
crates/iceberg/src/spec/table_metadata.rs:
##
@@ -109,50 +110,139 @@ pub struct TableMetadata {
}
impl TableMetadata {
+/// Returns format version of
liurenjie1024 commented on code in PR #89:
URL: https://github.com/apache/iceberg-rust/pull/89#discussion_r1378334579
##
crates/iceberg/src/spec/table_metadata.rs:
##
@@ -109,50 +110,139 @@ pub struct TableMetadata {
}
impl TableMetadata {
+/// Returns format version of
liurenjie1024 commented on code in PR #89:
URL: https://github.com/apache/iceberg-rust/pull/89#discussion_r1378325205
##
crates/catalog/rest/src/catalog.rs:
##
@@ -312,11 +316,43 @@ impl Catalog for RestCatalog {
}
/// Load table from the catalog.
-async fn load_
bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1378313906
##
site/README.md:
##
@@ -125,17 +124,16 @@ mkdocs serve
## Release process
Deploying a version of the docs is a two step process:
- 1. ~~Cut a new release fr
bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1378313906
##
site/README.md:
##
@@ -125,17 +124,16 @@ mkdocs serve
## Release process
Deploying a version of the docs is a two step process:
- 1. ~~Cut a new release fr
bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1378313906
##
site/README.md:
##
@@ -125,17 +124,16 @@ mkdocs serve
## Release process
Deploying a version of the docs is a two step process:
- 1. ~~Cut a new release fr
bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1378313906
##
site/README.md:
##
@@ -125,17 +124,16 @@ mkdocs serve
## Release process
Deploying a version of the docs is a two step process:
- 1. ~~Cut a new release fr
bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1378313362
##
site/README.md:
##
@@ -65,7 +65,7 @@ The non-versioned site pages are all the `/site/docs/.*md`
files and the docs ar
> This build process is currently missi
bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1378304225
##
site/README.md:
##
@@ -125,17 +124,16 @@ mkdocs serve
## Release process
Deploying a version of the docs is a two step process:
- 1. ~~Cut a new release fr
aokolnychyi commented on PR #8922:
URL: https://github.com/apache/iceberg/pull/8922#issuecomment-1788252194
I am not sure it is a good idea too, we always avoided respecting Spark
configs for the built-in file sources. Iceberg split planning is different.
--
This is an automated message
aokolnychyi opened a new pull request, #8962:
URL: https://github.com/apache/iceberg/pull/8962
This PR extends the idea from #8928 to `FastAppend` and `MergeAppend`, which
are used in data imports and migration.
--
This is an automated message from the Apache Git Service.
To respond to th
dramaticlly commented on issue #8951:
URL: https://github.com/apache/iceberg/issues/8951#issuecomment-1788241135
Thank you @zhangbutao , looks like this was indeed introduced by #7581,
where we changed to scan manifest entry instead of files. I am able to
reproduce the problem using provide
hussein-awala opened a new pull request, #8961:
URL: https://github.com/apache/iceberg/pull/8961
This PR removes outdated Python autoformatting command from the site doc,
where tox was replaced by pre-commit by #4811.
I'm preparing a new contribution guideline for iceberg-python, so n
github-actions[bot] commented on issue #7507:
URL: https://github.com/apache/iceberg/issues/7507#issuecomment-1788203155
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 occurs.
github-actions[bot] commented on issue #7522:
URL: https://github.com/apache/iceberg/issues/7522#issuecomment-1788203108
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 occurs.
vinitamaloo-asu commented on PR #7275:
URL: https://github.com/apache/iceberg/pull/7275#issuecomment-1788177314
@abmo-x any when do you plan to merge 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
dimas-b commented on code in PR #8909:
URL: https://github.com/apache/iceberg/pull/8909#discussion_r1378254490
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieUtil.java:
##
@@ -165,4 +180,77 @@ public static TableMetadata
updateTableMetadataWithNessieSpecificProperties(
dimas-b commented on code in PR #8909:
URL: https://github.com/apache/iceberg/pull/8909#discussion_r1378253133
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieViewOperations.java:
##
@@ -0,0 +1,159 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
dimas-b commented on code in PR #8909:
URL: https://github.com/apache/iceberg/pull/8909#discussion_r1378251837
##
core/src/test/java/org/apache/iceberg/view/ViewCatalogTests.java:
##
@@ -400,8 +400,15 @@ public void
replaceTableViaTransactionThatAlreadyExistsAsView() {
dimas-b commented on code in PR #8909:
URL: https://github.com/apache/iceberg/pull/8909#discussion_r1378251837
##
core/src/test/java/org/apache/iceberg/view/ViewCatalogTests.java:
##
@@ -400,8 +400,15 @@ public void
replaceTableViaTransactionThatAlreadyExistsAsView() {
jacobmarble opened a new pull request, #8960:
URL: https://github.com/apache/iceberg/pull/8960
Helps #8657
This change adds field `ChronoUnit unit` to `TimestampType`, such that
`TimestampType` now represents four specified types:
- `timestamp` (existing)
- `timestamptz` (existi
bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1378017052
##
site/README.md:
##
@@ -65,7 +65,7 @@ The non-versioned site pages are all the `/site/docs/.*md`
files and the docs ar
> This build process is currently missi
aokolnychyi commented on code in PR #8959:
URL: https://github.com/apache/iceberg/pull/8959#discussion_r1377997350
##
spark/v3.2/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -133,15 +143,17 @@ public RewriteManifestsSparkAction
re
aokolnychyi commented on code in PR #8928:
URL: https://github.com/apache/iceberg/pull/8928#discussion_r1377994540
##
spark/v3.2/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -337,7 +337,7 @@ private void replaceManifests(
ad
aokolnychyi opened a new pull request, #8959:
URL: https://github.com/apache/iceberg/pull/8959
This PR fixes our actions for rewriting manifests to use the provided
staging location only if the manifests are staged. If snapshot ID inheritance
is enabled (true for all V2 tables), the manifes
aokolnychyi merged PR #8957:
URL: https://github.com/apache/iceberg/pull/8957
--
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
aokolnychyi commented on PR #8957:
URL: https://github.com/apache/iceberg/pull/8957#issuecomment-1787717661
Thank you, @singhpk234 @nastra!
--
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 spe
aokolnychyi merged PR #8928:
URL: https://github.com/apache/iceberg/pull/8928
--
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
nastra merged PR #26:
URL: https://github.com/apache/iceberg-go/pull/26
--
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.apach
jacobmarble closed pull request #8658: API: add nanosecond variant of Timestamp
type
URL: https://github.com/apache/iceberg/pull/8658
--
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 com
nastra merged PR #22:
URL: https://github.com/apache/iceberg-go/pull/22
--
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.apach
Fokko merged PR #87:
URL: https://github.com/apache/iceberg-rust/pull/87
--
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.apac
dramaticlly commented on issue #8951:
URL: https://github.com/apache/iceberg/issues/8951#issuecomment-1787562193
I'll take a look and try to reproduce the problem today
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
nastra merged PR #8958:
URL: https://github.com/apache/iceberg/pull/8958
--
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.apac
nastra merged PR #21:
URL: https://github.com/apache/iceberg-go/pull/21
--
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.apach
rdblue commented on PR #8683:
URL: https://github.com/apache/iceberg/pull/8683#issuecomment-1787544377
@jacobmarble, I updated the table to be more clear about when types can be
used and merge this. Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the mes
rdblue merged PR #8683:
URL: https://github.com/apache/iceberg/pull/8683
--
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.apac
nastra merged PR #20:
URL: https://github.com/apache/iceberg-go/pull/20
--
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.apach
nastra merged PR #18:
URL: https://github.com/apache/iceberg-go/pull/18
--
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.apach
nastra merged PR #17:
URL: https://github.com/apache/iceberg-go/pull/17
--
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.apach
zhangbutao commented on issue #8951:
URL: https://github.com/apache/iceberg/issues/8951#issuecomment-1787442557
I want to know if this is Iceberg bug or not? Or this issue should be
handled by engine/user side?
Hope to get a feedback. Thanks in advance.
--
This is an automated message
Fokko commented on issue #8953:
URL: https://github.com/apache/iceberg/issues/8953#issuecomment-1787411953
Slack conversation for reference that provides some more interesting
details: https://apache-iceberg.slack.com/archives/C025PH0G1D4/p1698676018510089
--
This is an automated message
nastra merged PR #8946:
URL: https://github.com/apache/iceberg/pull/8946
--
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.apac
nastra merged PR #8945:
URL: https://github.com/apache/iceberg/pull/8945
--
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.apac
cccs-jc commented on issue #8902:
URL: https://github.com/apache/iceberg/issues/8902#issuecomment-1787207952
@singhpk234 I have a fix. Unit tested and ran it on our live data.
I will soon create a PR for you to review.
--
This is an automated message from the Apache Git Service.
To
bitsondatadev commented on PR #8958:
URL: https://github.com/apache/iceberg/pull/8958#issuecomment-1787193669
> @bitsondatadev: Should I also modify the same doc in `site` folder or
later you will sync from docs to site folder?
Continue with the same docs folder I will sync any chang
ajantha-bhat commented on PR #8958:
URL: https://github.com/apache/iceberg/pull/8958#issuecomment-1787119657
@bitsondatadev: Should I also modify the same doc in `site` folder or later
you will sync from docs to site folder?
--
This is an automated message from the Apache Git Service.
To
ajantha-bhat commented on PR #8958:
URL: https://github.com/apache/iceberg/pull/8958#issuecomment-1787117950
cc: @RussellSpitzer, @puchengy
--
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
ajantha-bhat opened a new pull request, #8958:
URL: https://github.com/apache/iceberg/pull/8958
We have testcase in
https://github.com/apache/iceberg/blob/b9a4478b0f8f5eeae553eb3900b08a7a08dbdb40/spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestSetWriteDistri
ajantha-bhat commented on issue #5071:
URL: https://github.com/apache/iceberg/issues/5071#issuecomment-1787065904
There is a test case also
https://github.com/apache/iceberg/blob/b9a4478b0f8f5eeae553eb3900b08a7a08dbdb40/spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/ex
ajantha-bhat closed issue #5071: Support UNSET of sortOrder from the SQL
URL: https://github.com/apache/iceberg/issues/5071
--
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 u
ajantha-bhat commented on issue #5071:
URL: https://github.com/apache/iceberg/issues/5071#issuecomment-1787059232
> Isn't this supported via UNORDERED or did that not get into OSS yet?
I saw the code, looks like syntax is there but no where documented or have
testcase.
I will add
nastra commented on PR #8725:
URL: https://github.com/apache/iceberg/pull/8725#issuecomment-1787057228
Closing this one as it has been superseded by
https://github.com/apache/iceberg/pull/8804
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
nastra closed pull request #8725: Thread.sleep() method is replaced with
Awaitility
URL: https://github.com/apache/iceberg/pull/8725
--
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 comm
Xuanwo commented on code in PR #89:
URL: https://github.com/apache/iceberg-rust/pull/89#discussion_r1377416089
##
crates/catalog/rest/src/catalog.rs:
##
@@ -312,11 +316,43 @@ impl Catalog for RestCatalog {
}
/// Load table from the catalog.
-async fn load_table(&
ajantha-bhat opened a new issue, #5071:
URL: https://github.com/apache/iceberg/issues/5071
Currently, users can set sort_order using `ALTER TABLE ... WRITE ORDER BY
`in SQL.
But if the users feel data loading is slow because of sorting and want to
reset it. There are no options from SQL.
RussellSpitzer commented on issue #5071:
URL: https://github.com/apache/iceberg/issues/5071#issuecomment-1786975161
Isn't this supported via UNORDERED or did that not get into OSS yet?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
nk1506 commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377381629
##
hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java:
##
@@ -166,8 +217,8 @@ public void testInitialize() {
assertThatNoException()
coded9 commented on code in PR #8945:
URL: https://github.com/apache/iceberg/pull/8945#discussion_r1377358392
##
flink/v1.15/flink/src/test/java/org/apache/iceberg/flink/source/TestStreamScanSql.java:
##
@@ -219,15 +219,13 @@ public void testConsumeFilesWithBranch() throws Excep
coded9 commented on code in PR #8945:
URL: https://github.com/apache/iceberg/pull/8945#discussion_r1377358822
##
flink/v1.15/flink/src/test/java/org/apache/iceberg/flink/source/TestStreamScanSql.java:
##
@@ -306,15 +304,13 @@ public void testConsumeFromStartTag() throws Exceptio
nastra merged PR #8948:
URL: https://github.com/apache/iceberg/pull/8948
--
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.apac
BsoBird commented on PR #7886:
URL: https://github.com/apache/iceberg/pull/7886#issuecomment-1786887107
Although I am only an iceberg user, I think this issue should be discussed
further. This solution solves the problem at hand, but it inherently hurts the
fairness of ICEBERG's support for
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377283966
##
hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java:
##
@@ -82,12 +83,20 @@
import org.apache.iceberg.types.Types;
import org.apache.iceberg.u
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377281522
##
hive-metastore/src/test/java/org/apache/iceberg/hive/HiveMetastoreExtension.java:
##
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377281522
##
hive-metastore/src/test/java/org/apache/iceberg/hive/HiveMetastoreExtension.java:
##
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
manuzhang commented on issue #8949:
URL: https://github.com/apache/iceberg/issues/8949#issuecomment-1786757877
If you check [Spark's
VectorizedColumnReader](https://github.com/apache/spark/blob/master/sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnR
JanKaul commented on issue #6420:
URL: https://github.com/apache/iceberg/issues/6420#issuecomment-1786737489
You are right, I was confused because the REST catalog stores the metadata
internally and I thought this would include the storage table. But it would be
possible to store the metada
nastra commented on code in PR #8907:
URL: https://github.com/apache/iceberg/pull/8907#discussion_r1377217101
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveViewOperations.java:
##
@@ -0,0 +1,346 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
nastra commented on code in PR #8907:
URL: https://github.com/apache/iceberg/pull/8907#discussion_r1377215123
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveViewOperations.java:
##
@@ -0,0 +1,346 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
nastra commented on code in PR #8907:
URL: https://github.com/apache/iceberg/pull/8907#discussion_r1377205765
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveTableOperations.java:
##
@@ -75,14 +74,9 @@ public class HiveTableOperations extends
BaseMetastoreTableOpera
nastra commented on code in PR #8907:
URL: https://github.com/apache/iceberg/pull/8907#discussion_r1377204878
##
core/src/test/java/org/apache/iceberg/view/ViewCatalogTests.java:
##
@@ -51,7 +51,7 @@ public abstract class ViewCatalogTests
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377190081
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -261,6 +261,15 @@ public void renameTable(TableIdentifier from,
TableIdentifier original
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377186180
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -261,6 +261,15 @@ public void renameTable(TableIdentifier from,
TableIdentifier original
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377186180
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -261,6 +261,15 @@ public void renameTable(TableIdentifier from,
TableIdentifier original
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377181258
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveTableOperations.java:
##
@@ -217,7 +217,7 @@ protected void doCommit(TableMetadata base, TableMetadata
metad
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1377179041
##
hive-metastore/src/test/java/org/apache/iceberg/hive/HiveMetastoreExtension.java:
##
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
coded9 commented on code in PR #8948:
URL: https://github.com/apache/iceberg/pull/8948#discussion_r1377177076
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRequiredDistributionAndOrdering.java:
##
@@ -186,21 +186,22 @@ public void testDisab
nastra commented on code in PR #8948:
URL: https://github.com/apache/iceberg/pull/8948#discussion_r1377154810
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestMerge.java:
##
@@ -2390,32 +2377,29 @@ public void testMergeWithInvalidUpdates() {
nastra commented on code in PR #8948:
URL: https://github.com/apache/iceberg/pull/8948#discussion_r1377152579
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestUpdate.java:
##
@@ -1425,59 +1420,57 @@ public void testUpdateWithInvalidAssignments
nastra commented on code in PR #8948:
URL: https://github.com/apache/iceberg/pull/8948#discussion_r1377150647
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestUpdate.java:
##
@@ -1333,45 +1332,42 @@ public void testUpdateWithInvalidUpdates() {
nastra commented on code in PR #8948:
URL: https://github.com/apache/iceberg/pull/8948#discussion_r1377149920
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestUpdate.java:
##
@@ -1333,45 +1332,42 @@ public void testUpdateWithInvalidUpdates() {
nastra commented on code in PR #8948:
URL: https://github.com/apache/iceberg/pull/8948#discussion_r1377146775
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRequiredDistributionAndOrdering.java:
##
@@ -186,21 +186,22 @@ public void testDisab
manuzhang commented on issue #8949:
URL: https://github.com/apache/iceberg/issues/8949#issuecomment-1786591885
@yabola timestamp96 column is using dictionary with encodings
`PLAIN_DICTIONARY` and `PLAIN`, which is not covered in your case.
--
This is an automated message from the Apache
nastra commented on issue #8950:
URL: https://github.com/apache/iceberg/issues/8950#issuecomment-1786574081
@FWLamb you might want to take a look at
https://tabular.io/blog/java-api-part-1/ to achieve that
--
This is an automated message from the Apache Git Service.
To respond to the mess
90 matches
Mail list logo
