JanKaul commented on issue #75:
URL: https://github.com/apache/iceberg-rust/issues/75#issuecomment-1750077811
A typical usage would look something like the following:
1. Get `Table` with `load_table` operation. (This already fetches the
`metadata_location` and `TableMetadata`)
2. C
amogh-jahagirdar commented on issue #8724:
URL: https://github.com/apache/iceberg/issues/8724#issuecomment-1750065550
I'm not very familiar with Hudi Deltastreamer but for Kafka ingest into
Iceberg tables, @bryanck is working on contributing a Kafka sink connector into
the project https://g
paulpaul1076 commented on issue #8729:
URL: https://github.com/apache/iceberg/issues/8729#issuecomment-1750061470
Wrong name in the title, sorry, copied it and didn't look.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
shreyanshR7 commented on PR #8725:
URL: https://github.com/apache/iceberg/pull/8725#issuecomment-1750055473
I've made the required changes :) @nk1506 @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 abo
nastra merged PR #276:
URL: https://github.com/apache/iceberg-docs/pull/276
--
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
nastra merged PR #277:
URL: https://github.com/apache/iceberg-docs/pull/277
--
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
amogh-jahagirdar commented on issue #8729:
URL: https://github.com/apache/iceberg/issues/8729#issuecomment-1750041439
The issue title and parts of the description refer to
`write.parquet.page-size-bytes` but what you are describing is the file size
and in your code also you refer to file si
nastra merged PR #8728:
URL: https://github.com/apache/iceberg/pull/8728
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apac
nastra merged PR #8727:
URL: https://github.com/apache/iceberg/pull/8727
--
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
nk1506 commented on code in PR #8725:
URL: https://github.com/apache/iceberg/pull/8725#discussion_r1348266266
##
flink/v1.15/flink/src/test/java/org/apache/iceberg/flink/source/TestStreamingMonitorFunction.java:
##
@@ -148,9 +150,10 @@ public void testConsumeFromStartSnapshotId(
paulpaul1076 opened a new issue, #8729:
URL: https://github.com/apache/iceberg/issues/8729
### Apache Iceberg version
None
### Query engine
None
### Please describe the bug 🐞
I have a job that reads orc files and writes them to a Spark table, the
files that
ZENOTME commented on issue #72:
URL: https://github.com/apache/iceberg-rust/issues/72#issuecomment-1749958215
cc @liurenjie1024 @Xuanwo @JanKaul @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
ZENOTME commented on issue #72:
URL: https://github.com/apache/iceberg-rust/issues/72#issuecomment-1749957317
We can integrate V1 and V2 into a writer
```
struck ManifestListWriter {
...
}
impl ManifestListWriter {
fn write_v1(&mut self, manifest_list: ManifestL
aokolnychyi commented on PR #277:
URL: https://github.com/apache/iceberg-docs/pull/277#issuecomment-1749945932
@amogh-jahagirdar @Fokko @nastra @rdblue @danielcweeks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
aokolnychyi opened a new pull request, #277:
URL: https://github.com/apache/iceberg-docs/pull/277
This PR adds versioned docs for the 1.4.0 release.
--
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
paulpaul1076 commented on issue #8721:
URL: https://github.com/apache/iceberg/issues/8721#issuecomment-1749943303
Calling this before my second insert of df2 solves the problem:
spark.catalog.refreshTable("db_nessie.schema_evolution15")
--
This is an automated message from the Apach
aokolnychyi commented on PR #276:
URL: https://github.com/apache/iceberg-docs/pull/276#issuecomment-1749941117
@amogh-jahagirdar @Fokko @nastra @rdblue @danielcweeks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
aokolnychyi opened a new pull request, #276:
URL: https://github.com/apache/iceberg-docs/pull/276
This PR is the first step to update our docs as described
[here](https://iceberg.apache.org/how-to-release/#common-documentation-update).
--
This is an automated message from the Apache Git S
aokolnychyi commented on PR #8727:
URL: https://github.com/apache/iceberg/pull/8727#issuecomment-1749928311
@Fokko @nastra @rdblue @RussellSpitzer
--
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
aokolnychyi commented on PR #8728:
URL: https://github.com/apache/iceberg/pull/8728#issuecomment-1749928447
@Fokko @nastra @rdblue @RussellSpitzer
--
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
aokolnychyi opened a new pull request, #8728:
URL: https://github.com/apache/iceberg/pull/8728
This PR adds 1.4.0 to our issue template.
--
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
bitsondatadev commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348198680
##
docs-new/mkdocs.yml:
##
@@ -0,0 +1,96 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTI
puchengy commented on issue #24:
URL: https://github.com/apache/iceberg-python/issues/24#issuecomment-1749897984
@Fokko Yes, I can help. Thanks.
--
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
ajantha-bhat commented on PR #8695:
URL: https://github.com/apache/iceberg/pull/8695#issuecomment-1749882115
Shall we merge this if no comments?
--
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
bitsondatadev commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348126895
##
docs-new/.github/bin/deploy_docs.sh:
##
@@ -0,0 +1,39 @@
+#!/bin/bash
+
+while [[ "$#" -gt 0 ]]; do
+case $1 in
+-v|--version) ICEBERG_VERSION="$2"
rdblue commented on code in PR #8701:
URL: https://github.com/apache/iceberg/pull/8701#discussion_r1348115184
##
kafka-connect/kafka-connect-events/src/main/java/org/apache/iceberg/connect/events/Element.java:
##
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apache Software Foundatio
rdblue commented on code in PR #8701:
URL: https://github.com/apache/iceberg/pull/8701#discussion_r1348114157
##
kafka-connect/kafka-connect-events/src/main/java/org/apache/iceberg/connect/events/CommitCompletePayload.java:
##
@@ -0,0 +1,97 @@
+/*
+ * Licensed to the Apache Soft
jacobmarble commented on PR #8683:
URL: https://github.com/apache/iceberg/pull/8683#issuecomment-1749799494
> What do you think? If you agree, then we should probably add `ms` types at
the same time.
Sounds good to me. I'll add milliseconds to this PR.
--
This is an automated messa
rdblue commented on code in PR #8683:
URL: https://github.com/apache/iceberg/pull/8683#discussion_r1348103716
##
format/spec.md:
##
@@ -167,30 +167,34 @@ A **`map`** is a collection of key-value pairs with a key
type and a value type.
Primitive Types
-| Primitive type
rdblue commented on PR #8683:
URL: https://github.com/apache/iceberg/pull/8683#issuecomment-1749792874
@jacobmarble, I was talking with @danielcweeks a couple days ago about v3
additions and we came up with a good reason to add `timestamp_ms` and
`timestamptz_ms`. Currently, type promotion
rdblue commented on code in PR #8673:
URL: https://github.com/apache/iceberg/pull/8673#discussion_r1348100841
##
core/src/main/java/org/apache/iceberg/GenericManifestFile.java:
##
@@ -84,15 +84,24 @@ public GenericManifestFile(Schema avroSchema) {
}
}
+ /**
+ * @dep
rdblue commented on code in PR #8673:
URL: https://github.com/apache/iceberg/pull/8673#discussion_r1348099841
##
core/src/main/java/org/apache/iceberg/V2Metadata.java:
##
@@ -39,7 +39,7 @@ private V2Metadata() {}
ManifestFile.MANIFEST_CONTENT.asRequired(),
rdblue commented on PR #42:
URL: https://github.com/apache/iceberg-python/pull/42#issuecomment-1749783634
I don't think that this is necessary and I think it also breaks future uses
of the visitor.
Despite the name, the visitor this updates doesn't actually perform
pre-order traversa
rdblue commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348089146
##
docs-new/README.md:
##
@@ -0,0 +1,175 @@
+
+
+# Iceberg site and documentation
+
+This holds the static files that define and build the documentation site for
Apache
rdblue commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348087991
##
docs-new/README.md:
##
@@ -0,0 +1,123 @@
+
+
+# making-iceberg-docs
+
+Testbed for the Iceberg docs.
+
+## Requirements
+
+* Python >=3.9
+* pip
+
+
+## Build proces
rdblue commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348087050
##
docs-new/.github/bin/deploy_docs.sh:
##
@@ -0,0 +1,39 @@
+#!/bin/bash
+
+while [[ "$#" -gt 0 ]]; do
+case $1 in
+-v|--version) ICEBERG_VERSION="$2"; shift
rdblue commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348077823
##
docs-new/mkdocs.yml:
##
@@ -0,0 +1,96 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
rdblue commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348077823
##
docs-new/mkdocs.yml:
##
@@ -0,0 +1,96 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
rdblue commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348076686
##
docs-new/mkdocs.yml:
##
@@ -0,0 +1,96 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
rdblue commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348075495
##
docs-new/mkdocs.yml:
##
@@ -0,0 +1,96 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
MehulBatra opened a new pull request, #42:
URL: https://github.com/apache/iceberg-python/pull/42
Add support to perform an action before and after field traversal in
preorder fashion for schema
for object: structtype, maptype & listtype
--
This is an automated message from the Apache
bitsondatadev commented on PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#issuecomment-1749722304
I've addressed the comments I intend to hit this review round, but have some
open questions:
- https://github.com/apache/iceberg/pull/8659#discussion_r1343890396
- https://gith
bitsondatadev commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1348043125
##
docs-new/mkdocs.yml:
##
@@ -0,0 +1,96 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTI
Fokko commented on code in PR #8673:
URL: https://github.com/apache/iceberg/pull/8673#discussion_r1347949961
##
api/src/main/java/org/apache/iceberg/ManifestFile.java:
##
@@ -49,7 +49,7 @@ public interface ManifestFile {
Types.LongType.get(),
"Lowest sequen
rdblue merged PR #8726:
URL: https://github.com/apache/iceberg/pull/8726
--
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
Fokko merged PR #5338:
URL: https://github.com/apache/iceberg/pull/5338
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apach
Fokko closed issue #8684: Spec disagrees with implementation on field names
URL: https://github.com/apache/iceberg/issues/8684
--
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
Fokko commented on PR #5338:
URL: https://github.com/apache/iceberg/pull/5338#issuecomment-1749550427
Thanks for the review @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 above to go to the specific
rdblue commented on PR #8726:
URL: https://github.com/apache/iceberg/pull/8726#issuecomment-1749494924
Thank you! Good to get this fixed quickly.
--
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
Fokko opened a new pull request, #8726:
URL: https://github.com/apache/iceberg/pull/8726
This reverts commit f7a7eb2c10cb4a9b6b3ea5bfdfc5d085be8b9c31.
This should stay required.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rdblue commented on code in PR #8672:
URL: https://github.com/apache/iceberg/pull/8672#discussion_r1347854144
##
format/spec.md:
##
@@ -128,13 +128,13 @@ Tables do not require rename, except for tables that use
atomic rename to implem
Writer requirements
-Some tables
rdblue commented on code in PR #8672:
URL: https://github.com/apache/iceberg/pull/8672#discussion_r1347851474
##
format/spec.md:
##
@@ -128,13 +128,13 @@ Tables do not require rename, except for tables that use
atomic rename to implem
Writer requirements
-Some tables
rdblue commented on code in PR #8672:
URL: https://github.com/apache/iceberg/pull/8672#discussion_r1347851058
##
format/spec.md:
##
@@ -128,13 +128,13 @@ Tables do not require rename, except for tables that use
atomic rename to implem
Writer requirements
-Some tables
jbonofre commented on PR #8722:
URL: https://github.com/apache/iceberg/pull/8722#issuecomment-1749444235
@rdblue no impact on the existing PRs: they will be "attached" to main
instead of master. The only impact is for the remotes for our users.
--
This is an automated message from the Ap
hankfanchiu commented on PR #2722:
URL: https://github.com/apache/iceberg/pull/2722#issuecomment-1749443748
@mderoy, thanks for the interest! I don't plan on following through with
apache/hive#2484, #2751, and this PR. Please feel free to take over.
--
This is an automated message from th
Fokko commented on issue #24:
URL: https://github.com/apache/iceberg-python/issues/24#issuecomment-1749431518
@puchengy Yes, certainly. Would this be something that you're interested in
working on? From the snapshot, we can load the manifest list, and from there
the manifests themselves, wh
rdblue commented on PR #8722:
URL: https://github.com/apache/iceberg/pull/8722#issuecomment-1749425255
@jbonofre, how will this affect existing PRs? Do all of them need to be
reopened against main instead of master?
--
This is an automated message from the Apache Git Service.
To respond t
rdblue commented on code in PR #8722:
URL: https://github.com/apache/iceberg/pull/8722#discussion_r1347814726
##
.github/workflows/python-release.yml:
##
@@ -51,7 +51,7 @@ jobs:
- name: Set version
run: python -m poetry version "${{ inputs.version }}"
wo
Fokko commented on code in PR #8673:
URL: https://github.com/apache/iceberg/pull/8673#discussion_r1347814559
##
core/src/main/java/org/apache/iceberg/GenericManifestFile.java:
##
@@ -92,7 +92,7 @@ public GenericManifestFile(Schema avroSchema) {
this.specId = specId;
th
rdblue commented on issue #8610:
URL: https://github.com/apache/iceberg/issues/8610#issuecomment-1749415259
Yes, I agree. The safety of the HadoopCatalog depends on the file system,
which is one reason why we don't recommend using that catalog.
--
This is an automated message from the Apa
rdblue closed issue #8610: Write data to iceberg with using java api multi
thread, cause data lost
URL: https://github.com/apache/iceberg/issues/8610
--
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
puchengy commented on issue #24:
URL: https://github.com/apache/iceberg-python/issues/24#issuecomment-1749412122
@Fokko gentle ping, thanks ^
--
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
shreyanshR7 commented on PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#issuecomment-1749406399
@nastra I made another pull request addressing the issue with some more
clean changes, please consider that.
--
This is an automated message from the Apache Git Service.
To respond
shreyanshR7 commented on PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#issuecomment-1749205309
Let me check, I will try my best to undo these unrelated changes, these
changes may be occurred due to adding awaitility method. sorry for
inconvenience @nastra
--
This is an auto
y0psolo commented on PR #62:
URL: https://github.com/apache/iceberg-rust/pull/62#issuecomment-1749197704
Ok i added some validation steps. Tell me if its ok
i don't know if we need to have all fields visible on the builder. Tell me.
We can choose to open and restrict later or do the rever
nastra commented on PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#issuecomment-1749152592
@shreyanshR7 there are still many unrelated changes to files. Can you please
fix all of those?
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
shreyanshR7 commented on PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#issuecomment-1749145628
I tried to fix the issues :) @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 t
amogh-jahagirdar commented on code in PR #7913:
URL: https://github.com/apache/iceberg/pull/7913#discussion_r1347539389
##
core/src/main/java/org/apache/iceberg/rest/CatalogHandlers.java:
##
@@ -374,4 +385,107 @@ static TableMetadata commit(TableOperations ops,
UpdateTableReque
Fokko commented on code in PR #8673:
URL: https://github.com/apache/iceberg/pull/8673#discussion_r1347466602
##
core/src/main/java/org/apache/iceberg/GenericManifestFile.java:
##
@@ -92,7 +92,7 @@ public GenericManifestFile(Schema avroSchema) {
this.specId = specId;
th
Fokko commented on code in PR #8673:
URL: https://github.com/apache/iceberg/pull/8673#discussion_r1347466602
##
core/src/main/java/org/apache/iceberg/GenericManifestFile.java:
##
@@ -92,7 +92,7 @@ public GenericManifestFile(Schema avroSchema) {
this.specId = specId;
th
Fokko commented on code in PR #8673:
URL: https://github.com/apache/iceberg/pull/8673#discussion_r1347459324
##
core/src/main/java/org/apache/iceberg/GenericManifestFile.java:
##
@@ -92,7 +92,7 @@ public GenericManifestFile(Schema avroSchema) {
this.specId = specId;
th
amogh-jahagirdar commented on code in PR #8673:
URL: https://github.com/apache/iceberg/pull/8673#discussion_r1347419881
##
core/src/main/java/org/apache/iceberg/GenericManifestFile.java:
##
@@ -92,7 +92,7 @@ public GenericManifestFile(Schema avroSchema) {
this.specId = spec
jbonofre commented on PR #8486:
URL: https://github.com/apache/iceberg/pull/8486#issuecomment-1748930989
Re-testing `revapi` on Gradle 8.4 (it should be the same as with Gradle 8.3).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
Fokko commented on code in PR #40:
URL: https://github.com/apache/iceberg-python/pull/40#discussion_r1345991299
##
pyiceberg/manifest.py:
##
@@ -262,15 +346,13 @@ class DataFile(Record):
"split_offsets",
"equality_ids",
"sort_order_id",
-"spec_
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347436052
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
jbonofre commented on issue #8485:
URL: https://github.com/apache/iceberg/issues/8485#issuecomment-1748905073
As gradle 8.4 is now available, I'm updating the PR (I'm also re-testing
`revapi` but I'm pretty sure it will be the same as in Gradle 8.4 :) ).
--
This is an automated message fr
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347411761
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347411761
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347395043
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347395043
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347395043
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347395043
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347395043
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
nastra commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347392925
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throws IOExcept
Xuanwo commented on issue #75:
URL: https://github.com/apache/iceberg-rust/issues/75#issuecomment-1748854108
> It would therefore be helpful to include references to the
`metadata_location` and the `TableMetadata` in the
[`TableCommit`](https://github.com/apache/iceberg-rust/blob/main/crate
ajantha-bhat commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347371858
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throws IO
shreyanshR7 commented on PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#issuecomment-1748822301
wait let me check
--
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
amogh-jahagirdar commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347356619
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throw
nastra commented on code in PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#discussion_r1347352377
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/source/TestIcebergSourceTablesBase.java:
##
@@ -1955,7 +1958,6 @@ public void testRemoveOrphanFilesActionSuppo
nastra commented on code in PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#discussion_r1347351835
##
flink/v1.16/flink/src/test/java/org/apache/iceberg/flink/source/TestIcebergSourceFailover.java:
##
@@ -208,7 +210,7 @@ private void testContinuousIcebergSource(Failov
nastra commented on code in PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#discussion_r1347351094
##
core/src/test/java/org/apache/iceberg/util/TestInMemoryLockManager.java:
##
@@ -112,7 +114,7 @@ public void testAcquireSingleProcess() throws Exception {
Complet
nastra commented on code in PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#discussion_r1347350422
##
api/src/test/java/org/apache/iceberg/metrics/TestDefaultTimer.java:
##
@@ -104,7 +104,7 @@ public void measureRunnable() {
Runnable runnable =
() -> {
shreyanshR7 commented on PR #8715:
URL: https://github.com/apache/iceberg/pull/8715#issuecomment-1748692640
I've made the changes :)@nastra
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the spe
jbonofre commented on code in PR #8722:
URL: https://github.com/apache/iceberg/pull/8722#discussion_r1347247075
##
docs/flink-writes.md:
##
@@ -270,4 +270,4 @@ INSERT INTO tableName /*+ OPTIONS('upsert-enabled'='true')
*/
...
```
-Check out all the options here:
[write-opt
ajantha-bhat commented on code in PR #8723:
URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347242330
##
core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java:
##
@@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen)
throws IO
bitsondatadev commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1347234000
##
docs-new/site/releases.md:
##
@@ -0,0 +1,777 @@
+---
+title: "Releases"
+---
+
+
+## Downloads
+
+The latest version of Iceberg is [{{ icebergVersion
}}](http
ajantha-bhat commented on code in PR #8722:
URL: https://github.com/apache/iceberg/pull/8722#discussion_r1347234482
##
docs/flink-writes.md:
##
@@ -270,4 +270,4 @@ INSERT INTO tableName /*+ OPTIONS('upsert-enabled'='true')
*/
...
```
-Check out all the options here:
[write
bitsondatadev commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1347234000
##
docs-new/site/releases.md:
##
@@ -0,0 +1,777 @@
+---
+title: "Releases"
+---
+
+
+## Downloads
+
+The latest version of Iceberg is [{{ icebergVersion
}}](http
bitsondatadev commented on code in PR #8659:
URL: https://github.com/apache/iceberg/pull/8659#discussion_r1347232147
##
docs-new/site/releases.md:
##
@@ -0,0 +1,777 @@
+---
+title: "Releases"
+---
+
+
+## Downloads
+
+The latest version of Iceberg is [{{ icebergVersion
}}](http
nastra opened a new pull request, #8723:
URL: https://github.com/apache/iceberg/pull/8723
(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
1 - 100 of 117 matches
Mail list logo