manuzhang commented on PR #6182:
URL: https://github.com/apache/iceberg/pull/6182#issuecomment-1982827028
I just submitted https://github.com/apache/iceberg/pull/9888
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
manuzhang opened a new pull request, #9888:
URL: https://github.com/apache/iceberg/pull/9888
Pick up #6182 with an alternative implementation
--
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 commented on code in PR #9878:
URL: https://github.com/apache/iceberg/pull/9878#discussion_r1515696765
##
docs/docs/spark-ddl.md:
##
@@ -566,3 +566,85 @@ Tags can be removed via the `DROP TAG` sql
```sql
ALTER TABLE prod.db.sample DROP TAG `historical-tag`
```
+
sdd commented on PR #207:
URL: https://github.com/apache/iceberg-rust/pull/207#issuecomment-1982809816
@Fokko are you able to take a look at this one yet please? 😁
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
nastra commented on code in PR #9878:
URL: https://github.com/apache/iceberg/pull/9878#discussion_r1515692131
##
docs/docs/spark-ddl.md:
##
@@ -566,3 +566,85 @@ Tags can be removed via the `DROP TAG` sql
```sql
ALTER TABLE prod.db.sample DROP TAG `historical-tag`
```
+
+### I
burriad closed issue #9881: Spark query pushdown error in SQL query
optimization phase
URL: https://github.com/apache/iceberg/issues/9881
--
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
burriad commented on issue #9881:
URL: https://github.com/apache/iceberg/issues/9881#issuecomment-1982761923
Sorry, I tried to provide a stripped down version of the code but I cannot
reproduce it anymore.
--
This is an automated message from the Apache Git Service.
To respond to the mess
HonahX commented on code in PR #504:
URL: https://github.com/apache/iceberg-python/pull/504#discussion_r1515657781
##
pyiceberg/table/__init__.py:
##
@@ -294,17 +295,21 @@ def upgrade_table_version(self, format_version:
Literal[1, 2]) -> Transaction:
return self
-
bitsondatadev opened a new pull request, #9887:
URL: https://github.com/apache/iceberg/pull/9887
Avoid internal [MkDocs relative
linking](https://www.mkdocs.org/user-guide/writing-your-docs/#linking-to-pages)
using the [mkdocs-site-urls](https://github.com/OctoPrint/mkdocs-site-urls)
plugi
Fokko merged PR #234:
URL: https://github.com/apache/iceberg-rust/pull/234
--
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 #9878:
URL: https://github.com/apache/iceberg/pull/9878#discussion_r1515655248
##
docs/docs/spark-ddl.md:
##
@@ -566,3 +566,85 @@ Tags can be removed via the `DROP TAG` sql
```sql
ALTER TABLE prod.db.sample DROP TAG `historical-tag`
```
+
+### I
nastra commented on code in PR #9878:
URL: https://github.com/apache/iceberg/pull/9878#discussion_r1515645687
##
docs/docs/spark-ddl.md:
##
@@ -566,3 +566,85 @@ Tags can be removed via the `DROP TAG` sql
```sql
ALTER TABLE prod.db.sample DROP TAG `historical-tag`
```
+
+### I
liurenjie1024 commented on PR #234:
URL: https://github.com/apache/iceberg-rust/pull/234#issuecomment-1982616229
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 opened a new pull request, #234:
URL: https://github.com/apache/iceberg-rust/pull/234
(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 unsubscr
HonahX commented on issue #502:
URL: https://github.com/apache/iceberg-python/issues/502#issuecomment-1982550509
Hi @kevinjqliu @syun64 I think currently we can also set properties using
dictionaries, just need `**` in front of the dictionary:
```python
properties: Dict[str, str] = {..
bitsondatadev commented on code in PR #9878:
URL: https://github.com/apache/iceberg/pull/9878#discussion_r1514954244
##
docs/docs/spark-ddl.md:
##
@@ -566,3 +566,85 @@ Tags can be removed via the `DROP TAG` sql
```sql
ALTER TABLE prod.db.sample DROP TAG `historical-tag`
```
+
RussellSpitzer commented on PR #9803:
URL: https://github.com/apache/iceberg/pull/9803#issuecomment-1982473177
Thanks @himadripal for this great improvement, Our general policy is once
one Spark Version is in you open another PR to all of the other currently
supported Branches with the same
RussellSpitzer merged PR #9803:
URL: https://github.com/apache/iceberg/pull/9803
--
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...@iceb
RussellSpitzer commented on code in PR #9803:
URL: https://github.com/apache/iceberg/pull/9803#discussion_r1515587628
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRewriteDataFilesAction.java:
##
@@ -1463,6 +1465,130 @@ public void testSnapshotProperty()
ajantha-bhat commented on code in PR #9877:
URL: https://github.com/apache/iceberg/pull/9877#discussion_r1515586577
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java:
##
@@ -308,6 +308,10 @@ public List listNamespaces(Namespace namespace)
throws NoSuchN
ajantha-bhat commented on code in PR #9877:
URL: https://github.com/apache/iceberg/pull/9877#discussion_r1515586056
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java:
##
@@ -380,6 +389,10 @@ public boolean removeProperties(Namespace namespace,
Set prope
bryanck commented on code in PR #9641:
URL: https://github.com/apache/iceberg/pull/9641#discussion_r1515539847
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -0,0 +1,508 @@
+/*
+ * Licensed to the Apache Software Foundati
bryanck commented on code in PR #9641:
URL: https://github.com/apache/iceberg/pull/9641#discussion_r1515525033
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -0,0 +1,508 @@
+/*
+ * Licensed to the Apache Software Foundati
bryanck commented on code in PR #9641:
URL: https://github.com/apache/iceberg/pull/9641#discussion_r1515525033
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -0,0 +1,508 @@
+/*
+ * Licensed to the Apache Software Foundati
kevinjqliu opened a new pull request, #504:
URL: https://github.com/apache/iceberg-python/pull/504
Requires #503 to be merged first.
This PR modifies `Transaction`'s `set_properties` API to accept table
properties as type `Dict[str, Any]`, either as dictionary or kwargs.
For
odysa opened a new pull request, #233:
URL: https://github.com/apache/iceberg-rust/pull/233
Create a `runtime` module exposing functions like `spawn`
Example usage:
```rust
let tasks = manifest_list
.entries()
.iter()
.map(|manife
nk1506 commented on PR #9862:
URL: https://github.com/apache/iceberg/pull/9862#issuecomment-1982292262
> I would suggest to either focus on a subset of tests in `iceberg-core` or
on `iceberg-data`
@nastra , Most of the changes are coming here because of change in
`WriterTestBase` and
kevinjqliu opened a new pull request, #503:
URL: https://github.com/apache/iceberg-python/pull/503
Resolves #502
With this PR, table properties can be set with `set_properties` transaction
by passing in a dictionary
```
with table.transaction() as transaction:
transactio
liurenjie1024 merged PR #228:
URL: https://github.com/apache/iceberg-rust/pull/228
--
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
liurenjie1024 commented on issue #232:
URL: https://github.com/apache/iceberg-rust/issues/232#issuecomment-1982235386
> When `add_schema` has been called, both `schemas` and `current_schema_id`
will be updated? The same question to `partition_specs`.
Yes, the `add_schema` process cons
Xuanwo commented on issue #232:
URL: https://github.com/apache/iceberg-rust/issues/232#issuecomment-1982224663
When `add_schema` has been called, both `schemas` and `current_schema_id`
will be updated? The same question to `partition_specs`.
--
This is an automated message from the Apache
liurenjie1024 opened a new issue, #232:
URL: https://github.com/apache/iceberg-rust/issues/232
# Problem statement
`TableMetadataBuilder` is useful in modifying/creating `TableMetadata`, and
is a core data structure of transaction api. There are already some efforts to
create one usi
liurenjie1024 commented on issue #232:
URL: https://github.com/apache/iceberg-rust/issues/232#issuecomment-1982200217
cc @Xuanwo @ZENOTME @Fokko @JanKaul @y0psolo
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
hiloboy0119 commented on code in PR #8409:
URL: https://github.com/apache/iceberg/pull/8409#discussion_r1515373674
##
core/src/main/java/org/apache/iceberg/schema/UnionByNameVisitor.java:
##
@@ -90,6 +94,15 @@ public Boolean struct(
Types.NestedField field = field
tomtongue commented on PR #9849:
URL: https://github.com/apache/iceberg/pull/9849#issuecomment-1982188699
@nastra Thank you! Reflected the minor comment. If there's any part that I
should change, please let me know.
--
This is an automated message from the Apache Git Service.
To respond t
liurenjie1024 commented on issue #216:
URL: https://github.com/apache/iceberg-rust/issues/216#issuecomment-1982178258
Looks cool!
--
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
manuzhang commented on PR #9832:
URL: https://github.com/apache/iceberg/pull/9832#issuecomment-1982140277
Any more comments for 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 above to go to the specif
artyrian opened a new pull request, #9886:
URL: https://github.com/apache/iceberg/pull/9886
https://github.com/apache/iceberg/pull/9484
and there is also a confilict with jackson 2.15.4 and spark 3.5 on gradle
8.4:
`java.lang.IllegalArgumentException: Unsupported class file maj
artyrian closed pull request #9882: revert apache 9726
URL: https://github.com/apache/iceberg/pull/9882
--
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:
amogh-jahagirdar closed issue #482: Spark <> Iceberg bug integration test
URL: https://github.com/apache/iceberg-python/issues/482
--
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
amogh-jahagirdar merged PR #501:
URL: https://github.com/apache/iceberg-python/pull/501
--
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.
amogh-jahagirdar commented on PR #501:
URL: https://github.com/apache/iceberg-python/pull/501#issuecomment-1982123757
Sweet, thanks @kevinjqliu! I'm going to go ahead and merge this now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
syun64 commented on issue #502:
URL: https://github.com/apache/iceberg-python/issues/502#issuecomment-1982120843
I think this is a great idea @kevinjqliu . Most of the table properties have
`.` in the key anyways, and hence we can't pass them as kwargs anyways.
Changing the input parameter
syun64 commented on PR #501:
URL: https://github.com/apache/iceberg-python/pull/501#issuecomment-1982096927
Great idea @kevinjqliu ! Thanks for adding this
--
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
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1515180358
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,368 @@
+/*
+ * Licensed to the Apache Software Founda
hiloboy0119 commented on PR #8409:
URL: https://github.com/apache/iceberg/pull/8409#issuecomment-1982085238
@amogh-jahagirdar sorry for the long delay. Finally had time to write tests
for the various edge cases and fix a few things.
The examples of schema evolution are all shown in t
github-actions[bot] commented on issue #1914:
URL: https://github.com/apache/iceberg/issues/1914#issuecomment-1982079679
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 #1886:
URL: https://github.com/apache/iceberg/issues/1886#issuecomment-1982079661
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.
kevinjqliu commented on code in PR #501:
URL: https://github.com/apache/iceberg-python/pull/501#discussion_r1515301588
##
tests/integration/test_writes.py:
##
@@ -355,6 +355,26 @@ def test_data_files(spark: SparkSession, session_catalog:
Catalog, arrow_table_w
assert [row.
amogh-jahagirdar commented on code in PR #501:
URL: https://github.com/apache/iceberg-python/pull/501#discussion_r1515300598
##
tests/integration/test_writes.py:
##
@@ -355,6 +355,26 @@ def test_data_files(spark: SparkSession, session_catalog:
Catalog, arrow_table_w
assert
amogh-jahagirdar commented on PR #501:
URL: https://github.com/apache/iceberg-python/pull/501#issuecomment-1982063186
Thanks @kevinjqliu I think this change makes sense. I don't think there's
ever a reason on the Python side where we want to have the spark caching
enabled. On the Iceberg Ja
kevinjqliu commented on PR #444:
URL: https://github.com/apache/iceberg-python/pull/444#issuecomment-1982045743
thanks for the review @HonahX. I've rebased off `main` and addressed your
comments.
I also added more tests after figuring out the fix for #482
--
This is an automated m
caldempsey commented on issue #216:
URL: https://github.com/apache/iceberg-rust/issues/216#issuecomment-1982004560
--
This is an automated message from the Apache Git Service.
To respo
caldempsey commented on issue #216:
URL: https://github.com/apache/iceberg-rust/issues/216#issuecomment-1982002725
I'd prefer a letter I with the RUST gears surrounding it ⚙️
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
fqaiser94 commented on code in PR #9641:
URL: https://github.com/apache/iceberg/pull/9641#discussion_r1515260098
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -0,0 +1,508 @@
+/*
+ * Licensed to the Apache Software Founda
kevinjqliu commented on issue #502:
URL: https://github.com/apache/iceberg-python/issues/502#issuecomment-1981976531
Another workaround can be to use
```
with tbl.transaction() as transaction:
transaction._apply((SetPropertiesUpdate(updates=properties),))
```
--
This i
kevinjqliu commented on code in PR #444:
URL: https://github.com/apache/iceberg-python/pull/444#discussion_r1515227170
##
tests/integration/test_writes.py:
##
@@ -424,6 +424,30 @@ def test_data_files(spark: SparkSession, session_catalog:
Catalog, arrow_table_w
assert [row.
leonidmakarovsky commented on issue #500:
URL: https://github.com/apache/iceberg-python/issues/500#issuecomment-1981884781
My python version is 3.9.7. My colleague told me that pyiceberg works with
3.9.11 and later. It didn't work with 3.9.7 in his env as well. I'm trying to
upgrade Python
kevinjqliu commented on issue #500:
URL: https://github.com/apache/iceberg-python/issues/500#issuecomment-1981880634
seems like an issue related to `pydantic` library. What version of it do you
have installed?
Here's my env setup using `make install`:
```
(venv) ➜ iceberg-pyth
amogh-jahagirdar opened a new pull request, #9885:
URL: https://github.com/apache/iceberg/pull/9885
Bad gateway or gateway timeouts should be able to be safely retried. A bad
gateway could be caused by a load balancer or proxy between the client and the
actual server, when there's some netw
jaychia commented on code in PR #9836:
URL: https://github.com/apache/iceberg/pull/9836#discussion_r1515142038
##
docs/docs/daft.md:
##
@@ -0,0 +1,146 @@
+---
+title: "Daft"
+---
+
+
+# Daft
Review Comment:
I was able to successfully build the site with `mkdocs serve`!
``
jaychia commented on code in PR #9836:
URL: https://github.com/apache/iceberg/pull/9836#discussion_r1515139832
##
docs/docs/daft.md:
##
@@ -0,0 +1,146 @@
+---
+title: "Daft"
+---
+
+
+# Daft
+
+[Daft](www.getdaft.io) is a distributed query engine written in Python and
Rust, two
jaychia commented on code in PR #9836:
URL: https://github.com/apache/iceberg/pull/9836#discussion_r1515138488
##
docs/docs/daft.md:
##
@@ -0,0 +1,146 @@
+---
+title: "Daft"
+---
+
+
+# Daft
+
+[Daft](www.getdaft.io) is a distributed query engine written in Python and
Rust, two
himadripal commented on PR #9839:
URL: https://github.com/apache/iceberg/pull/9839#issuecomment-1981669062
Does all of these optional parameters need to be added in the response as
well? @flyrain
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
himadripal commented on code in PR #9839:
URL: https://github.com/apache/iceberg/pull/9839#discussion_r1514845097
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -178,15 +178,18 @@ public void initialize(String name, Map
unresolved) {
ConfigRes
leonidmakarovsky opened a new issue, #500:
URL: https://github.com/apache/iceberg-python/issues/500
### Apache Iceberg version
0.6.0 (latest release)
### Please describe the bug 🐞
Running on Mac OS Monterey 12.6.7, Spyder IDE (Python 3.9.7 64-bit | Qt
5.9.7 | PyQt5 5.9.2
flyrain commented on issue #479:
URL: https://github.com/apache/iceberg-python/issues/479#issuecomment-1981523086
This is fixed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comme
flyrain closed issue #479: Make the OAuth2 request audience configurable
URL: https://github.com/apache/iceberg-python/issues/479
--
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.
flyrain merged PR #486:
URL: https://github.com/apache/iceberg-python/pull/486
--
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
himadripal commented on code in PR #9839:
URL: https://github.com/apache/iceberg/pull/9839#discussion_r1514955976
##
aws/src/main/java/org/apache/iceberg/aws/s3/signer/S3V4RestSignerClient.java:
##
@@ -207,7 +212,13 @@ private AuthSession authSession() {
t
jbonofre commented on code in PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#discussion_r1514932065
##
core/src/main/java/org/apache/iceberg/io/ResolvingFileIO.java:
##
@@ -122,11 +124,20 @@ public void initialize(Map newProperties)
{
close(); // close and disc
jbonofre commented on code in PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#discussion_r1514931349
##
core/src/main/java/org/apache/iceberg/io/ResolvingFileIO.java:
##
@@ -219,7 +230,7 @@ DelegateFileIO io(String location) {
@VisibleForTesting
String implFro
ajantha-bhat commented on code in PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#discussion_r1514921889
##
core/src/main/java/org/apache/iceberg/io/ResolvingFileIO.java:
##
@@ -122,11 +124,20 @@ public void initialize(Map newProperties)
{
close(); // close and
ajantha-bhat commented on code in PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#discussion_r1514921889
##
core/src/main/java/org/apache/iceberg/io/ResolvingFileIO.java:
##
@@ -122,11 +124,20 @@ public void initialize(Map newProperties)
{
close(); // close and
zachdisc commented on PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#issuecomment-1981412777
R4
Renamed `sort` to `clusterBy` - more honest and aligns with the core API as
well.
Have not made progress on using the custom UDF for both standard partition
column ord
nastra commented on code in PR #9849:
URL: https://github.com/apache/iceberg/pull/9849#discussion_r1514880737
##
core/src/test/java/org/apache/iceberg/TestFindFiles.java:
##
@@ -184,33 +181,33 @@ public void testCaseSensitivity() {
.withMetadataMatching(Expressions.
ajantha-bhat commented on code in PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#discussion_r1514874407
##
core/src/main/java/org/apache/iceberg/io/ResolvingFileIO.java:
##
@@ -219,7 +230,7 @@ DelegateFileIO io(String location) {
@VisibleForTesting
String imp
ajantha-bhat commented on code in PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#discussion_r1514871547
##
core/src/main/java/org/apache/iceberg/io/ResolvingFileIO.java:
##
@@ -219,7 +230,7 @@ DelegateFileIO io(String location) {
@VisibleForTesting
String imp
tomtongue commented on PR #9849:
URL: https://github.com/apache/iceberg/pull/9849#issuecomment-1981376570
@nastra Update with your comments. When you have time, could you review the
new commit?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
himadripal commented on code in PR #9839:
URL: https://github.com/apache/iceberg/pull/9839#discussion_r1514845097
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -178,15 +178,18 @@ public void initialize(String name, Map
unresolved) {
ConfigRes
himadripal commented on code in PR #9839:
URL: https://github.com/apache/iceberg/pull/9839#discussion_r1514845097
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -178,15 +178,18 @@ public void initialize(String name, Map
unresolved) {
ConfigRes
jbonofre commented on PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#issuecomment-1981347201
@ajantha-bhat I'm fixing yeah.
--
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 specifi
ajantha-bhat commented on PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#issuecomment-1981329539
Testcase failed, probably because of `schemeToFileIo.clear(); `
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
himadripal commented on PR #486:
URL: https://github.com/apache/iceberg-python/pull/486#issuecomment-1981317217
@flyrain and @Fokko thank you for reviewing this. Please take a look again
when you can.
--
This is an automated message from the Apache Git Service.
To respond to the message
jbonofre commented on PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#issuecomment-1981255063
@ajantha-bhat or also new IO implementations, interesting in Europe 😄 (like
wasabi, leviia, ovh store, scaleway store, ...)
--
This is an automated message from the Apache Git Service
ajantha-bhat commented on code in PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#discussion_r1514759756
##
core/src/main/java/org/apache/iceberg/io/ResolvingFileIO.java:
##
@@ -122,6 +124,15 @@ public void initialize(Map newProperties) {
close(); // close and di
tomtongue commented on code in PR #9849:
URL: https://github.com/apache/iceberg/pull/9849#discussion_r1514762594
##
core/src/test/java/org/apache/iceberg/TestMetadataTableScans.java:
##
@@ -1357,27 +1344,27 @@ private void testPositionDeletesBaseTableFilter(boolean
transactiona
tomtongue commented on code in PR #9849:
URL: https://github.com/apache/iceberg/pull/9849#discussion_r1514762022
##
core/src/test/java/org/apache/iceberg/TestMetadataTableScans.java:
##
@@ -1357,27 +1344,27 @@ private void testPositionDeletesBaseTableFilter(boolean
transactiona
adutra commented on code in PR #9884:
URL: https://github.com/apache/iceberg/pull/9884#discussion_r1514756075
##
core/src/main/java/org/apache/iceberg/io/ResolvingFileIO.java:
##
@@ -53,20 +53,22 @@ public class ResolvingFileIO implements HadoopConfigurable,
DelegateFileIO {
jbonofre opened a new pull request, #9884:
URL: https://github.com/apache/iceberg/pull/9884
This PR adds a way to load custom schemes (in addition of the default ones)
in `ResolvingFileIO` using properties prefixed with `resolving-io.schemes.`.
This closes #9883
--
This is an
nastra commented on PR #9882:
URL: https://github.com/apache/iceberg/pull/9882#issuecomment-1981000150
@artyrian can you please add some context why this revert is necessary?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
artyrian opened a new pull request, #9882:
URL: https://github.com/apache/iceberg/pull/9882
(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
RussellSpitzer commented on issue #9881:
URL: https://github.com/apache/iceberg/issues/9881#issuecomment-1980957406
Could you share the query as well as the table schema?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
nastra commented on code in PR #9877:
URL: https://github.com/apache/iceberg/pull/9877#discussion_r1514509086
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java:
##
@@ -308,6 +308,10 @@ public List listNamespaces(Namespace namespace)
throws NoSuchNamespa
nastra commented on code in PR #9877:
URL: https://github.com/apache/iceberg/pull/9877#discussion_r1514502547
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java:
##
@@ -380,6 +389,10 @@ public boolean removeProperties(Namespace namespace,
Set properties)
ajantha-bhat commented on code in PR #9877:
URL: https://github.com/apache/iceberg/pull/9877#discussion_r1514493828
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java:
##
@@ -380,6 +389,10 @@ public boolean removeProperties(Namespace namespace,
Set prope
ajantha-bhat commented on code in PR #9877:
URL: https://github.com/apache/iceberg/pull/9877#discussion_r1514489991
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java:
##
@@ -308,6 +308,10 @@ public List listNamespaces(Namespace namespace)
throws NoSuchN
burriad opened a new issue, #9881:
URL: https://github.com/apache/iceberg/issues/9881
### Apache Iceberg version
1.4.3 (latest release)
### Query engine
Spark
### Please describe the bug 🐞
When upgrading from Iceberg version 1.3.1 to 1.4.3 we get the followi
adutra commented on code in PR #9877:
URL: https://github.com/apache/iceberg/pull/9877#discussion_r1514477117
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java:
##
@@ -308,6 +308,10 @@ public List listNamespaces(Namespace namespace)
throws NoSuchNamespa
ajantha-bhat commented on issue #9880:
URL: https://github.com/apache/iceberg/issues/9880#issuecomment-1980862061
maybe cc: @nastra, @danielcweeks, @rdblue
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
1 - 100 of 137 matches
Mail list logo
