HonahX commented on code in PR #444:
URL: https://github.com/apache/iceberg-python/pull/444#discussion_r1513978231
##
pyiceberg/io/pyarrow.py:
##
@@ -1717,54 +1717,65 @@ def fill_parquet_file_metadata(
def write_file(table: Table, tasks: Iterator[WriteTask], file_schema:
O
Dysprosium0626 commented on PR #227:
URL: https://github.com/apache/iceberg-rust/pull/227#issuecomment-1980224024
@liurenjie1024 Hi, this PR is ready for review. Thx~
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
hashhar commented on PR #5707:
URL: https://github.com/apache/iceberg/pull/5707#issuecomment-1980199024
after this change while you can drop partition columns such tables become
broken if they hadn't been optimized before dropping the column. Basically see
https://github.com/apache/iceberg/
hashhar commented on issue #4563:
URL: https://github.com/apache/iceberg/issues/4563#issuecomment-1980198119
🤦 I didn't realise this was issue under `apache/iceberg` and not
`trinodb/trino`. Sorry for the Trino specific discussion.
I now realise that the issue is exactly asking for wh
HonahX commented on issue #499:
URL: https://github.com/apache/iceberg-python/issues/499#issuecomment-1980189490
@ndrluis Thanks for reporting this! I think we should fix these to import
from the right module. Do you want to work on this?
--
This is an automated message from the Apache Gi
hashhar commented on issue #4563:
URL: https://github.com/apache/iceberg/issues/4563#issuecomment-1980186447
cc: @rdblue / @RussellSpitzer Probably Spark should also disallow dropping
partition column if it's referenced in the live table files.
Here's `files` metadata table AFTER dro
hashhar commented on issue #4563:
URL: https://github.com/apache/iceberg/issues/4563#issuecomment-1980182977
hmmm, looks like BEFORE dropping a partition column I need to `optimize` the
table otherwise subsequent reads fail (both in Spark and Trino).
Here's a sample sequence of steps:
hashhar commented on issue #4563:
URL: https://github.com/apache/iceberg/issues/4563#issuecomment-1980169111
Iceberg 1.1.0 had
https://github.com/apache/iceberg/commit/3b65cca8b5122157d112f232a8e110be93a740e5
which seems to address something. Can we now allow this on Trino?
@alexjo21
nk1506 commented on code in PR #9852:
URL: https://github.com/apache/iceberg/pull/9852#discussion_r1513864795
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -269,6 +284,160 @@ public void renameTable(TableIdentifier from,
TableIdentifier origina
Xuanwo commented on issue #113:
URL: https://github.com/apache/iceberg-rust/issues/113#issuecomment-1980137973
> I setup the basis test-infra for the hms-catalog. In the next couple of
days I will try to work on / finish the implementation of the catalog.
That's great!
--
This is a
nk1506 commented on code in PR #9852:
URL: https://github.com/apache/iceberg/pull/9852#discussion_r1513851518
##
core/src/main/java/org/apache/iceberg/BaseMetastoreTableOperations.java:
##
@@ -309,65 +304,20 @@ protected enum CommitStatus {
* @return Commit Status of Success
marvinlanhenke commented on issue #113:
URL: https://github.com/apache/iceberg-rust/issues/113#issuecomment-1980128393
@Xuanwo
Yes, I thought the same - mocking the HMS would not be beneficial.
I setup the basis test-infra for the hms-catalog. In the next couple of days
I will try to
RussellSpitzer commented on issue #9869:
URL: https://github.com/apache/iceberg/issues/9869#issuecomment-1980120089
No it also removes data files, the only exception is files which are not
referenced by the Iceberg table.
--
This is an automated message from the Apache Git Service.
To res
HonahX commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513787586
##
pyiceberg/catalog/glue.py:
##
@@ -437,46 +471,69 @@ def _commit_table(self, table_request:
CommitTableRequest) -> CommitTableRespons
)
databas
nqvuong1998 commented on issue #9869:
URL: https://github.com/apache/iceberg/issues/9869#issuecomment-1980057905
In my case, when using PURGE, Iceberg only deletes metadata files. Do I need
any conf to be able to delete data files?
--
This is an automated message from the Apache Git Serv
HonahX commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513787586
##
pyiceberg/catalog/glue.py:
##
@@ -437,46 +471,69 @@ def _commit_table(self, table_request:
CommitTableRequest) -> CommitTableRespons
)
databas
danielcweeks commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513777158
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients
danielcweeks commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513776780
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients
danielcweeks commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513775889
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients
danielcweeks commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513775619
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients
manuzhang commented on code in PR #9819:
URL: https://github.com/apache/iceberg/pull/9819#discussion_r1513757865
##
docs/docs/flink.md:
##
@@ -24,20 +24,20 @@ Apache Iceberg supports both [Apache
Flink](https://flink.apache.org/)'s DataStr
| Feature support
syun64 commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513742498
##
pyiceberg/catalog/glue.py:
##
@@ -437,46 +471,69 @@ def _commit_table(self, table_request:
CommitTableRequest) -> CommitTableRespons
)
databas
HonahX commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513718061
##
pyiceberg/catalog/glue.py:
##
@@ -358,6 +365,33 @@ def _get_glue_table(self, database_name: str, table_name:
str) -> TableTypeDef:
except self.glue.exc
syun64 closed issue #464: Support ID Tokens in Rest Catalog
URL: https://github.com/apache/iceberg-python/issues/464
--
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 unsubscr
syun64 commented on issue #464:
URL: https://github.com/apache/iceberg-python/issues/464#issuecomment-1979940847
Awesome. Thanks for raising this issue in the first place and walking the
community through this discussion!
--
This is an automated message from the Apache Git Service.
To res
HonahX commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513709599
##
pyiceberg/table/__init__.py:
##
@@ -753,6 +807,143 @@ def update_table_metadata(base_metadata: TableMetadata,
updates: Tuple[TableUpda
return new_metadata.
HonahX merged PR #473:
URL: https://github.com/apache/iceberg-python/pull/473
--
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
HonahX commented on PR #473:
URL: https://github.com/apache/iceberg-python/pull/473#issuecomment-1979926180
Hi @Fokko @syun64
> I feel a bit awkward adding it to the write-options because the current
list of write-options map to pyarrow parquet properties.
I have a similar feeling
liurenjie1024 commented on PR #228:
URL: https://github.com/apache/iceberg-rust/pull/228#issuecomment-1979916413
cc @Xuanwo @ZENOTME @Fokko PTAL
--
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
liurenjie1024 commented on code in PR #228:
URL: https://github.com/apache/iceberg-rust/pull/228#discussion_r1513696354
##
crates/iceberg/src/expr/mod.rs:
##
@@ -30,7 +30,7 @@ pub use predicate::*;
/// The discriminant of this enum is used for determining the type of the
opera
syun64 commented on PR #473:
URL: https://github.com/apache/iceberg-python/pull/473#issuecomment-1979882949
Shall we merge this in if the place in the documentation (Write options vs
Backward Compatibility) is the only subject for debate? We can always update
the docs. Would appreciate it i
flyrain commented on issue #464:
URL: https://github.com/apache/iceberg-python/issues/464#issuecomment-1979866702
Yeah, it is normally an anti-pattern to use id-token for resource servers.
For example, the id token will carry all audiences that the client has, which
could be misused, e.g.,
github-actions[bot] commented on issue #1413:
URL: https://github.com/apache/iceberg/issues/1413#issuecomment-1979847605
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 Gi
github-actions[bot] commented on issue #1412:
URL: https://github.com/apache/iceberg/issues/1412#issuecomment-1979847579
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 Gi
github-actions[bot] closed issue #1413: Support Parquet modular encryption
URL: https://github.com/apache/iceberg/issues/1413
--
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
github-actions[bot] closed issue #1412: Consider a builder in TableMetadata
URL: https://github.com/apache/iceberg/issues/1412
--
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.
T
github-actions[bot] closed issue #1403: Flink: Improvements for iceberg sink
connector.
URL: https://github.com/apache/iceberg/issues/1403
--
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
github-actions[bot] commented on issue #1403:
URL: https://github.com/apache/iceberg/issues/1403#issuecomment-1979847544
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 Gi
github-actions[bot] closed issue #1398: No retries on snapshot commit on
eventual consistent file system
URL: https://github.com/apache/iceberg/issues/1398
--
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
github-actions[bot] commented on issue #1398:
URL: https://github.com/apache/iceberg/issues/1398#issuecomment-1979847511
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 Gi
ajantha-bhat commented on PR #9875:
URL: https://github.com/apache/iceberg/pull/9875#issuecomment-1979833138
cc: @stevenzwu, @pvary
--
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
syun64 commented on issue #464:
URL: https://github.com/apache/iceberg-python/issues/464#issuecomment-1979774474
@flyrain do you have any thoughts on this? Does this issue require more
discussion, or could we close this with the understanding that ID tokens should
not be used against a reso
mkavinashkumar opened a new issue, #9874:
URL: https://github.com/apache/iceberg/issues/9874
### Apache Iceberg version
1.4.3 (latest release)
### Query engine
Spark
### Please describe the bug 🐞
I get an error when I try to append data using the writeTo api
bitsondatadev commented on PR #9836:
URL: https://github.com/apache/iceberg/pull/9836#issuecomment-1979739317
@nastra, I tend to agree with @jaychia on this one. I don't want to split up
the documentation any more than necessary. Any compute engine that runs on
Iceberg, I want to
I
odysa commented on code in PR #229:
URL: https://github.com/apache/iceberg-rust/pull/229#discussion_r1513340583
##
crates/catalog/sql/src/catalog.rs:
##
@@ -0,0 +1,397 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
ndrluis opened a new issue, #499:
URL: https://github.com/apache/iceberg-python/issues/499
### Feature Request / Improvement
Hello, when I was trying to solve the #497 issue, I noticed that we are
exposing the private API and we have some imports through modules.
For example,
martin-g commented on code in PR #229:
URL: https://github.com/apache/iceberg-rust/pull/229#discussion_r1513532822
##
crates/catalog/sql/src/error.rs:
##
@@ -0,0 +1,27 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
martin-g commented on code in PR #229:
URL: https://github.com/apache/iceberg-rust/pull/229#discussion_r1513531369
##
crates/catalog/sql/src/catalog.rs:
##
@@ -0,0 +1,397 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements
martin-g commented on code in PR #229:
URL: https://github.com/apache/iceberg-rust/pull/229#discussion_r1513530747
##
crates/catalog/sql/src/catalog.rs:
##
@@ -0,0 +1,397 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements
flyrain commented on code in PR #486:
URL: https://github.com/apache/iceberg-python/pull/486#discussion_r1513528807
##
pyiceberg/catalog/rest.py:
##
@@ -289,12 +291,25 @@ def auth_url(self) -> str:
else:
return self.url(Endpoints.get_token, prefixed=False)
ndrluis closed issue #497: Import statement unexpectedly executes functions in
__init__.py
URL: https://github.com/apache/iceberg-python/issues/497
--
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
ndrluis commented on issue #497:
URL: https://github.com/apache/iceberg-python/issues/497#issuecomment-1979673270
I found the problem, it's related to Python 3.9.7. I updated to 3.9.12 and
fixed the error.
From the project's perspective, I believe that the isolation of the public
API
ndrluis commented on issue #497:
URL: https://github.com/apache/iceberg-python/issues/497#issuecomment-1979663033
I was wrong about my hypothesis, but I believe that it is something
occurring during the import process because the stacktrace error shows that the
import is executing certain o
flyrain commented on issue #463:
URL: https://github.com/apache/iceberg-python/issues/463#issuecomment-1979651924
Closed. Thanks @danielcweeks and @syun64 !
--
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
flyrain closed issue #463: Support OAuth2 Client credential flow
URL: https://github.com/apache/iceberg-python/issues/463
--
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 uns
jaychia commented on PR #9836:
URL: https://github.com/apache/iceberg/pull/9836#issuecomment-1979626594
> Given that this mainly targets pyiceberg, I think this doc should live in
https://github.com/apache/iceberg-python/
Hello! The Daft query engine integrates with `pyiceberg` curren
huaxingao commented on PR #9721:
URL: https://github.com/apache/iceberg/pull/9721#issuecomment-1979578692
cc @aokolnychyi @rdblue @RussellSpitzer @flyrain
Here is the Comet/iceberg integration
[PR](https://github.com/apache/iceberg/pull/9841). Please take a look when you
have time. Tha
szehon-ho commented on code in PR #9852:
URL: https://github.com/apache/iceberg/pull/9852#discussion_r1513414672
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -222,53 +230,188 @@ public boolean dropTable(TableIdentifier identifier,
boolean purg
szehon-ho commented on code in PR #9852:
URL: https://github.com/apache/iceberg/pull/9852#discussion_r1513413754
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -269,6 +284,160 @@ public void renameTable(TableIdentifier from,
TableIdentifier orig
szehon-ho commented on code in PR #9852:
URL: https://github.com/apache/iceberg/pull/9852#discussion_r1513407885
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -222,53 +230,188 @@ public boolean dropTable(TableIdentifier identifier,
boolean purg
szehon-ho commented on code in PR #9852:
URL: https://github.com/apache/iceberg/pull/9852#discussion_r1513399291
##
core/src/main/java/org/apache/iceberg/BaseMetastoreTableOperations.java:
##
@@ -291,7 +286,7 @@ public long newSnapshotId() {
};
}
- protected enum Comm
szehon-ho commented on code in PR #9852:
URL: https://github.com/apache/iceberg/pull/9852#discussion_r1513399291
##
core/src/main/java/org/apache/iceberg/BaseMetastoreTableOperations.java:
##
@@ -291,7 +286,7 @@ public long newSnapshotId() {
};
}
- protected enum Comm
sdd commented on code in PR #229:
URL: https://github.com/apache/iceberg-rust/pull/229#discussion_r1513373458
##
crates/catalog/sql/src/catalog.rs:
##
@@ -0,0 +1,397 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. Se
aokolnychyi commented on code in PR #9873:
URL: https://github.com/apache/iceberg/pull/9873#discussion_r1513354406
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/functions/BaseScalarFunction.java:
##
@@ -0,0 +1,40 @@
+/*
+ * Licensed to the Apache Software Foundation
aokolnychyi commented on code in PR #9873:
URL: https://github.com/apache/iceberg/pull/9873#discussion_r1513354406
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/functions/BaseScalarFunction.java:
##
@@ -0,0 +1,40 @@
+/*
+ * Licensed to the Apache Software Foundation
aokolnychyi commented on code in PR #9873:
URL: https://github.com/apache/iceberg/pull/9873#discussion_r1513352554
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestSystemFunctionPushDownInRowLevelOperations.java:
##
@@ -0,0 +1,348 @@
+/*
+ * L
aokolnychyi commented on code in PR #9873:
URL: https://github.com/apache/iceberg/pull/9873#discussion_r1513352554
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestSystemFunctionPushDownInRowLevelOperations.java:
##
@@ -0,0 +1,348 @@
+/*
+ * L
aokolnychyi commented on code in PR #9873:
URL: https://github.com/apache/iceberg/pull/9873#discussion_r1513351770
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/SparkPlanUtil.java:
##
@@ -53,6 +58,49 @@ private static SparkPlan actualPlan(Spark
aokolnychyi commented on code in PR #9873:
URL: https://github.com/apache/iceberg/pull/9873#discussion_r1513350467
##
spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceStaticInvoke.scala:
##
@@ -40,21 +48,39 @@ import org.apache.spark.sql.
aokolnychyi commented on code in PR #9873:
URL: https://github.com/apache/iceberg/pull/9873#discussion_r1513349734
##
spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceStaticInvoke.scala:
##
@@ -40,21 +48,39 @@ import org.apache.spark.sql.
aokolnychyi commented on code in PR #9873:
URL: https://github.com/apache/iceberg/pull/9873#discussion_r1513348794
##
spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceStaticInvoke.scala:
##
@@ -40,21 +48,39 @@ import org.apache.spark.sql.
aokolnychyi opened a new pull request, #9873:
URL: https://github.com/apache/iceberg/pull/9873
This PR fixes the system function predicate pushdown in CoW MERGE operations
and adds tests for other use cases. Prior to this PR, we did not cover
`ReplaceData` and `Join` nodes that are particip
zachdisc commented on PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#issuecomment-1979434799
> I see that's a good point, using the UDF function definitely is less
performant than sorting native columns, in that case we can keep it as is,
unless anyone has a way to fix the perf.
rdblue commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513332111
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients to mak
rdblue commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513331183
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients to mak
danielcweeks commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513326436
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients
rdblue commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513322269
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients to mak
rdblue commented on code in PR #9872:
URL: https://github.com/apache/iceberg/pull/9872#discussion_r1513321631
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1610,13 +1610,19 @@ components:
PageToken:
description:
-An opaque token which allows clients to mak
danielcweeks merged PR #9871:
URL: https://github.com/apache/iceberg/pull/9871
--
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
danielcweeks opened a new pull request, #9872:
URL: https://github.com/apache/iceberg/pull/9872
(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,
syun64 commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513259769
##
pyiceberg/table/__init__.py:
##
@@ -753,6 +807,143 @@ def update_table_metadata(base_metadata: TableMetadata,
updates: Tuple[TableUpda
return new_metadata.
syun64 commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513264836
##
pyiceberg/catalog/glue.py:
##
@@ -437,46 +471,69 @@ def _commit_table(self, table_request:
CommitTableRequest) -> CommitTableRespons
)
databas
syun64 commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513259769
##
pyiceberg/table/__init__.py:
##
@@ -753,6 +807,143 @@ def update_table_metadata(base_metadata: TableMetadata,
updates: Tuple[TableUpda
return new_metadata.
jackye1995 commented on PR #9866:
URL: https://github.com/apache/iceberg/pull/9866#issuecomment-1979323286
I think we still need to figure out `GET /tables?pageToken` vs `GET
/tables?pageToken=` or both.
As I said in the old PR, in https://www.youtube.com/watch?v=uAQVGd5zV4I,
startin
nastra commented on code in PR #9871:
URL: https://github.com/apache/iceberg/pull/9871#discussion_r1513237163
##
deploy.gradle:
##
@@ -22,6 +22,11 @@ if (project.hasProperty('release') && jdkVersion != '8') {
}
subprojects {
+ if (it.name == 'iceberg-open-api') {
Review Co
syun64 commented on code in PR #498:
URL: https://github.com/apache/iceberg-python/pull/498#discussion_r1513218389
##
pyiceberg/catalog/glue.py:
##
@@ -358,6 +365,33 @@ def _get_glue_table(self, database_name: str, table_name:
str) -> TableTypeDef:
except self.glue.exc
ajantha-bhat commented on code in PR #9870:
URL: https://github.com/apache/iceberg/pull/9870#discussion_r1513198334
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1444,7 +1444,7 @@ components:
schema:
type: string
required: true
- description: An opti
syun64 commented on issue #463:
URL: https://github.com/apache/iceberg-python/issues/463#issuecomment-1979175057
Given that we've confirmed that Client Credentials flow is already
supported, and we've fixed the current model to make attributes optional, is
this issue good to close @flyrain
zachdisc commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1513132291
##
api/src/main/java/org/apache/iceberg/actions/RewriteManifests.java:
##
@@ -44,6 +47,39 @@ public interface RewriteManifests
*/
RewriteManifests rewriteIf(Pre
syun64 commented on PR #473:
URL: https://github.com/apache/iceberg-python/pull/473#issuecomment-1979164521
> This looks great, one minor suggestion:
>
> Could you add the `legacy-current-snapshot-id` key to the write options
table as well:
https://github.com/apache/iceberg-python/bl
mudit-97 commented on PR #9694:
URL: https://github.com/apache/iceberg/pull/9694#issuecomment-1979163696
sure, I already have a thread there, let me add you also there
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
stevenzwu commented on PR #9694:
URL: https://github.com/apache/iceberg/pull/9694#issuecomment-1979161305
@mudit-97 next community sync meeting might be focused on materialized view.
the other option to get broader feedback is to start a discussion thread on
dev@, which tends to get more at
ajantha-bhat commented on code in PR #9871:
URL: https://github.com/apache/iceberg/pull/9871#discussion_r1513107056
##
deploy.gradle:
##
@@ -22,6 +22,11 @@ if (project.hasProperty('release') && jdkVersion != '8') {
}
subprojects {
+ if (it.name == 'iceberg-open-api') {
Rev
jackye1995 commented on PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#issuecomment-1979126601
> I don't think it yields quite as performant a result unfortunately.
I see that's a good point, using the UDF function definitely is less
performant than sorting native columns
AjayChitumalla commented on issue #8718:
URL: https://github.com/apache/iceberg/issues/8718#issuecomment-1979125075
> Is this for a V2 table? I have seen deleting rows working using V2 table,
Java code with the stream API, but I yet to try out SQL.
Can you share a reference for perfor
nastra opened a new pull request, #9871:
URL: https://github.com/apache/iceberg/pull/9871
(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
kevinjqliu commented on PR #289:
URL: https://github.com/apache/iceberg-python/pull/289#issuecomment-1979115277
Thanks for the suggestions, @Fokko
--
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
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1513085638
##
api/src/main/java/org/apache/iceberg/actions/RewriteManifests.java:
##
@@ -44,6 +47,39 @@ public interface RewriteManifests
*/
RewriteManifests rewriteIf(P
nastra commented on code in PR #9819:
URL: https://github.com/apache/iceberg/pull/9819#discussion_r1513086680
##
docs/docs/flink.md:
##
@@ -24,20 +24,20 @@ Apache Iceberg supports both [Apache
Flink](https://flink.apache.org/)'s DataStr
| Feature support
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1513085638
##
api/src/main/java/org/apache/iceberg/actions/RewriteManifests.java:
##
@@ -44,6 +47,39 @@ public interface RewriteManifests
*/
RewriteManifests rewriteIf(P
1 - 100 of 131 matches
Mail list logo
