github-actions[bot] commented on PR #4898:
URL: https://github.com/apache/iceberg/pull/4898#issuecomment-227895
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
github-actions[bot] commented on PR #4809:
URL: https://github.com/apache/iceberg/pull/4809#issuecomment-2278899955
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
github-actions[bot] commented on PR #4842:
URL: https://github.com/apache/iceberg/pull/4842#issuecomment-2278899968
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
github-actions[bot] commented on PR #4775:
URL: https://github.com/apache/iceberg/pull/4775#issuecomment-2278899946
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
github-actions[bot] closed issue #4889: Support ORDERED BY in CTAS statement
URL: https://github.com/apache/iceberg/issues/4889
--
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.
github-actions[bot] commented on issue #4889:
URL: https://github.com/apache/iceberg/issues/4889#issuecomment-227887
This issue has been closed because it has not received any activity in the
last 14 days since being marked as 'stale'
--
This is an automated message from the Apache Gi
github-actions[bot] commented on PR #4855:
URL: https://github.com/apache/iceberg/pull/4855#issuecomment-2278899989
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
sdd commented on PR #535:
URL: https://github.com/apache/iceberg-rust/pull/535#issuecomment-2278895706
Fixes the error suppression issue reported in
https://github.com/apache/iceberg-rust/issues/495
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
sdd commented on issue #495:
URL: https://github.com/apache/iceberg-rust/issues/495#issuecomment-2278895420
I have fixed the issue where errors were not returned to the user, in
https://github.com/apache/iceberg-rust/pull/535
--
This is an automated message from the Apache Git Service.
To
sdd opened a new pull request, #535:
URL: https://github.com/apache/iceberg-rust/pull/535
Ensure that errors get propagated back to the caller when encountered during
generation of the file plan.
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
sdd commented on code in PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#discussion_r1712265470
##
crates/iceberg/src/spec/manifest_list.rs:
##
@@ -78,6 +78,11 @@ impl ManifestList {
pub fn entries(&self) -> &[ManifestFile] {
&self.entries
}
+
+
sdd commented on PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#issuecomment-2278812113
Thanks for the review @Xuanwo, much appreciated! Back to you, I've addressed
your comments.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
sdd commented on code in PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#discussion_r1712263915
##
crates/iceberg/src/scan.rs:
##
@@ -199,298 +256,588 @@ impl<'a> TableScanBuilder<'a> {
field_ids.push(field_id);
}
-Ok(TableScan {
+
sdd commented on code in PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#discussion_r1712259304
##
crates/iceberg/src/io/object_cache.rs:
##
@@ -0,0 +1,161 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
sdd commented on code in PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#discussion_r1712257994
##
crates/iceberg/src/io/object_cache.rs:
##
@@ -0,0 +1,161 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
sdd commented on code in PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#discussion_r1712256232
##
crates/iceberg/src/io/object_cache.rs:
##
@@ -0,0 +1,161 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.
sdd commented on code in PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#discussion_r1712255347
##
crates/iceberg/src/table.rs:
##
@@ -16,28 +16,156 @@
// under the License.
//! Table API for Apache Iceberg
-use typed_builder::TypedBuilder;
+
+use std::sync::Ar
sdd commented on code in PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#discussion_r1712254637
##
crates/iceberg/src/table.rs:
##
@@ -16,28 +16,156 @@
// under the License.
//! Table API for Apache Iceberg
-use typed_builder::TypedBuilder;
+
+use std::sync::Ar
kevinjqliu commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1712169542
##
mkdocs/docs/configuration.md:
##
@@ -77,13 +77,14 @@ For the FileIO there are several configuration options
available:
| Key | Examp
guitcastro commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1712144018
##
mkdocs/docs/configuration.md:
##
@@ -77,13 +77,14 @@ For the FileIO there are several configuration options
available:
| Key | Examp
kevinjqliu commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1712053032
##
mkdocs/docs/configuration.md:
##
@@ -77,13 +77,14 @@ For the FileIO there are several configuration options
available:
| Key | Examp
callum-ryan opened a new pull request, #534:
URL: https://github.com/apache/iceberg-rust/pull/534
Carrying on from [PR #524
](https://github.com/apache/iceberg-rust/pull/524), this is the implementation
of the namespace functionality
--
This is an automated message from the Apache Git Se
guitcastro commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1712047933
##
mkdocs/docs/configuration.md:
##
@@ -84,6 +84,7 @@ For the FileIO there are several configuration options
available:
| s3.session-token | AQoDYXdzEJr
guitcastro commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1712047613
##
pyiceberg/io/__init__.py:
##
@@ -58,6 +58,8 @@
S3_PROXY_URI = "s3.proxy-uri"
S3_CONNECT_TIMEOUT = "s3.connect-timeout"
S3_SIGNER_URI = "s3.signer.uri"
+
kevinjqliu commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1712044507
##
mkdocs/docs/configuration.md:
##
@@ -84,6 +84,7 @@ For the FileIO there are several configuration options
available:
| s3.session-token | AQoDYXdzEJr
guitcastro commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1712042160
##
mkdocs/docs/configuration.md:
##
@@ -84,6 +84,7 @@ For the FileIO there are several configuration options
available:
| s3.session-token | AQoDYXdzEJr
kevinjqliu commented on issue #1032:
URL:
https://github.com/apache/iceberg-python/issues/1032#issuecomment-2278578863
okay, this doesn't look like an issue with reading many metadata files.
I wonder if the `limit` is respected for table scans.
Things I want to compare
* readin
jkleinkauff commented on issue #1032:
URL:
https://github.com/apache/iceberg-python/issues/1032#issuecomment-2278564297
Hey, thank you for taking a time to answer me!
1. My files are in S3.
2. Sure! It's something I could do on my end? Do you have any recommendation
on that?
(I
kevinjqliu commented on issue #1032:
URL:
https://github.com/apache/iceberg-python/issues/1032#issuecomment-2278556792
There's a nontrivial cost in reading metadata files in Iceberg.
Can you run this command,
```
table.inspect.manifests().to_pandas()
```
This will show the nu
sdd opened a new pull request, #533:
URL: https://github.com/apache/iceberg-rust/pull/533
Fixes: https://github.com/apache/iceberg-rust/issues/532.
Timezone needs to be explicitly set to `UTC` to match values written by
Iceberg to underlying Parquet files.
--
This is an automated m
sdd opened a new issue, #532:
URL: https://github.com/apache/iceberg-rust/issues/532
If attempting a table scan where one of the columns in the table is a
`Timestamptz` or a `Timestamp`, you will encounter an error like this:
```
called `Result::unwrap()` on an `Err` value: Feature
kevinjqliu commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1711978272
##
mkdocs/docs/configuration.md:
##
@@ -84,6 +84,7 @@ For the FileIO there are several configuration options
available:
| s3.session-token | AQoDYXdzEJr
ndrluis commented on code in PR #1007:
URL: https://github.com/apache/iceberg-python/pull/1007#discussion_r1711967438
##
pyiceberg/io/pyarrow.py:
##
@@ -388,7 +389,14 @@ def _initialize_fs(self, scheme: str, netloc:
Optional[str] = None) -> FileSyste
gcs_kwargs
ndrluis commented on code in PR #1007:
URL: https://github.com/apache/iceberg-python/pull/1007#discussion_r1711967825
##
pyiceberg/io/fsspec.py:
##
@@ -76,6 +77,7 @@
OutputStream,
)
from pyiceberg.typedef import Properties
+from pyiceberg.utils.deprecated import deprecate
ndrluis commented on code in PR #1007:
URL: https://github.com/apache/iceberg-python/pull/1007#discussion_r1711966599
##
pyiceberg/io/fsspec.py:
##
@@ -158,6 +160,14 @@ def _gs(properties: Properties) -> AbstractFileSystem:
# https://gcsfs.readthedocs.io/en/latest/api.html#
ndrluis commented on code in PR #1007:
URL: https://github.com/apache/iceberg-python/pull/1007#discussion_r1711963854
##
pyiceberg/io/fsspec.py:
##
@@ -158,6 +160,14 @@ def _gs(properties: Properties) -> AbstractFileSystem:
# https://gcsfs.readthedocs.io/en/latest/api.html#
ndrluis commented on code in PR #961:
URL: https://github.com/apache/iceberg-python/pull/961#discussion_r1711960193
##
pyiceberg/io/fsspec.py:
##
@@ -175,14 +184,50 @@ def _gs(properties: Properties) -> AbstractFileSystem:
def _adlfs(properties: Properties) -> AbstractFileSyste
laksh-krishna-sharma commented on PR #1015:
URL: https://github.com/apache/iceberg-python/pull/1015#issuecomment-2278491228
Thank you, @kevinjqliu I am grateful for the opportunity to contribute and
for your patient guidance. I would like to work more under your guidance in
this repository
sungwy commented on issue #1032:
URL:
https://github.com/apache/iceberg-python/issues/1032#issuecomment-2278456915
Hi @jkleinkauff , that's indeed an interesting observation.
I have some follow up questions to help us understand it better.
1. Where are your files stored?
2. Is t
sungwy merged PR #1034:
URL: https://github.com/apache/iceberg-python/pull/1034
--
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...@icebe
pvary commented on issue #10892:
URL: https://github.com/apache/iceberg/issues/10892#issuecomment-2278393063
> I would say no. multiple jobs writing to the same table is a good example.
Theoretically it is possible to revert only the changes done by the given
job (for append only tabl
dependabot[bot] opened a new pull request, #1035:
URL: https://github.com/apache/iceberg-python/pull/1035
Bumps [aiohttp](https://github.com/aio-libs/aiohttp) from 3.10.1 to 3.10.2.
Release notes
Sourced from https://github.com/aio-libs/aiohttp/releases";>aiohttp's
releases.
kevinjqliu commented on issue #1008:
URL:
https://github.com/apache/iceberg-python/issues/1008#issuecomment-2278370759
someone asked about merge/upserts use cases in the slack channel as well,
similar to `overwrite_filter`
--
This is an automated message from the Apache Git Service.
To r
sungwy opened a new pull request, #1034:
URL: https://github.com/apache/iceberg-python/pull/1034
(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
RussellSpitzer commented on issue #10907:
URL: https://github.com/apache/iceberg/issues/10907#issuecomment-2278344995
Expire snapshots only removes data files which are no longer needed by any
remaining snapshots. The output of your command shows no files were needed to
be removed. Based on
guitcastro commented on issue #1028:
URL:
https://github.com/apache/iceberg-python/issues/1028#issuecomment-2278328228
Create the PR #1029 and #1033 to fix it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
guitcastro opened a new pull request, #1033:
URL: https://github.com/apache/iceberg-python/pull/1033
This PR fix the hardcoded `X-Iceberg-Access-Delegation` header, the second
point of https://github.com/apache/iceberg-python/issues/1028.
It's based on the #1029 , and must be merged
kevinjqliu commented on code in PR #752:
URL: https://github.com/apache/iceberg-python/pull/752#discussion_r1711764892
##
tests/catalog/test_hive.py:
##
@@ -1195,3 +1195,40 @@ def test_hive_wait_for_lock() -> None:
with pytest.raises(WaitingForLockException):
catal
jkleinkauff opened a new issue, #1032:
URL: https://github.com/apache/iceberg-python/issues/1032
### Question
Hey, thanks for this very convenient library.
This is not a bug, just want to better understand something.
I have a question regarding the performance - ie time t
kevinjqliu commented on issue #314:
URL: https://github.com/apache/iceberg-python/issues/314#issuecomment-2278310300
> Why do we choose to include non-iceberg tables in the result in python?
I don't think we should. Using HMS for both hive and iceberg tables is
pretty common, we shoul
kevinjqliu commented on issue #206:
URL: https://github.com/apache/iceberg-python/issues/206#issuecomment-2278308121
This is already done using [`session_catalog_hive`
](https://github.com/apache/iceberg-python/blob/678facc09b99394acf4c1c12b5a8a67bf2a20f5d/tests/conftest.py#L2202-L2213)
in
kevinjqliu closed issue #206: Set table properties through Hive
URL: https://github.com/apache/iceberg-python/issues/206
--
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 unsu
kevinjqliu commented on PR #766:
URL: https://github.com/apache/iceberg-python/pull/766#issuecomment-2278296348
Hi @yothinix do you mind rebasing with main? I think this is very close to
being merged, I can work with you to get it through.
--
This is an automated message from the Apa
kevinjqliu commented on code in PR #961:
URL: https://github.com/apache/iceberg-python/pull/961#discussion_r1711732413
##
pyiceberg/io/fsspec.py:
##
@@ -175,14 +184,50 @@ def _gs(properties: Properties) -> AbstractFileSystem:
def _adlfs(properties: Properties) -> AbstractFileSy
kevinjqliu commented on PR #1015:
URL: https://github.com/apache/iceberg-python/pull/1015#issuecomment-2278268207
Thanks for the contribution, @laksh-krishna-sharma
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
kevinjqliu merged PR #1015:
URL: https://github.com/apache/iceberg-python/pull/1015
--
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...@i
kevinjqliu closed issue #949: [documentation] library version upgrade fails
`test_version_format`
URL: https://github.com/apache/iceberg-python/issues/949
--
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
kevinjqliu commented on PR #1015:
URL: https://github.com/apache/iceberg-python/pull/1015#issuecomment-2278229829
I can run `make lint` for this branch
--
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
kevinjqliu commented on PR #1015:
URL: https://github.com/apache/iceberg-python/pull/1015#issuecomment-2278229440
That's odd, it might be due to the specific Python version on Windows. I
found a similar issue
https://github.com/python-poetry/poetry/issues/3894#issuecomment-1130034154
--
nastra commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711679815
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -215,8 +232,32 @@ public static String encodeNamespace(Namespace ns) {
* @return a namespace
nastra commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711677062
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -215,8 +232,32 @@ public static String encodeNamespace(Namespace ns) {
* @return a namespace
nastra commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711532749
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -194,15 +192,34 @@ public static String decodeString(String encoded) {
* @return UTF-8 encoded s
nastra merged PR #10899:
URL: https://github.com/apache/iceberg/pull/10899
--
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
guitcastro commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1711612526
##
mkdocs/docs/configuration.md:
##
@@ -84,6 +84,7 @@ For the FileIO there are several configuration options
available:
| s3.session-token | AQoDYXdzEJr
sungwy commented on PR #1031:
URL: https://github.com/apache/iceberg-python/pull/1031#issuecomment-2278054652
Done - I'll start the release process on
https://github.com/apache/iceberg-python/tree/pyiceberg-0.7.x
--
This is an automated message from the Apache Git Service.
To respond to t
nastra commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711532749
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -194,15 +192,34 @@ public static String decodeString(String encoded) {
* @return UTF-8 encoded s
nastra commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711532266
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -194,15 +192,34 @@ public static String decodeString(String encoded) {
* @return UTF-8 encoded s
nastra commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711530996
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -215,8 +232,32 @@ public static String encodeNamespace(Namespace ns) {
* @return a namespace
nastra commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711529328
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -215,8 +232,32 @@ public static String encodeNamespace(Namespace ns) {
* @return a namespace
nastra commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711528454
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -194,15 +192,34 @@ public static String decodeString(String encoded) {
* @return UTF-8 encoded s
sungwy commented on PR #1031:
URL: https://github.com/apache/iceberg-python/pull/1031#issuecomment-2278010259
I'll get that done. Thank you for the input @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
sungwy closed pull request #1031: Patch Release 0.7.1
URL: https://github.com/apache/iceberg-python/pull/1031
--
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-
Fokko commented on PR #1031:
URL: https://github.com/apache/iceberg-python/pull/1031#issuecomment-2278008651
@sungwy Looks good! I'm also okay with just cherry-picking these directly on
the branch. All the commits are good because they have already been merged. I
think a PR makes sense when
sungwy opened a new pull request, #1031:
URL: https://github.com/apache/iceberg-python/pull/1031
(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
sungwy merged PR #1030:
URL: https://github.com/apache/iceberg-python/pull/1030
--
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...@icebe
Fokko commented on code in PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#discussion_r1711510285
##
mkdocs/docs/configuration.md:
##
@@ -84,6 +84,7 @@ For the FileIO there are several configuration options
available:
| s3.session-token | AQoDYXdzEJr...
guitcastro commented on PR #1029:
URL: https://github.com/apache/iceberg-python/pull/1029#issuecomment-2277988150
@Fokko @ndrluis I've applied the code suggestions. Can you please review
again?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
stevenzwu commented on issue #10892:
URL: https://github.com/apache/iceberg/issues/10892#issuecomment-2277963876
> whether we think that the Iceberg table state is part of the Flink state,
or not.
I would say no. multiple jobs writing to the same table is a good example.
> I ha
jdockerty commented on code in PR #520:
URL: https://github.com/apache/iceberg-rust/pull/520#discussion_r1711376787
##
.github/workflows/ci.yml:
##
@@ -113,7 +113,7 @@ jobs:
run: cargo test --no-fail-fast --all-targets --all-features --workspace
- name: Async-s
sungwy opened a new pull request, #1030:
URL: https://github.com/apache/iceberg-python/pull/1030
(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
sungwy commented on code in PR #1016:
URL: https://github.com/apache/iceberg-python/pull/1016#discussion_r1711386006
##
mkdocs/docs/configuration.md:
##
@@ -32,6 +32,7 @@ Iceberg tables support table properties to configure table
behavior.
| ---
Xuanwo commented on PR #520:
URL: https://github.com/apache/iceberg-rust/pull/520#issuecomment-2277849379
The opendal release is about to start, I expect to finish in next monday.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
Xuanwo commented on code in PR #520:
URL: https://github.com/apache/iceberg-rust/pull/520#discussion_r1711383831
##
.github/workflows/ci.yml:
##
@@ -113,7 +113,7 @@ jobs:
run: cargo test --no-fail-fast --all-targets --all-features --workspace
- name: Async-std
jdockerty commented on code in PR #520:
URL: https://github.com/apache/iceberg-rust/pull/520#discussion_r1711376787
##
.github/workflows/ci.yml:
##
@@ -113,7 +113,7 @@ jobs:
run: cargo test --no-fail-fast --all-targets --all-features --workspace
- name: Async-s
jdockerty commented on code in PR #520:
URL: https://github.com/apache/iceberg-rust/pull/520#discussion_r1711376787
##
.github/workflows/ci.yml:
##
@@ -113,7 +113,7 @@ jobs:
run: cargo test --no-fail-fast --all-targets --all-features --workspace
- name: Async-s
jdockerty commented on code in PR #520:
URL: https://github.com/apache/iceberg-rust/pull/520#discussion_r1711376787
##
.github/workflows/ci.yml:
##
@@ -113,7 +113,7 @@ jobs:
run: cargo test --no-fail-fast --all-targets --all-features --workspace
- name: Async-s
Xuanwo commented on code in PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#discussion_r1711342057
##
crates/iceberg/src/io/object_cache.rs:
##
@@ -0,0 +1,161 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreemen
ajantha-bhat commented on code in PR #10176:
URL: https://github.com/apache/iceberg/pull/10176#discussion_r1711350427
##
data/src/main/java/org/apache/iceberg/data/PartitionStatsGenerator.java:
##
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
ajantha-bhat commented on code in PR #10176:
URL: https://github.com/apache/iceberg/pull/10176#discussion_r1711350427
##
data/src/main/java/org/apache/iceberg/data/PartitionStatsGenerator.java:
##
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
ajantha-bhat commented on code in PR #10176:
URL: https://github.com/apache/iceberg/pull/10176#discussion_r1711348696
##
data/src/main/java/org/apache/iceberg/data/PartitionStatsWriterUtil.java:
##
@@ -0,0 +1,93 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
toien opened a new issue, #10907:
URL: https://github.com/apache/iceberg/issues/10907
### Query engine
Spark SQL on AWS EMR(7.1.0)
Versions:
- Spark: 3.5.0
- Iceberg: 1.4.3
- Flink: 1.18 (Managed Apache Flink of AWS)
### Question
First i create an iceberg
laksh-krishna-sharma commented on PR #1015:
URL: https://github.com/apache/iceberg-python/pull/1015#issuecomment-2277796260
While running `make install`, I encountered the error shown in the above
s
laksh-krishna-sharma commented on PR #1015:
URL: https://github.com/apache/iceberg-python/pull/1015#issuecomment-225803
Yes, I followed all the steps mentioned in the contribution guide and ran
`make install` and `make lint`. I also resolved the issues that came up. Please
review the up
sdd commented on PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#issuecomment-222962
Sure, will do so in a few hours time
--
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
snazy commented on code in PR #10877:
URL: https://github.com/apache/iceberg/pull/10877#discussion_r1711255861
##
core/src/main/java/org/apache/iceberg/rest/RESTUtil.java:
##
@@ -194,15 +192,34 @@ public static String decodeString(String encoded) {
* @return UTF-8 encoded st
adutra commented on code in PR #10753:
URL: https://github.com/apache/iceberg/pull/10753#discussion_r1711222487
##
aws/src/main/java/org/apache/iceberg/aws/RESTSigV4AuthManager.java:
##
@@ -0,0 +1,51 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or mo
adutra commented on code in PR #10753:
URL: https://github.com/apache/iceberg/pull/10753#discussion_r1711196190
##
core/src/main/java/org/apache/iceberg/rest/auth/HttpRequestFacade.java:
##
@@ -0,0 +1,76 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * o
adutra commented on code in PR #10753:
URL: https://github.com/apache/iceberg/pull/10753#discussion_r1711190677
##
core/src/main/java/org/apache/iceberg/rest/auth/HttpRequestFacade.java:
##
@@ -0,0 +1,76 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * o
liurenjie1024 commented on PR #512:
URL: https://github.com/apache/iceberg-rust/pull/512#issuecomment-2277578710
cc @sdd Would you mind to update pr to resolve conflicts?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
nastra merged PR #10818:
URL: https://github.com/apache/iceberg/pull/10818
--
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
1 - 100 of 131 matches
Mail list logo
