odysa commented on issue #177:
URL: https://github.com/apache/iceberg-rust/issues/177#issuecomment-1925188382
> The code expect()s current_schema_id should not be None
`current_schema_id` is impossible to be `None` in `TableMetadata`
> But it is valid to be None in TableMetadata
aokolnychyi commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476993584
##
core/src/main/java/org/apache/iceberg/PartitionData.java:
##
@@ -171,6 +169,10 @@ public PartitionData copy() {
return new PartitionData(this);
}
+ pu
aokolnychyi commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476991582
##
core/src/main/java/org/apache/iceberg/PartitionData.java:
##
@@ -171,6 +169,10 @@ public PartitionData copy() {
return new PartitionData(this);
}
+ pu
zeodtr commented on issue #177:
URL: https://github.com/apache/iceberg-rust/issues/177#issuecomment-1925160320
In the following code, from:
https://github.com/apache/iceberg-rust/blob/9ae9e13fb48ea8af20d76644f27dcb2fc8773396/crates/iceberg/src/spec/table_metadata.rs#L679
```rust
ZENOTME commented on code in PR #176:
URL: https://github.com/apache/iceberg-rust/pull/176#discussion_r1476988389
##
crates/iceberg/src/writer/file_writer/mod.rs:
##
@@ -18,16 +18,20 @@
//! This module contains the writer for data file format supported by iceberg:
parquet, orc
odysa opened a new pull request, #185:
URL: https://github.com/apache/iceberg-rust/pull/185
Add serialize for `TableMetadata` and use `TryFrom` as discussed in #177
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
jbonofre commented on PR #8486:
URL: https://github.com/apache/iceberg/pull/8486#issuecomment-1925103805
As gradle 8.6 is available, I will resume this PR upgrading to 8.6.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
stevenzwu commented on issue #9597:
URL: https://github.com/apache/iceberg/issues/9597#issuecomment-1925055682
yeah. we are on the same page now. `TaskType` enum can be defined in
`FileScanTaskParser`.
regarding the `StructLike` row serialization, I am thinking maybe we should
implem
liurenjie1024 commented on code in PR #176:
URL: https://github.com/apache/iceberg-rust/pull/176#discussion_r1476921216
##
crates/iceberg/src/writer/file_writer/mod.rs:
##
@@ -18,16 +18,20 @@
//! This module contains the writer for data file format supported by iceberg:
parque
abmo-x commented on code in PR #9619:
URL: https://github.com/apache/iceberg/pull/9619#discussion_r1476934235
##
core/src/main/java/org/apache/iceberg/hadoop/HadoopCatalog.java:
##
@@ -108,7 +108,10 @@ public void initialize(String name, Map
properties) {
"Cannot initi
liurenjie1024 commented on issue #177:
URL: https://github.com/apache/iceberg-rust/issues/177#issuecomment-1925047108
I remember why we use `expect` for `TableMetadata` serialization, see [this
comment](https://github.com/apache/iceberg-rust/blob/c91aeaec2aa713a1efdc513e1769220dd53cf443/crat
odysa commented on issue #177:
URL: https://github.com/apache/iceberg-rust/issues/177#issuecomment-1925037344
> We may need to modify the From to TryFrom.
`serde` does not support the attribute `try_into`.
https://github.com/serde-rs/serde/issues/1524 The `into` attribute requires
`
liurenjie1024 commented on issue #124:
URL: https://github.com/apache/iceberg-rust/issues/124#issuecomment-1925022721
> Do you want users to choose their own runtime like
[sqlx](https://github.com/launchbadge/sqlx/tree/main#install)?
Yes, exactly. I don't think we should bind to some
liurenjie1024 commented on issue #177:
URL: https://github.com/apache/iceberg-rust/issues/177#issuecomment-1925021474
@zeodtr Sorry for the mistake, yes you are right, expect still may cause
panic. Let's reopen it. cc @odysa We may need to modify the `From` to `TryFrom`.
--
This is an au
aokolnychyi commented on code in PR #9563:
URL: https://github.com/apache/iceberg/pull/9563#discussion_r1476914683
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/source/SparkPlanningUtil.java:
##
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
epgif commented on PR #9008:
URL: https://github.com/apache/iceberg/pull/9008#issuecomment-1924975062
https://github.com/apache/iceberg/actions/runs/7717197684/job/21172136322?pr=9008
looks like a spurious failure? All the rest passed, and even
`spark-3x-java-17-tests (3.5, 2.13)` pass
aokolnychyi commented on code in PR #9563:
URL: https://github.com/apache/iceberg/pull/9563#discussion_r1476899638
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkReadConf.java:
##
@@ -331,4 +331,24 @@ private long driverMaxResultSize() {
SparkConf sparkConf
aokolnychyi commented on code in PR #9563:
URL: https://github.com/apache/iceberg/pull/9563#discussion_r1476899638
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkReadConf.java:
##
@@ -331,4 +331,24 @@ private long driverMaxResultSize() {
SparkConf sparkConf
aokolnychyi commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476895581
##
core/src/main/java/org/apache/iceberg/util/TableScanUtil.java:
##
@@ -188,23 +189,6 @@ public static
List> planTaskGroup
return taskGroups;
}
- priv
amogh-jahagirdar commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476890643
##
core/src/main/java/org/apache/iceberg/util/TableScanUtil.java:
##
@@ -188,23 +189,6 @@ public static
List> planTaskGroup
return taskGroups;
}
-
amogh-jahagirdar commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476890643
##
core/src/main/java/org/apache/iceberg/util/TableScanUtil.java:
##
@@ -188,23 +189,6 @@ public static
List> planTaskGroup
return taskGroups;
}
-
github-actions[bot] closed issue #257: Have tests which test against the shaded
runtime artifacts
URL: https://github.com/apache/iceberg/issues/257
--
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
github-actions[bot] commented on issue #189:
URL: https://github.com/apache/iceberg/issues/189#issuecomment-1924927624
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 Git
github-actions[bot] closed issue #189: Allow reading non-optional unions as
struct of optional fields
URL: https://github.com/apache/iceberg/issues/189
--
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 t
github-actions[bot] commented on issue #168:
URL: https://github.com/apache/iceberg/issues/168#issuecomment-1924927598
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 Git
github-actions[bot] closed issue #128: Iceberg Table snapshots/manifests using
relative path fails to read data
URL: https://github.com/apache/iceberg/issues/128
--
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
github-actions[bot] commented on issue #797:
URL: https://github.com/apache/iceberg/issues/797#issuecomment-1924927799
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. T
github-actions[bot] commented on issue #791:
URL: https://github.com/apache/iceberg/issues/791#issuecomment-1924927780
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. T
github-actions[bot] commented on issue #257:
URL: https://github.com/apache/iceberg/issues/257#issuecomment-1924927637
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 Git
github-actions[bot] commented on issue #128:
URL: https://github.com/apache/iceberg/issues/128#issuecomment-1924927573
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 Git
github-actions[bot] closed issue #168: Large number of external java packages
are not relocated in iceberg-runtime.jar and iceberg-presto-runtime.jar
URL: https://github.com/apache/iceberg/issues/168
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
tomtongue commented on PR #9624:
URL: https://github.com/apache/iceberg/pull/9624#issuecomment-1924917685
Sure, thank you for the suggestion. The size of diff would be a bit big if I
gather all the diffs in `extension` package in this PR. So, I will separately
create PRs, but first add rele
zeodtr commented on PR #183:
URL: https://github.com/apache/iceberg-rust/pull/183#issuecomment-1924916796
@odysa @liurenjie1024 @Xuanwo @Fokko Please see my comment:
https://github.com/apache/iceberg-rust/issues/177#issuecomment-1924916131
Thank you.
--
This is an automated message fro
zeodtr commented on issue #177:
URL: https://github.com/apache/iceberg-rust/issues/177#issuecomment-1924916131
@Fokko @liurenjie1024 I think this issue has not been resolved, since I'm
kind of disagree with the pull request.
What I'm concerned about are as follows:
* `expect()` also s
rdblue commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476874138
##
core/src/main/java/org/apache/iceberg/PartitionData.java:
##
@@ -171,6 +169,10 @@ public PartitionData copy() {
return new PartitionData(this);
}
+ public
aokolnychyi commented on code in PR #9603:
URL: https://github.com/apache/iceberg/pull/9603#discussion_r1476868911
##
spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/source/BaseReader.java:
##
@@ -279,5 +284,29 @@ protected void markRowDeleted(InternalRow row) {
aokolnychyi merged PR #9603:
URL: https://github.com/apache/iceberg/pull/9603
--
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 code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476864901
##
core/src/main/java/org/apache/iceberg/PartitionData.java:
##
@@ -171,6 +169,10 @@ public PartitionData copy() {
return new PartitionData(this);
}
+ pu
aokolnychyi commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476864901
##
core/src/main/java/org/apache/iceberg/PartitionData.java:
##
@@ -171,6 +169,10 @@ public PartitionData copy() {
return new PartitionData(this);
}
+ pu
aokolnychyi commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476862386
##
core/src/main/java/org/apache/iceberg/PartitionData.java:
##
@@ -221,4 +223,32 @@ public static Object[] copyData(Types.StructType type,
Object[] data) {
aokolnychyi commented on code in PR #9629:
URL: https://github.com/apache/iceberg/pull/9629#discussion_r1476863172
##
core/src/main/java/org/apache/iceberg/util/TableScanUtil.java:
##
@@ -188,23 +189,6 @@ public static
List> planTaskGroup
return taskGroups;
}
- priv
danielcweeks commented on code in PR #9628:
URL: https://github.com/apache/iceberg/pull/9628#discussion_r1476859199
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1453,6 +1456,23 @@ components:
type: string
example: "sales"
+data-access:
+ name: X-Iceb
danielcweeks commented on code in PR #9628:
URL: https://github.com/apache/iceberg/pull/9628#discussion_r1476857183
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1453,6 +1456,23 @@ components:
type: string
example: "sales"
+data-access:
+ name: X-Iceb
danielcweeks merged PR #9622:
URL: https://github.com/apache/iceberg/pull/9622
--
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 merged PR #309:
URL: https://github.com/apache/iceberg-docs/pull/309
--
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...@ice
rdblue commented on code in PR #9628:
URL: https://github.com/apache/iceberg/pull/9628#discussion_r1476851450
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1453,6 +1456,23 @@ components:
type: string
example: "sales"
+data-access:
+ name: X-Iceberg-Da
rdblue commented on code in PR #9628:
URL: https://github.com/apache/iceberg/pull/9628#discussion_r1476850002
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1453,6 +1456,23 @@ components:
type: string
example: "sales"
+data-access:
+ name: X-Iceberg-Da
bitsondatadev opened a new pull request, #309:
URL: https://github.com/apache/iceberg-docs/pull/309
(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
aokolnychyi commented on code in PR #9563:
URL: https://github.com/apache/iceberg/pull/9563#discussion_r1476795233
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/source/SparkPlanningUtil.java:
##
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
aokolnychyi commented on code in PR #9563:
URL: https://github.com/apache/iceberg/pull/9563#discussion_r1476795233
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/source/SparkPlanningUtil.java:
##
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
jaychia opened a new pull request, #355:
URL: https://github.com/apache/iceberg-python/pull/355
1. Adds a new optional installation arg `daft`, so that `pip install
pyiceberg[daft]` will pull Daft in as a dependency
2. Adds a new `Table.to_daft()` method to convert a table into a Daft
da
odysa commented on issue #124:
URL: https://github.com/apache/iceberg-rust/issues/124#issuecomment-1924672646
> Do you want users to choose their own runtime like
[sqlx](https://github.com/launchbadge/sqlx/tree/main?rgh-link-date=2024-02-02T17%3A02%3A32Z#install)?
They are building an abstr
jacobmarble commented on issue #8657:
URL: https://github.com/apache/iceberg/issues/8657#issuecomment-1924559864
Maintainers: please also assign @epgif
--
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 g
amogh-jahagirdar commented on code in PR #9603:
URL: https://github.com/apache/iceberg/pull/9603#discussion_r1476567943
##
spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/source/BaseReader.java:
##
@@ -279,5 +284,29 @@ protected void markRowDeleted(InternalRow row) {
amogh-jahagirdar commented on code in PR #9603:
URL: https://github.com/apache/iceberg/pull/9603#discussion_r1476567943
##
spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/source/BaseReader.java:
##
@@ -279,5 +284,29 @@ protected void markRowDeleted(InternalRow row) {
szehon-ho commented on PR #8583:
URL: https://github.com/apache/iceberg/pull/8583#issuecomment-1924504708
Merged, thanks @dramaticlly , and thanks all for additional reviews.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
szehon-ho closed issue #8582: Switch to using ProcedureInput for
rewriteDataFiles
URL: https://github.com/apache/iceberg/issues/8582
--
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
szehon-ho merged PR #8583:
URL: https://github.com/apache/iceberg/pull/8583
--
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.a
rodmeneses opened a new pull request, #9627:
URL: https://github.com/apache/iceberg/pull/9627
1.17 came out clean
1.16 came out clean, after adding couple extra lines on `TestStreamScanSql`
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
pvary merged PR #9625:
URL: https://github.com/apache/iceberg/pull/9625
--
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
jqin61 commented on code in PR #353:
URL: https://github.com/apache/iceberg-python/pull/353#discussion_r1476487793
##
pyiceberg/table/__init__.py:
##
@@ -2467,3 +2462,131 @@ def commit(self) -> Snapshot:
)
return snapshot
+
+
+@dataclass(frozen=True)
+cla
szehon-ho commented on code in PR #9603:
URL: https://github.com/apache/iceberg/pull/9603#discussion_r1476484200
##
spark/v3.4/spark/src/test/java/org/apache/iceberg/spark/TestSparkExecutorCache.java:
##
@@ -0,0 +1,444 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
jbonofre commented on PR #9487:
URL: https://github.com/apache/iceberg/pull/9487#issuecomment-1924405491
@nastra @danielcweeks @rdblue I updated the PR. You can already do a new
round (I'm checking a couple of stuff but ready to review already).
--
This is an automated message from the Ap
rdblue commented on issue #7938:
URL: https://github.com/apache/iceberg/issues/7938#issuecomment-1924377622
I think we can call this one done with all of the PRs from @nastra that
we've merged lately. Thanks @jzhuge and @nastra for getting this ready!
--
This is an automated message from
rdblue closed issue #7938: Add View Support to Spark
URL: https://github.com/apache/iceberg/issues/7938
--
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:
jbonofre commented on code in PR #9487:
URL: https://github.com/apache/iceberg/pull/9487#discussion_r1475826840
##
core/src/main/java/org/apache/iceberg/jdbc/JdbcTableOperations.java:
##
@@ -182,18 +169,13 @@ private void createTable(String newMetadataLocation)
throws SQLExcept
jbonofre commented on code in PR #9487:
URL: https://github.com/apache/iceberg/pull/9487#discussion_r1475821544
##
core/src/main/java/org/apache/iceberg/jdbc/JdbcCatalog.java:
##
@@ -503,6 +550,84 @@ public boolean namespaceExists(Namespace namespace) {
return JdbcUtil.name
aokolnychyi commented on code in PR #9563:
URL: https://github.com/apache/iceberg/pull/9563#discussion_r1476443719
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/source/SparkPlanningUtil.java:
##
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
nastra commented on PR #9624:
URL: https://github.com/apache/iceberg/pull/9624#issuecomment-1924340442
> Will add all sub-classes in this PR
depending on the size of the diff it's also fine to split this into 2-3 PRs.
You could probably start within a specific package and combine subc
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476417816
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
nastra commented on issue #9597:
URL: https://github.com/apache/iceberg/issues/9597#issuecomment-1924328488
@stevenzwu this is only because `ReportMetricsRequest` is a REST request
class for a `MetricsReport`. So in the case of this issue here we'd define the
enum type at the JSON level in
odysa commented on issue #124:
URL: https://github.com/apache/iceberg-rust/issues/124#issuecomment-1924284690
> I mean we may need an extra layer for task scheduling, so that we can be
adopted to any async runtime such as tokio, async-std.
Do you want users to choose their own runtime
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476325368
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476307388
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
syun64 commented on code in PR #353:
URL: https://github.com/apache/iceberg-python/pull/353#discussion_r1476291241
##
pyiceberg/table/__init__.py:
##
@@ -2467,3 +2462,131 @@ def commit(self) -> Snapshot:
)
return snapshot
+
+
+@dataclass(frozen=True)
+cla
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476289111
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476285119
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476280534
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476276022
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
kevinjqliu commented on issue #326:
URL: https://github.com/apache/iceberg-python/issues/326#issuecomment-1924204635
++, this lowers the barrier to entry by a lot. It's a lot of work to spin up
docker/s3/minio integration and hive 😱!
--
This is an automated message from the Apache Git Ser
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476270090
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
bitsondatadev commented on issue #326:
URL: https://github.com/apache/iceberg-python/issues/326#issuecomment-1924196245
Yeah I want to sprinkle this literally everywhere in the docs so please go
for it. I think this will be my preferred way of teaching Iceberg.
--
This is an automated mes
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476264616
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/SchemaUtils.java:
##
@@ -0,0 +1,375 @@
+/*
+ * Licensed to the Apache Software Foundation (
rdblue commented on PR #6981:
URL: https://github.com/apache/iceberg/pull/6981#issuecomment-1924192337
Oops, I didn't mean to close this! I want to work on getting it in next
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
huaxingao opened a new pull request, #6981:
URL: https://github.com/apache/iceberg/pull/6981
Push down min/max/count with group by if group by is on partition columns
For example:
```
CREATE TABLE test (id LONG, ts TIMESTAMP, data INT) USING iceberg
PARTITIONED BY (id, ts);
S
rdblue closed pull request #6981: Push down group by for partition columns
URL: https://github.com/apache/iceberg/pull/6981
--
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
rdblue commented on PR #6981:
URL: https://github.com/apache/iceberg/pull/6981#issuecomment-1924191483
@amogh-jahagirdar, can you
--
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
rdblue merged PR #9621:
URL: https://github.com/apache/iceberg/pull/9621
--
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
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476258875
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/IcebergWriterFactory.java:
##
@@ -0,0 +1,112 @@
+/*
+ * Licensed to the Apache Software Fou
bryanck commented on code in PR #9466:
URL: https://github.com/apache/iceberg/pull/9466#discussion_r1476250156
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/IcebergWriter.java:
##
@@ -0,0 +1,118 @@
+/*
+ * Licensed to the Apache Software Foundation
kevinjqliu commented on issue #341:
URL: https://github.com/apache/iceberg-python/issues/341#issuecomment-1924165256
So if I understand correctly, TableIdentifier consists of the namespace and
the table name. Namespace can be multiple parts, for example
("com"."apache"."iceberg") or "com.ap
idrissa-mgs commented on issue #5965:
URL: https://github.com/apache/iceberg/issues/5965#issuecomment-1924162113
@vshel Did you finally find a long term solution rather than asking for an
increase of the aws soft limit on tables versionning ?
Did the skipArchive flag help ?
--
This is
kevinjqliu commented on issue #326:
URL: https://github.com/apache/iceberg-python/issues/326#issuecomment-1924154815
that's a great idea!
I'm thinking of adding this in [Getting started with
PyIceberg](https://github.com/apache/iceberg-python/blob/main/mkdocs/docs/index.md)
WDYT?
stevenzwu commented on issue #9597:
URL: https://github.com/apache/iceberg/issues/9597#issuecomment-1924110032
@nastra `ReportMetricsRequest` has the type at API level.
```
ReportType reportType();
```
--
This is an automated message from the Apache Git Service.
To respond to t
jbonofre commented on code in PR #9487:
URL: https://github.com/apache/iceberg/pull/9487#discussion_r1476197844
##
core/src/main/java/org/apache/iceberg/jdbc/JdbcCatalog.java:
##
@@ -503,6 +550,84 @@ public boolean namespaceExists(Namespace namespace) {
return JdbcUtil.name
amogh-jahagirdar commented on code in PR #245:
URL: https://github.com/apache/iceberg-python/pull/245#discussion_r1476195413
##
pyiceberg/table/__init__.py:
##
@@ -2271,3 +2331,242 @@ def commit(self) -> Snapshot:
)
return snapshot
+
+
+class UpdateSpec:
nastra commented on code in PR #9596:
URL: https://github.com/apache/iceberg/pull/9596#discussion_r1476195226
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkCatalog.java:
##
@@ -608,6 +608,53 @@ public View createView(
"Creating a view is not supported
amogh-jahagirdar commented on code in PR #245:
URL: https://github.com/apache/iceberg-python/pull/245#discussion_r1476173833
##
pyiceberg/table/__init__.py:
##
@@ -2271,3 +2331,242 @@ def commit(self) -> Snapshot:
)
return snapshot
+
+
+class UpdateSpec:
nastra commented on code in PR #9487:
URL: https://github.com/apache/iceberg/pull/9487#discussion_r1476145516
##
core/src/main/java/org/apache/iceberg/jdbc/JdbcCatalog.java:
##
@@ -503,6 +550,84 @@ public boolean namespaceExists(Namespace namespace) {
return JdbcUtil.namesp
wooyeong commented on code in PR #9455:
URL: https://github.com/apache/iceberg/pull/9455#discussion_r1476145477
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/source/SparkTable.java:
##
@@ -131,12 +133,12 @@ public SparkTable(Table icebergTable, boolean
refreshEager
1 - 100 of 141 matches
Mail list logo
