Xuanwo commented on PR #203:
URL: https://github.com/apache/iceberg-rust/pull/203#issuecomment-1960788671
> @Xuanwo I'm not following this reasoning. Should we just get this in?
Yep, let's go.
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500225930
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -250,12 +282,40 @@ private List writeUnpartitionedManifest
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500223758
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -250,12 +282,40 @@ private List writeUnpartitionedManifest
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500220133
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -160,6 +164,34 @@ public RewriteManifestsSparkAction stagi
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500219507
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -250,12 +282,40 @@ private List writeUnpartitionedManifest
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500219305
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -160,6 +164,34 @@ public RewriteManifestsSparkAction stagi
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500218491
##
api/src/main/java/org/apache/iceberg/actions/RewriteManifests.java:
##
@@ -44,6 +45,20 @@ public interface RewriteManifests
*/
RewriteManifests rewriteIf(P
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500218118
##
api/src/main/java/org/apache/iceberg/actions/RewriteManifests.java:
##
@@ -44,6 +45,20 @@ public interface RewriteManifests
*/
RewriteManifests rewriteIf(P
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500217705
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -90,13 +94,13 @@ public class RewriteManifestsSparkAction
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1500217411
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteManifestsSparkAction.java:
##
@@ -90,13 +94,13 @@ public class RewriteManifestsSparkAction
jackye1995 commented on PR #9783:
URL: https://github.com/apache/iceberg/pull/9783#issuecomment-1960741059
Side note, we should also deprecate the catalog in python, probably in a
separated PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
Xuanwo commented on code in PR #219:
URL: https://github.com/apache/iceberg-rust/pull/219#discussion_r1500172102
##
website/src/download.md:
##
@@ -17,3 +17,45 @@
~ under the License.
-->
+# Apache Iceberg-Rustâ„¢ Downloads
+
+The official Apache Iceberg-Rust releases are pr
zachdisc commented on PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#issuecomment-1960673099
## Rev 3
* Fixed styling which caused CI/CD to fail
* Changed the clustering column to be a struct referencing the actual
columns instead of a contacted string `p1::p2::p3`.
kevinjqliu commented on code in PR #444:
URL: https://github.com/apache/iceberg-python/pull/444#discussion_r1499783029
##
pyiceberg/io/pyarrow.py:
##
@@ -1715,53 +1715,65 @@ def fill_parquet_file_metadata(
def write_file(table: Table, tasks: Iterator[WriteTask]) -> Iterator
zachdisc commented on PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#issuecomment-1960639662
> Looks like the CI test failed, @zachdisc could you take a look and fix?
>
> Overall, we have the 2 options based on this thread: [#9731
(comment)](https://github.com/apache/icebe
manuzhang commented on PR #9779:
URL: https://github.com/apache/iceberg/pull/9779#issuecomment-1960631535
@bitsondatadev @nastra please help review
--
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
rahil-c commented on code in PR #9660:
URL: https://github.com/apache/iceberg/pull/9660#discussion_r1500140547
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1581,6 +1607,17 @@ components:
type: string
example: [ "accounting", "tax" ]
+PageToken:
+ des
ajantha-bhat commented on code in PR #9783:
URL: https://github.com/apache/iceberg/pull/9783#discussion_r1500113028
##
aws/src/main/java/org/apache/iceberg/aws/dynamodb/DynamoDbTableOperations.java:
##
@@ -44,6 +44,8 @@
import software.amazon.awssdk.services.dynamodb.model.PutI
ajantha-bhat commented on code in PR #9783:
URL: https://github.com/apache/iceberg/pull/9783#discussion_r1500113608
##
aws/src/main/java/org/apache/iceberg/aws/dynamodb/DynamoDbCatalog.java:
##
@@ -84,7 +84,12 @@
import
software.amazon.awssdk.services.dynamodb.model.TransactWr
ajantha-bhat commented on code in PR #9783:
URL: https://github.com/apache/iceberg/pull/9783#discussion_r1500112881
##
aws/src/main/java/org/apache/iceberg/aws/dynamodb/DynamoDbCatalog.java:
##
@@ -84,7 +84,12 @@
import
software.amazon.awssdk.services.dynamodb.model.TransactWr
ajantha-bhat commented on code in PR #9783:
URL: https://github.com/apache/iceberg/pull/9783#discussion_r1500113028
##
aws/src/main/java/org/apache/iceberg/aws/dynamodb/DynamoDbTableOperations.java:
##
@@ -44,6 +44,8 @@
import software.amazon.awssdk.services.dynamodb.model.PutI
ajantha-bhat commented on code in PR #9783:
URL: https://github.com/apache/iceberg/pull/9783#discussion_r1500112881
##
aws/src/main/java/org/apache/iceberg/aws/dynamodb/DynamoDbCatalog.java:
##
@@ -84,7 +84,12 @@
import
software.amazon.awssdk.services.dynamodb.model.TransactWr
syun64 commented on issue #464:
URL: https://github.com/apache/iceberg-python/issues/464#issuecomment-1960585674
> 1. Client sends id tokens and resource to the authorization server to get
access token.
>
> 2. Client sends access tokens to access a resource.
I think you describ
flyrain commented on issue #464:
URL: https://github.com/apache/iceberg-python/issues/464#issuecomment-1960559522
That's definitely true. That's actually what id token and access token are
designed for. In that case, the authorization and authentication are separated
calls.
1. Client se
aokolnychyi commented on code in PR #9611:
URL: https://github.com/apache/iceberg/pull/9611#discussion_r1500091692
##
api/src/main/java/org/apache/iceberg/actions/RewriteDataFiles.java:
##
@@ -52,6 +52,13 @@ public interface RewriteDataFiles
int PARTIAL_PROGRESS_MAX_COMMITS
github-actions[bot] commented on issue #962:
URL: https://github.com/apache/iceberg/issues/962#issuecomment-1960546952
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 #962: BuildORCProjection does not preserve
field ids for container types
URL: https://github.com/apache/iceberg/issues/962
--
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 #943:
URL: https://github.com/apache/iceberg/issues/943#issuecomment-1960546930
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 #943: Bypass the conversion to/from
intermediate format if not needed
URL: https://github.com/apache/iceberg/issues/943
--
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
github-actions[bot] closed issue #926: Support notification on table writes
URL: https://github.com/apache/iceberg/issues/926
--
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] commented on issue #926:
URL: https://github.com/apache/iceberg/issues/926#issuecomment-1960546904
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 #893:
URL: https://github.com/apache/iceberg/issues/893#issuecomment-1960546881
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 #893: upsertion/deletion for tables with
primary key
URL: https://github.com/apache/iceberg/issues/893
--
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 c
github-actions[bot] closed issue #890: Add an iceberg-mr-runtime module
URL: https://github.com/apache/iceberg/issues/890
--
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
github-actions[bot] commented on issue #890:
URL: https://github.com/apache/iceberg/issues/890#issuecomment-1960546858
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
syun64 commented on issue #464:
URL: https://github.com/apache/iceberg-python/issues/464#issuecomment-1960526232
My understanding is that when a backend client is talking to an API server,
we should only support Client Credentials Flow or the direct use of access
tokens. We are validating t
rahil-c commented on code in PR #9782:
URL: https://github.com/apache/iceberg/pull/9782#discussion_r1500065202
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -1045,6 +1094,27 @@ public void commitTransaction(SessionContext context,
List commits)
flyrain commented on issue #463:
URL: https://github.com/apache/iceberg-python/issues/463#issuecomment-1960507583
`token_type` are required by both flows in RFC. It is fine to be mandatory
for both. The problem is the `issued_token_type`, The token exchange flow
[RFC](https://datatracker.ie
flyrain opened a new issue, #464:
URL: https://github.com/apache/iceberg-python/issues/464
### Feature Request / Improvement
[ID Tokens](https://auth0.com/docs/secure/tokens/id-tokens) are commonly
used in a scenario that a server combines authentication and authorization in a
single
danielcweeks commented on issue #463:
URL: https://github.com/apache/iceberg-python/issues/463#issuecomment-1960499113
According to the
[RFC](https://datatracker.ietf.org/doc/html/rfc6749#section-4.4.3) it should be
`token_type`, but we currently only support `bearer`. The exchange does wo
flyrain commented on issue #463:
URL: https://github.com/apache/iceberg-python/issues/463#issuecomment-1960485657
The client credential flow doesn't return a field `issued_token_type`.
PyIceberg failed at validation:
```
ValidationError: 1 validation error for TokenResponse
issued_
danielcweeks commented on issue #463:
URL: https://github.com/apache/iceberg-python/issues/463#issuecomment-1960467569
The client credential flow is already implemented: [see
here](https://github.com/apache/iceberg-python/blob/82d88920aaa7cba8593e93b5df9f3a87b13c88da/pyiceberg/catalog/rest.p
flyrain opened a new issue, #463:
URL: https://github.com/apache/iceberg-python/issues/463
### Feature Request / Improvement
It seems PyIceberg only supports OAuth2 [token exchange
flow](https://datatracker.ietf.org/doc/html/rfc8693#name-token-exchange-request-and-).
Here is the toke
amogh-jahagirdar commented on code in PR #9783:
URL: https://github.com/apache/iceberg/pull/9783#discussion_r1500025586
##
aws/src/main/java/org/apache/iceberg/aws/dynamodb/DynamoDbCatalog.java:
##
@@ -84,7 +84,12 @@
import
software.amazon.awssdk.services.dynamodb.model.Transa
jackye1995 commented on PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#issuecomment-1960392226
Looks like the CI test failed, @zachdisc could you take a look and fix?
Overall, we have the 2 options based on this thread:
https://github.com/apache/iceberg/pull/9731#discussio
Fokko commented on PR #8625:
URL: https://github.com/apache/iceberg/pull/8625#issuecomment-1960379373
@aokolnychyi Hmm, I did some quick checks and that seems to be correct. I'm
pretty sure that it was using the code because I was seeing exceptions and
differences in the benchmarks. Let me
geruh commented on code in PR #9783:
URL: https://github.com/apache/iceberg/pull/9783#discussion_r1499969849
##
aws/src/main/java/org/apache/iceberg/aws/dynamodb/DynamoDbCatalog.java:
##
@@ -84,7 +84,12 @@
import
software.amazon.awssdk.services.dynamodb.model.TransactWriteItem
jackye1995 commented on code in PR #9782:
URL: https://github.com/apache/iceberg/pull/9782#discussion_r1499968208
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -1045,6 +1094,27 @@ public void commitTransaction(SessionContext context,
List commits)
jackye1995 commented on code in PR #9782:
URL: https://github.com/apache/iceberg/pull/9782#discussion_r1499966845
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -224,6 +229,12 @@ public void initialize(String name, Map
unresolved) {
jackye1995 commented on code in PR #9782:
URL: https://github.com/apache/iceberg/pull/9782#discussion_r1499965680
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -224,6 +229,12 @@ public void initialize(String name, Map
unresolved) {
geruh commented on code in PR #9782:
URL: https://github.com/apache/iceberg/pull/9782#discussion_r1499953848
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -224,6 +229,12 @@ public void initialize(String name, Map
unresolved) {
clien
RussellSpitzer commented on code in PR #9611:
URL: https://github.com/apache/iceberg/pull/9611#discussion_r1499925187
##
api/src/main/java/org/apache/iceberg/actions/RewriteDataFiles.java:
##
@@ -52,6 +52,13 @@ public interface RewriteDataFiles
int PARTIAL_PROGRESS_MAX_COMM
RussellSpitzer commented on code in PR #9611:
URL: https://github.com/apache/iceberg/pull/9611#discussion_r1499923578
##
api/src/main/java/org/apache/iceberg/actions/RewriteDataFiles.java:
##
@@ -52,6 +52,13 @@ public interface RewriteDataFiles
int PARTIAL_PROGRESS_MAX_COMM
jqin61 commented on code in PR #453:
URL: https://github.com/apache/iceberg-python/pull/453#discussion_r1499922547
##
pyiceberg/partitioning.py:
##
@@ -215,3 +236,54 @@ def assign_fresh_partition_spec_ids(spec: PartitionSpec,
old_schema: Schema, fre
)
)
geruh opened a new pull request, #9783:
URL: https://github.com/apache/iceberg/pull/9783
As discussed in the [community
sync](https://youtu.be/uAQVGd5zV4I?si=cj0xpfprgvJIGYIm&t=1323), we want to
reduce the scope of supported catalogs in Iceberg. There are currently many
options, which crea
rahil-c commented on code in PR #9782:
URL: https://github.com/apache/iceberg/pull/9782#discussion_r1499894821
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -490,12 +522,29 @@ public void createNamespace(
@Override
public List listNamespace
geruh commented on code in PR #9660:
URL: https://github.com/apache/iceberg/pull/9660#discussion_r1499882109
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1482,6 +1490,24 @@ components:
explode: false
example: "vended-credentials,remote-signing"
+page-token:
geruh commented on code in PR #9660:
URL: https://github.com/apache/iceberg/pull/9660#discussion_r1499882109
##
open-api/rest-catalog-open-api.yaml:
##
@@ -1482,6 +1490,24 @@ components:
explode: false
example: "vended-credentials,remote-signing"
+page-token:
flyrain commented on issue #462:
URL: https://github.com/apache/iceberg-python/issues/462#issuecomment-1960183527
Thanks for conform, @syun64 . Glad it was taken care in Java.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
geruh commented on code in PR #9782:
URL: https://github.com/apache/iceberg/pull/9782#discussion_r1499864124
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -490,12 +522,29 @@ public void createNamespace(
@Override
public List listNamespaces(
geruh commented on code in PR #9782:
URL: https://github.com/apache/iceberg/pull/9782#discussion_r1499864124
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -490,12 +522,29 @@ public void createNamespace(
@Override
public List listNamespaces(
syun64 commented on issue #462:
URL: https://github.com/apache/iceberg-python/issues/462#issuecomment-1960161562
Hi @flyrain thanks for raising this. I just took a look into the [Java
code](https://github.com/apache/iceberg/blob/main/core/src/main/java/org/apache/iceberg/rest/RESTSessionCata
syun64 commented on issue #354:
URL: https://github.com/apache/iceberg-python/issues/354#issuecomment-1960143413
Hey @kevinjqliu thank you for raising this. I'm interested in writing up the
PyIceberg implementation for
[add_files](https://iceberg.apache.org/docs/latest/spark-procedures/#add
flyrain opened a new issue, #462:
URL: https://github.com/apache/iceberg-python/issues/462
### Feature Request / Improvement
[OAuth2 Scope](https://oauth.net/2/scope/) is a mechanism to limit an
application's access to a user's account. It can also be used to ask for more
information
aokolnychyi commented on code in PR #9611:
URL: https://github.com/apache/iceberg/pull/9611#discussion_r1499825432
##
api/src/main/java/org/apache/iceberg/actions/RewriteDataFiles.java:
##
@@ -52,6 +52,13 @@ public interface RewriteDataFiles
int PARTIAL_PROGRESS_MAX_COMMITS
aokolnychyi commented on PR #8625:
URL: https://github.com/apache/iceberg/pull/8625#issuecomment-1960120064
@Fokko, aren't we using `DataFileWriter` from Avro for Iceberg metadata?
Yeah, I fully support the idea, it is just my preliminary analysis showed it
would have no effect on currently
aokolnychyi commented on PR #9721:
URL: https://github.com/apache/iceberg/pull/9721#issuecomment-1960107350
@huaxingao, we can open up some utilities on Iceberg side, if needed.
Unfortunately, the logic will be fairly coupled any way. I kind of hope we can
offset some of the duplicating by
rahil-c opened a new pull request, #9782:
URL: https://github.com/apache/iceberg/pull/9782
Implemented pagination in list apis based on the spec:
https://github.com/apache/iceberg/pull/9660
@jackye1995 @nastra @danielcweeks @geruh
## Testing
* Ran gradle build
* Manual test
syun64 commented on code in PR #453:
URL: https://github.com/apache/iceberg-python/pull/453#discussion_r1499729828
##
pyiceberg/partitioning.py:
##
@@ -215,3 +236,54 @@ def assign_fresh_partition_spec_ids(spec: PartitionSpec,
old_schema: Schema, fre
)
)
boroknagyz opened a new pull request, #9781:
URL: https://github.com/apache/iceberg/pull/9781
RewriteFiles might be used to compact a table, e.g. it can replace lots of
small files with a few big files. Concurrent RowDelta (DELETE/ UPDATE)
operations must fail, because they can put the tabl
jessiedanwang closed issue #9766: Issue with 'writeTo'
URL: https://github.com/apache/iceberg/issues/9766
--
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
jessiedanwang commented on issue #9766:
URL: https://github.com/apache/iceberg/issues/9766#issuecomment-1960025819
It turns out that there is a bug in our code, after that's fixed, we have no
problem with 'writeTo' or 'merge into'. Thanks
--
This is an automated message from the Apache Gi
syun64 commented on issue #281:
URL: https://github.com/apache/iceberg-python/issues/281#issuecomment-1959949307
There's a PR in progress that will introduce 'REPLACE TABLE' support, but I
don't think we've come to a consensus yet on how we would want to support 'AS
SELECT' semantics in PyI
syun64 commented on issue #438:
URL: https://github.com/apache/iceberg-python/issues/438#issuecomment-1959943274
> Duplicate of #281
This was my bad attempt at decoupling the introduction of REPALCE TABLE
support from the discussion of how we should support `... AS SELECT` semantic -
syun64 closed issue #438: "CREATE TABLE (REPLACE TABLE) ... AS SELECT" Support
URL: https://github.com/apache/iceberg-python/issues/438
--
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 co
Fokko commented on issue #438:
URL: https://github.com/apache/iceberg-python/issues/438#issuecomment-1959898488
Duplicate of https://github.com/apache/iceberg-python/issues/281
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
jackye1995 commented on code in PR #9731:
URL: https://github.com/apache/iceberg/pull/9731#discussion_r1499588047
##
api/src/main/java/org/apache/iceberg/actions/RewriteManifests.java:
##
@@ -44,6 +45,16 @@ public interface RewriteManifests
*/
RewriteManifests rewriteIf(P
nastra merged PR #9780:
URL: https://github.com/apache/iceberg/pull/9780
--
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
jqin61 commented on PR #453:
URL: https://github.com/apache/iceberg-python/pull/453#issuecomment-1959848004
> @jqin61 I wanted to do a second round, but I think you forgot to push? :)
Hi Fokko sorry for the delayed push of the fixes. It took a little time to
think through how to use t
Fokko commented on PR #9775:
URL: https://github.com/apache/iceberg/pull/9775#issuecomment-1959716808
Thanks for fixing this @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 specific
Fokko merged PR #9775:
URL: https://github.com/apache/iceberg/pull/9775
--
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
manuzhang opened a new pull request, #9779:
URL: https://github.com/apache/iceberg/pull/9779
(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-
syun64 commented on PR #394:
URL: https://github.com/apache/iceberg-python/pull/394#issuecomment-1959670410
> Looks like we broke something already đŸ˜¸ Can we make a list to allow
breaking changes? Similar to
https://github.com/apache/parquet-mr/blob/d8396086b3e3fefc6829f8640917c3bbde0fa9c4/p
wgtmac commented on PR #9772:
URL: https://github.com/apache/iceberg/pull/9772#issuecomment-1959648527
I plan to resolve https://github.com/apache/iceberg/issues/7162 by adding
vectorized readers for all v2 encodings. This is the 1st patch. Would you mind
taking a look? @rdblue @nastra @Fok
nastra commented on code in PR #9760:
URL: https://github.com/apache/iceberg/pull/9760#discussion_r1499387190
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRemoveOrphanFilesProcedure.java:
##
@@ -106,7 +96,7 @@ public void testRemoveOrphanF
nastra commented on code in PR #9760:
URL: https://github.com/apache/iceberg/pull/9760#discussion_r1499386214
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestMigrateTableProcedure.java:
##
@@ -18,51 +18,45 @@
*/
package org.apache.iceberg.
nastra commented on code in PR #9724:
URL: https://github.com/apache/iceberg/pull/9724#discussion_r1499380088
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRewriteDataFilesAction.java:
##
@@ -334,6 +344,208 @@ public void testBinPackWithDeletes() {
nastra commented on code in PR #9724:
URL: https://github.com/apache/iceberg/pull/9724#discussion_r1499378001
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java:
##
@@ -172,11 +192,18 @@ public RewriteDataFiles.Result execute() {
nastra commented on code in PR #9724:
URL: https://github.com/apache/iceberg/pull/9724#discussion_r1499376818
##
api/src/main/java/org/apache/iceberg/RemoveDanglingDeletesMode.java:
##
@@ -0,0 +1,64 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or mor
nastra commented on code in PR #9724:
URL: https://github.com/apache/iceberg/pull/9724#discussion_r1499375427
##
api/src/main/java/org/apache/iceberg/RemoveDanglingDeletesMode.java:
##
@@ -0,0 +1,64 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or mor
ajantha-bhat commented on code in PR #9778:
URL: https://github.com/apache/iceberg/pull/9778#discussion_r1499322386
##
.github/ISSUE_TEMPLATE/iceberg_bug_report.yml:
##
@@ -38,14 +39,6 @@ body:
- "1.2.0"
- "1.1.0"
- "1.0.0"
-- "0.14.1"
Review
tomtongue commented on code in PR #9760:
URL: https://github.com/apache/iceberg/pull/9760#discussion_r1499265736
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestCallStatementParser.java:
##
@@ -188,25 +184,26 @@ private void checkArg(
tomtongue commented on code in PR #9760:
URL: https://github.com/apache/iceberg/pull/9760#discussion_r1499265736
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestCallStatementParser.java:
##
@@ -188,25 +184,26 @@ private void checkArg(
tomtongue commented on code in PR #9760:
URL: https://github.com/apache/iceberg/pull/9760#discussion_r1499265736
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestCallStatementParser.java:
##
@@ -188,25 +184,26 @@ private void checkArg(
nastra commented on code in PR #9760:
URL: https://github.com/apache/iceberg/pull/9760#discussion_r1499256325
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestCallStatementParser.java:
##
@@ -188,25 +184,26 @@ private void checkArg(
if
nastra opened a new pull request, #9776:
URL: https://github.com/apache/iceberg/pull/9776
We currently have two markdown files that aren't fully in-sync about how to
contribute to Iceberg. The one is only in the source tree, but not referenced.
The other is the one on the website.
I've
liurenjie1024 closed issue #18: Tracking issues of iceberg-rust v0.2
URL: https://github.com/apache/iceberg-rust/issues/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 un
liurenjie1024 commented on issue #18:
URL: https://github.com/apache/iceberg-rust/issues/18#issuecomment-1959421151
Close this issue as 0.2 has been release. Feel free to reopen when necessary.
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
nastra opened a new pull request, #9775:
URL: https://github.com/apache/iceberg/pull/9775
The current list item layout on https://iceberg.apache.org/releases/ is
wrongly intended, which is being fixed by this PR. Additionally, this adds
Spark 3.5 to the supported engine page.
/cc @bi
adamyasharma2797 commented on PR #9774:
URL: https://github.com/apache/iceberg/pull/9774#issuecomment-1959279749
@rdblue @ajantha-bhat Please check this small PR and throw light on the
understanding gap if any.
--
This is an automated message from the Apache Git Service.
To respond to the
1 - 100 of 129 matches
Mail list logo
