yabola commented on code in PR #9710:
URL: https://github.com/apache/iceberg/pull/9710#discussion_r1510203932
##
api/src/test/java/org/apache/iceberg/transforms/TestNotStartsWith.java:
##
@@ -106,23 +75,23 @@ public void
testTruncateStringWhenProjectedPredicateTermIsLongerThanW
yabola commented on code in PR #9710:
URL: https://github.com/apache/iceberg/pull/9710#discussion_r1510203932
##
api/src/test/java/org/apache/iceberg/transforms/TestNotStartsWith.java:
##
@@ -106,23 +75,23 @@ public void
testTruncateStringWhenProjectedPredicateTermIsLongerThanW
yabola commented on PR #9710:
URL: https://github.com/apache/iceberg/pull/9710#issuecomment-1975070635
@amogh-jahagirdar Thanks for your review! I have updated my code.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
yabola commented on code in PR #9710:
URL: https://github.com/apache/iceberg/pull/9710#discussion_r1510200997
##
api/src/test/java/org/apache/iceberg/expressions/TestEvaluator.java:
##
@@ -348,6 +352,102 @@ public void testNotStartsWith() {
.isTrue();
}
+ @Test
+
ajantha-bhat commented on PR #9853:
URL: https://github.com/apache/iceberg/pull/9853#issuecomment-1975066720
LGTM.
How can we catch these errors early? Can we add the testcase?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
dependabot[bot] commented on PR #9801:
URL: https://github.com/apache/iceberg/pull/9801#issuecomment-1975023077
Superseded by #9859.
--
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
dependabot[bot] closed pull request #9801: Build: Bump mkdocs-material from
9.5.9 to 9.5.11
URL: https://github.com/apache/iceberg/pull/9801
--
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 speci
dependabot[bot] opened a new pull request, #9859:
URL: https://github.com/apache/iceberg/pull/9859
Bumps [mkdocs-material](https://github.com/squidfunk/mkdocs-material) from
9.5.9 to 9.5.12.
Release notes
Sourced from https://github.com/squidfunk/mkdocs-material/releases";>mkdocs-m
dependabot[bot] opened a new pull request, #9858:
URL: https://github.com/apache/iceberg/pull/9858
Bumps
[com.palantir.baseline:gradle-baseline-java](https://github.com/palantir/gradle-baseline)
from 4.42.0 to 5.43.0.
Release notes
Sourced from https://github.com/palantir/gradle-b
dependabot[bot] opened a new pull request, #9857:
URL: https://github.com/apache/iceberg/pull/9857
Bumps [com.azure:azure-sdk-bom](https://github.com/azure/azure-sdk-for-java)
from 1.2.20 to 1.2.21.
Commits
https://github.com/Azure/azure-sdk-for-java/commit/d39c06b791872b97433b
dependabot[bot] closed pull request #9795: Build: Bump
software.amazon.awssdk:bom from 2.24.5 to 2.24.10
URL: https://github.com/apache/iceberg/pull/9795
--
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
dependabot[bot] commented on PR #9795:
URL: https://github.com/apache/iceberg/pull/9795#issuecomment-1975015172
Superseded by #9856.
--
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
dependabot[bot] opened a new pull request, #9856:
URL: https://github.com/apache/iceberg/pull/9856
Bumps software.amazon.awssdk:bom from 2.24.5 to 2.25.1.
[ -> Dat
return database_input
bitsondatadev commented on code in PR #9843:
URL: https://github.com/apache/iceberg/pull/9843#discussion_r1510039686
##
format/spec.md:
##
@@ -791,7 +788,7 @@ Each version of table metadata is stored in a metadata
folder under the table’
Notes:
-1. The file system table sc
rtadepalli commented on code in PR #68:
URL: https://github.com/apache/iceberg-go/pull/68#discussion_r1510032756
##
cmd/iceberg/main.go:
##
@@ -53,12 +58,22 @@ Arguments:
VALUE value to set
Options:
- -h --help show this helpe messages and exit
--cata
rtadepalli opened a new pull request, #68:
URL: https://github.com/apache/iceberg-go/pull/68
Add some verbose descriptions (similar to PyIceberg) for the CLI options
when `iceberg --help` is used.
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
kevinjqliu closed issue #326: Add example of using PyIceberg with minimal
external dependencies
URL: https://github.com/apache/iceberg-python/issues/326
--
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
kevinjqliu opened a new issue, #492:
URL: https://github.com/apache/iceberg-python/issues/492
### Apache Iceberg version
main (development)
### Please describe the bug 🐞
As of [current
commit](https://github.com/apache/iceberg-python/commit/36b56eb7bbc4cff91e4b9bfdf71ca0
rdblue commented on PR #9853:
URL: https://github.com/apache/iceberg/pull/9853#issuecomment-1974887622
Okay, I can confirm that this fixes the problem locally.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
kevinjqliu commented on issue #290:
URL: https://github.com/apache/iceberg-python/issues/290#issuecomment-1974887039
Awesome. Thank you @anupam-saini
I have another follow-up PR that adds `schema_id` checks in places where it
was previously implicitly checked #487. We should be good
rdblue commented on code in PR #9853:
URL: https://github.com/apache/iceberg/pull/9853#discussion_r1510022119
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -1058,12 +1058,17 @@ public List listViews(SessionContext
context, Namespace namespa
pub
danielcweeks commented on code in PR #9853:
URL: https://github.com/apache/iceberg/pull/9853#discussion_r1510022083
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -1058,12 +1058,17 @@ public List listViews(SessionContext
context, Namespace namespa
rdblue opened a new pull request, #9853:
URL: https://github.com/apache/iceberg/pull/9853
This is a fix for Spark table loading when a REST catalog does not support
view extensions.
The problem is that Spark runs `ResolveViews` before `ResolveTables` in the
substitution batch of rule
viirya commented on code in PR #226:
URL: https://github.com/apache/iceberg-rust/pull/226#discussion_r1510017267
##
crates/iceberg/src/spec/values.rs:
##
@@ -265,7 +265,7 @@ impl Datum {
}
}
-/// Creates a date in `%Y-%m-%d` format, assume in utc timezone.
+
viirya commented on code in PR #226:
URL: https://github.com/apache/iceberg-rust/pull/226#discussion_r1510017188
##
crates/iceberg/src/spec/datatypes.rs:
##
@@ -171,13 +171,13 @@ pub enum PrimitiveType {
Int,
/// 64-bit signed integer
Long,
-/// 32-bit IEEE 75
viirya commented on code in PR #226:
URL: https://github.com/apache/iceberg-rust/pull/226#discussion_r1510017188
##
crates/iceberg/src/spec/datatypes.rs:
##
@@ -171,13 +171,13 @@ pub enum PrimitiveType {
Int,
/// 64-bit signed integer
Long,
-/// 32-bit IEEE 75
viirya opened a new pull request, #226:
URL: https://github.com/apache/iceberg-rust/pull/226
(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-
nk1506 commented on PR #9682:
URL: https://github.com/apache/iceberg/pull/9682#issuecomment-1974868108
> Can we please continue this effort , it has been stalled 2 weeks?
>
> Let's have a PR that is the end to end version (please not on your own
github, but against Iceberg github).
nk1506 opened a new pull request, #9852:
URL: https://github.com/apache/iceberg/pull/9852
This changes include:
1. Introduction of common metadata interface(BaseMetadata) for table and
view.
2. Refactor for HiveTableOperation to have common code for table and view
commits.
Ref
rustyconover commented on PR #8625:
URL: https://github.com/apache/iceberg/pull/8625#issuecomment-1974820592
Any updates on this?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comm
srilman commented on issue #491:
URL: https://github.com/apache/iceberg-python/issues/491#issuecomment-1974819638
> When the filter is always true for all output files, we can skip the
row-level filter in DataScan.to_arrow
I have a somewhat working implementation of this optimization
srilman opened a new issue, #491:
URL: https://github.com/apache/iceberg-python/issues/491
### Feature Request / Improvement
I noticed that in DataScan.plan_files, when we apply filters at a partition
and file metadata level, all we try to determine is whether the file has rows
that
jrouly commented on code in PR #490:
URL: https://github.com/apache/iceberg-python/pull/490#discussion_r1509829128
##
pyiceberg/catalog/glue.py:
##
@@ -252,6 +259,22 @@ def _construct_database_input(database_name: str,
properties: Properties) -> Dat
return database_input
Fokko commented on PR #490:
URL: https://github.com/apache/iceberg-python/pull/490#issuecomment-1974732024
@HonahX since you're more familiar with Glue, would you be able to do this
review? :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
44 matches
Mail list logo
