sdd commented on issue #624:
URL: https://github.com/apache/iceberg-rust/issues/624#issuecomment-2342825385
I will be working on adding support for positional and identity deletes to
the table scans next. Hopefully this will help towards this goal.
--
This is an automated message from the
nastra commented on code in PR #10929:
URL: https://github.com/apache/iceberg/pull/10929#discussion_r1753345154
##
core/src/main/java/org/apache/iceberg/rest/Endpoint.java:
##
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contri
nastra commented on code in PR #10929:
URL: https://github.com/apache/iceberg/pull/10929#discussion_r1753354122
##
core/src/main/java/org/apache/iceberg/rest/responses/ConfigResponse.java:
##
@@ -153,8 +173,18 @@ public Builder withOverrides(Map
overridesToAdd) {
return
nastra commented on code in PR #10929:
URL: https://github.com/apache/iceberg/pull/10929#discussion_r1753360024
##
core/src/main/java/org/apache/iceberg/rest/Endpoint.java:
##
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contri
nastra commented on code in PR #10929:
URL: https://github.com/apache/iceberg/pull/10929#discussion_r1753366181
##
core/src/main/java/org/apache/iceberg/rest/Endpoint.java:
##
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contri
nastra commented on code in PR #10929:
URL: https://github.com/apache/iceberg/pull/10929#discussion_r1753408304
##
core/src/main/java/org/apache/iceberg/rest/ResourcePaths.java:
##
@@ -26,6 +26,56 @@
public class ResourcePaths {
private static final Joiner SLASH = Joiner.on(
nastra commented on code in PR #10929:
URL: https://github.com/apache/iceberg/pull/10929#discussion_r1753475044
##
core/src/main/java/org/apache/iceberg/rest/Endpoint.java:
##
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contri
nastra closed pull request #0: WIP: Upgrade google-java-format
URL: https://github.com/apache/iceberg/pull/0
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscr
nastra commented on PR #8337:
URL: https://github.com/apache/iceberg/pull/8337#issuecomment-2342944753
@jasonmfehr can you please fix the conflicts?
--
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
nastra opened a new issue, #1:
URL: https://github.com/apache/iceberg/issues/1
### Apache Iceberg version
main (development)
### Query engine
Kafka Connect
### Please describe the bug 🐞
```
> Task :iceberg-kafka-connect:iceberg-kafka-connect-runti
nastra commented on code in PR #11050:
URL: https://github.com/apache/iceberg/pull/11050#discussion_r1753546454
##
flink/v1.19/flink/src/test/java/org/apache/iceberg/flink/sink/TestIcebergCommitter.java:
##
@@ -1441,5 +1441,6 @@ public int hashCode() {
public TypeSerializer
SML0127 commented on issue #11081:
URL: https://github.com/apache/iceberg/issues/11081#issuecomment-2343005989
@pvary
Thx pvary! I have one question.
As I understand under code, EqualityDeletes.filter() function don't filter
delete file that has same seq number with data file.
nastra merged PR #11050:
URL: https://github.com/apache/iceberg/pull/11050
--
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
pvary commented on PR #11010:
URL: https://github.com/apache/iceberg/pull/11010#issuecomment-2343218837
@stevenzwu: I didn't get answer for my question on the Flink dev list about
the output of the PostCommitTopology. See:
https://lists.apache.org/thread/28qb1q6b7kz30dqjdbsw855osx4t38s7
nastra commented on code in PR #10995:
URL: https://github.com/apache/iceberg/pull/10995#discussion_r1753988942
##
core/src/test/java/org/apache/iceberg/TestFastAppend.java:
##
@@ -252,11 +252,36 @@ public void testFailure() {
assertThat(new File(newManifest.path())).doesNo
nastra commented on code in PR #10995:
URL: https://github.com/apache/iceberg/pull/10995#discussion_r1754002089
##
core/src/main/java/org/apache/iceberg/FastAppend.java:
##
@@ -24,20 +24,14 @@
import java.util.Set;
import org.apache.iceberg.encryption.EncryptedOutputFile;
imp
manuzhang commented on code in PR #11048:
URL: https://github.com/apache/iceberg/pull/11048#discussion_r1754081441
##
spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/execution/datasources/v2/CreateOrReplaceTagExec.scala:
##
@@ -48,7 +48,8 @@ case class CreateOrRe
nastra commented on code in PR #11048:
URL: https://github.com/apache/iceberg/pull/11048#discussion_r1754101092
##
spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/execution/datasources/v2/CreateOrReplaceTagExec.scala:
##
@@ -48,7 +48,8 @@ case class CreateOrRepla
nastra commented on code in PR #11048:
URL: https://github.com/apache/iceberg/pull/11048#discussion_r1754106397
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestTagDDL.java:
##
@@ -45,19 +43,9 @@
public class TestTagDDL extends ExtensionsTest
nastra commented on code in PR #11048:
URL: https://github.com/apache/iceberg/pull/11048#discussion_r1754106397
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestTagDDL.java:
##
@@ -45,19 +43,9 @@
public class TestTagDDL extends ExtensionsTest
nastra closed pull request #9940: OpenAPI: Express server capabilities via
/config endpoint
URL: https://github.com/apache/iceberg/pull/9940
--
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
nastra commented on PR #9940:
URL: https://github.com/apache/iceberg/pull/9940#issuecomment-2343350495
closing this in favor of #10928
--
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 specifi
chenzl25 opened a new issue, #627:
URL: https://github.com/apache/iceberg-rust/issues/627
As we know, `FileScanTask` has two fields `project_field_ids` and `schema`.
I think the `RecordBatch` from the reader of this `FileScanTask` should always
follow the schema specified in `FileScanTask`.
manuzhang closed pull request #11048: Spark 3.5: Fix TestTagDDL only testing
with SparkSessionCatalog
URL: https://github.com/apache/iceberg/pull/11048
--
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
manuzhang commented on code in PR #11048:
URL: https://github.com/apache/iceberg/pull/11048#discussion_r1754220227
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestTagDDL.java:
##
@@ -45,19 +43,9 @@
public class TestTagDDL extends ExtensionsT
manuzhang commented on PR #10811:
URL: https://github.com/apache/iceberg/pull/10811#issuecomment-2343454242
cc @nastra 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 the specific com
nastra commented on code in PR #10811:
URL: https://github.com/apache/iceberg/pull/10811#discussion_r1754328006
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/source/TestDataFrameWrites.java:
##
@@ -419,5 +421,13 @@ public void testFaultToleranceOnWrite() throws IOEx
slessard commented on code in PR #10953:
URL: https://github.com/apache/iceberg/pull/10953#discussion_r1753042474
##
arrow/src/test/java/org/apache/iceberg/arrow/vectorized/ArrowReaderTest.java:
##
@@ -262,6 +263,89 @@ public void testReadColumnFilter2() throws Exception {
ndrluis commented on code in PR #1161:
URL: https://github.com/apache/iceberg-python/pull/1161#discussion_r1754541566
##
pyiceberg/table/update/spec.py:
##
@@ -280,7 +280,7 @@ def _partition_field(self, transform_key: Tuple[int,
Transform[Any, Any]], name:
for fiel
Xuanwo commented on PR #628:
URL: https://github.com/apache/iceberg-rust/pull/628#issuecomment-2343720482
Hi, @sungwy, would you like to take a review? I believe this way is more
pythonic.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
pvary commented on issue #11081:
URL: https://github.com/apache/iceberg/issues/11081#issuecomment-2343745305
@SML0127: I would suggest to write some unit test, or chose an existing one
to test the behaviour. I have not seen complains about this yet.
--
This is an automated message fro
zeroshade commented on PR #145:
URL: https://github.com/apache/iceberg-go/pull/145#issuecomment-2343773844
@alex-kar we could probably leverage "testable examples"
https://go.dev/blog/examples with go and just use `// Output:` stuff to point
out our expected output.
--
This is an automat
bryanck commented on code in PR #11075:
URL: https://github.com/apache/iceberg/pull/11075#discussion_r1754685987
##
kafka-connect/kafka-connect-runtime/src/integration/java/org/apache/iceberg/connect/TestContext.java:
##
@@ -51,6 +52,7 @@ public class TestContext {
private Te
nastra commented on code in PR #10722:
URL: https://github.com/apache/iceberg/pull/10722#discussion_r1754684549
##
open-api/rest-catalog-open-api.yaml:
##
@@ -2747,6 +2747,81 @@ components:
uuid:
type: string
+ADLSCredentials:
+ type: object
+
nastra commented on code in PR #10722:
URL: https://github.com/apache/iceberg/pull/10722#discussion_r1754694285
##
open-api/rest-catalog-open-api.yaml:
##
@@ -2747,6 +2747,81 @@ components:
uuid:
type: string
+ADLSCredentials:
+ type: object
+
nastra commented on code in PR #10722:
URL: https://github.com/apache/iceberg/pull/10722#discussion_r1754684549
##
open-api/rest-catalog-open-api.yaml:
##
@@ -2747,6 +2747,81 @@ components:
uuid:
type: string
+ADLSCredentials:
+ type: object
+
JanKrl commented on issue #1093:
URL:
https://github.com/apache/iceberg-python/issues/1093#issuecomment-2343901577
> Adding some context to the Avro decoder build process.
>
> We use Poetry to build the Avro decoder via this script
>
>
https://github.com/apache/iceberg-python/
nastra closed pull request #10839: OpenAPI: Use %2E as namespace separator
instead of %1F
URL: https://github.com/apache/iceberg/pull/10839
--
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
nastra commented on PR #10839:
URL: https://github.com/apache/iceberg/pull/10839#issuecomment-2343901990
closing this in favor of https://github.com/apache/iceberg/pull/10877
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
ajantha-bhat commented on PR #10176:
URL: https://github.com/apache/iceberg/pull/10176#issuecomment-2343906548
> I just learned that there is an ongoing effort to support writes with the
internal Iceberg object model, which will nicely solve this problem for us
(given that it is already in
jwtryg commented on code in PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#discussion_r1754878148
##
catalog/catalog.go:
##
@@ -146,9 +147,9 @@ type Catalog interface {
// identifiers containing the information required to load the table
via that catalog.
jwtryg commented on code in PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#discussion_r1754883143
##
catalog/catalog.go:
##
@@ -146,9 +147,9 @@ type Catalog interface {
// identifiers containing the information required to load the table
via that catalog.
jshmchenxi closed issue #1865: Add support for building runtime jars containing
the sources
URL: https://github.com/apache/iceberg/issues/1865
--
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
amogh-jahagirdar commented on code in PR #11044:
URL: https://github.com/apache/iceberg/pull/11044#discussion_r1754922979
##
core/src/main/java/org/apache/iceberg/InheritableMetadataFactory.java:
##
@@ -46,11 +49,14 @@ static class BaseInheritableMetadata implements
Inheritable
jwtryg commented on code in PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#discussion_r1754949535
##
partitions.go:
##
@@ -117,6 +117,10 @@ func (ps PartitionSpec) Equals(other PartitionSpec) bool {
return ps.id == other.id && slices.Equal(ps.fields, other.f
jwtryg commented on code in PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#discussion_r1754951127
##
table/metadata.go:
##
@@ -80,20 +86,512 @@ type Metadata interface {
SnapshotByName(name string) *Snapshot
// CurrentSnapshot returns the table's cur
jwtryg commented on code in PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#discussion_r1754957335
##
table/metadata.go:
##
@@ -80,20 +86,512 @@ type Metadata interface {
SnapshotByName(name string) *Snapshot
// CurrentSnapshot returns the table's cur
jwtryg commented on code in PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#discussion_r1754962455
##
table/metadata.go:
##
@@ -80,20 +86,512 @@ type Metadata interface {
SnapshotByName(name string) *Snapshot
// CurrentSnapshot returns the table's cur
jwtryg commented on code in PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#discussion_r1754963661
##
table/metadata.go:
##
@@ -80,20 +86,512 @@ type Metadata interface {
SnapshotByName(name string) *Snapshot
// CurrentSnapshot returns the table's cur
Xuanwo commented on PR #628:
URL: https://github.com/apache/iceberg-rust/pull/628#issuecomment-2344001364
> There's an outstanding issue with pyo3 that prevents the functions defined
within a module to be imported like below:
Oh, I see the problem. Let me take a look.
--
This is an
jwtryg commented on code in PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#discussion_r1754973597
##
table/metadata.go:
##
@@ -370,26 +876,26 @@ func (c *commonMetadata) validate() error {
func (c *commonMetadata) Version() int { return c.FormatVersion }
-type
Xuanwo commented on PR #628:
URL: https://github.com/apache/iceberg-rust/pull/628#issuecomment-2344019903
Hi, @sungwy, I fixed this issue and add a test for it. Please take a look,
thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
nastra closed issue #1: KC Integration tests occasionally fail
URL: https://github.com/apache/iceberg/issues/1
--
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 unsubs
nastra commented on code in PR #11075:
URL: https://github.com/apache/iceberg/pull/11075#discussion_r1755050156
##
kafka-connect/kafka-connect-runtime/src/integration/java/org/apache/iceberg/connect/TestContext.java:
##
@@ -51,6 +52,7 @@ public class TestContext {
private Tes
Xuanwo merged PR #144:
URL: https://github.com/apache/iceberg-go/pull/144
--
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.apa
nastra commented on PR #141:
URL: https://github.com/apache/iceberg-go/pull/141#issuecomment-2344084492
@dependabot rebase
--
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
dependabot[bot] commented on PR #141:
URL: https://github.com/apache/iceberg-go/pull/141#issuecomment-2344084514
Looks like github.com/aws/aws-sdk-go-v2 is up-to-date now, so this is no
longer needed.
--
This is an automated message from the Apache Git Service.
To respond to the message,
dependabot[bot] commented on PR #141:
URL: https://github.com/apache/iceberg-go/pull/141#issuecomment-2344084630
Looks like this PR is closed. If you re-open it, I'll rebase it, as long as
no-one else has edited it.
--
This is an automated message from the Apache Git Service.
To respo
dependabot[bot] closed pull request #141: build(deps): bump
github.com/aws/aws-sdk-go-v2 from 1.30.4 to 1.30.5
URL: https://github.com/apache/iceberg-go/pull/141
--
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
Xuanwo merged PR #140:
URL: https://github.com/apache/iceberg-go/pull/140
--
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.apa
dependabot[bot] closed pull request #143: build(deps): bump
github.com/aws/aws-sdk-go-v2/credentials from 1.17.29 to 1.17.32
URL: https://github.com/apache/iceberg-go/pull/143
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
dependabot[bot] commented on PR #143:
URL: https://github.com/apache/iceberg-go/pull/143#issuecomment-2344096285
Looks like github.com/aws/aws-sdk-go-v2/credentials is up-to-date now, so
this is no longer needed.
--
This is an automated message from the Apache Git Service.
To respond to t
szehon-ho commented on code in PR #10774:
URL: https://github.com/apache/iceberg/pull/10774#discussion_r1755110580
##
spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/catalyst/parser/extensions/IcebergSqlExtensionsAstBuilder.scala:
##
@@ -226,11 +226,13 @@ class I
nastra commented on code in PR #10920:
URL: https://github.com/apache/iceberg/pull/10920#discussion_r1755114815
##
core/src/main/java/org/apache/iceberg/actions/BaseRewriteTablePathActionResult.java:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) un
szehon-ho commented on code in PR #10774:
URL: https://github.com/apache/iceberg/pull/10774#discussion_r1755120322
##
spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/catalyst/parser/extensions/IcebergSqlExtensionsAstBuilder.scala:
##
@@ -226,11 +226,13 @@ class I
Xuanwo commented on PR #142:
URL: https://github.com/apache/iceberg-go/pull/142#issuecomment-2344131523
Hi, @zeroshade, the tests failed for:
```shell
=== RUN
TestScanner/test_partitioned_by_identity_GreaterThanEqual(term=Reference(name='ts'),_literal=2023-03-05T00:00:00+00:00)
kevinjqliu commented on issue #1093:
URL:
https://github.com/apache/iceberg-python/issues/1093#issuecomment-2344154285
Great to hear. did you clean the env manually or use `make clean`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
JanKrl commented on issue #1093:
URL:
https://github.com/apache/iceberg-python/issues/1093#issuecomment-2344182977
> Great to hear. did you clean the env manually or use `make clean`?
I use venv, so I removed `.venv` directory and created it again.
--
This is an automated message f
kevinjqliu commented on code in PR #628:
URL: https://github.com/apache/iceberg-rust/pull/628#discussion_r1755178187
##
bindings/python/src/transform.rs:
##
@@ -15,24 +15,55 @@
// specific language governing permissions and limitations
// under the License.
+use arrow::array
hashhar commented on issue #4563:
URL: https://github.com/apache/iceberg/issues/4563#issuecomment-2344223755
cc: @rdblue Seems this is important. See
https://github.com/apache/iceberg/pull/5707#issuecomment-1980199024 for why
this is important.
--
This is an automated message from the Ap
haizhou-zhao commented on issue #11103:
URL: https://github.com/apache/iceberg/issues/11103#issuecomment-2344239473
I realized I've been putting many thought threads that doesn't directly
related to this problem, making it hard for people to read.
The problem really is: the current R
kevinjqliu opened a new issue, #1162:
URL: https://github.com/apache/iceberg-python/issues/1162
### Apache Iceberg version
main (development)
### Please describe the bug 🐞
On Mac, anyone having issue running `make test-integration` on the current
`main` branch?
I
flyrain commented on code in PR #10722:
URL: https://github.com/apache/iceberg/pull/10722#discussion_r1755210069
##
open-api/rest-catalog-open-api.yaml:
##
@@ -3103,6 +3103,81 @@ components:
uuid:
type: string
+ADLSCredentials:
+ type: object
+
Xuanwo commented on code in PR #628:
URL: https://github.com/apache/iceberg-rust/pull/628#discussion_r1755216295
##
bindings/python/src/transform.rs:
##
@@ -15,24 +15,55 @@
// specific language governing permissions and limitations
// under the License.
+use arrow::array::{m
kevinjqliu commented on PR #950:
URL: https://github.com/apache/iceberg-python/pull/950#issuecomment-2344286460
@HonahX do you mind taking a look at this when you get a chance?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
kevinjqliu commented on code in PR #628:
URL: https://github.com/apache/iceberg-rust/pull/628#discussion_r1755222868
##
bindings/python/src/transform.rs:
##
@@ -15,24 +15,55 @@
// specific language governing permissions and limitations
// under the License.
+use arrow::array
dramaticlly commented on code in PR #10935:
URL: https://github.com/apache/iceberg/pull/10935#discussion_r1755259576
##
core/src/main/java/org/apache/iceberg/BaseIncrementalChangelogScan.java:
##
@@ -63,33 +60,43 @@ protected CloseableIterable doPlanFiles(
return Closeabl
Xuanwo commented on PR #628:
URL: https://github.com/apache/iceberg-rust/pull/628#issuecomment-2344348879
Thank you @sungwy and @kevinjqliu for the quick review. We can implement
more transforms in following PRs. And I will merge this one first 😘
--
This is an automated message from the A
Xuanwo merged PR #628:
URL: https://github.com/apache/iceberg-rust/pull/628
--
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
huaxingao commented on code in PR #11106:
URL: https://github.com/apache/iceberg/pull/11106#discussion_r1755274103
##
spark/v3.3/build.gradle:
##
@@ -59,6 +59,7 @@
project(":iceberg-spark:iceberg-spark-${sparkMajorVersion}_${scalaVersion}") {
implementation project(':icebe
sdd commented on PR #608:
URL: https://github.com/apache/iceberg-rust/pull/608#issuecomment-2344385358
We've been very selective when it comes to adding new values to `ErrorKind`.
I'd personally go for `Unexpected` here - but maybe @liurenjie1024 or @Xuanwo
can confirm what would be best.
soumya-ghosh commented on issue #1162:
URL:
https://github.com/apache/iceberg-python/issues/1162#issuecomment-2344408943
Been facing the same issue since yesterday when I rebased latest main branch
to my dev branch. Just realized that tests are not running on main branch
itself.
I did
hesham-medhat commented on code in PR #11039:
URL: https://github.com/apache/iceberg/pull/11039#discussion_r1755319360
##
build.gradle:
##
@@ -647,20 +647,43 @@ project(':iceberg-gcp') {
implementation project(':iceberg-common')
implementation project(':iceberg-core')
hesham-medhat commented on code in PR #11039:
URL: https://github.com/apache/iceberg/pull/11039#discussion_r1755322285
##
.gitignore:
##
@@ -66,3 +66,6 @@ metastore_db/
# Spark/metastore files
spark-warehouse/
derby.log
+
+# BigQuery/metastore files
+gcp/db_folder/
Review Co
hesham-medhat commented on code in PR #11039:
URL: https://github.com/apache/iceberg/pull/11039#discussion_r1755320886
##
build.gradle:
##
@@ -647,20 +647,43 @@ project(':iceberg-gcp') {
implementation project(':iceberg-common')
implementation project(':iceberg-core')
isc-patrick commented on PR #1155:
URL: https://github.com/apache/iceberg-python/pull/1155#issuecomment-2344422315
I can certainly add tests, but that is really testing the
Metadata.create_all() function in SQLAlchemy and not pyiceberg code. I think
that CREATE TABLE IF NOT EXISTS requires
aniket-s-kulkarni commented on code in PR #10722:
URL: https://github.com/apache/iceberg/pull/10722#discussion_r1755358145
##
open-api/rest-catalog-open-api.yaml:
##
@@ -3103,6 +3103,81 @@ components:
uuid:
type: string
+ADLSCredentials:
+ type: ob
dataders opened a new issue, #1163:
URL: https://github.com/apache/iceberg-python/issues/1163
### Apache Iceberg version
0.7.1 (latest release)
### Please describe the bug 🐞
the docstring for `catalog.list_tables()` says
> If namespace not provided, will list all table
rdblue commented on code in PR #11044:
URL: https://github.com/apache/iceberg/pull/11044#discussion_r1755367412
##
core/src/main/java/org/apache/iceberg/BaseFile.java:
##
@@ -240,6 +242,10 @@ public void setDataSequenceNumber(Long dataSequenceNumber)
{
this.dataSequenceNum
kevinjqliu closed issue #1164: Cannot parse missing long: current-snapshot-id
URL: https://github.com/apache/iceberg-python/issues/1164
--
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
kevinjqliu commented on issue #1164:
URL:
https://github.com/apache/iceberg-python/issues/1164#issuecomment-2344451634
Also from slack,
Sung:
This is actually an issue with some of the older Java applications making
the incorrect assumption that the current_snapshot_id is a requir
kevinjqliu opened a new issue, #1164:
URL: https://github.com/apache/iceberg-python/issues/1164
### Apache Iceberg version
None
### Please describe the bug 🐞
From slack,
Hi! We’ve noticed that after creating an empty table in the Glue catalog
with PyIceberg, init
HemantMarve commented on issue #10078:
URL: https://github.com/apache/iceberg/issues/10078#issuecomment-2344456352
@clamar14 , @nastra Try after adding below property:
`.config("spark.sql.catalog.AwsDataCatalog.client.region","us-south")`
`.config("spark.sql.catalog.your_catalog
kevinjqliu commented on issue #1163:
URL:
https://github.com/apache/iceberg-python/issues/1163#issuecomment-2344471589
Thanks for reporting this! I noticed it too when reviewing #1140.
I think the spec only allows listing tables in a namespace.
https://github.com/apache/iceberg/
kevinjqliu commented on issue #1162:
URL:
https://github.com/apache/iceberg-python/issues/1162#issuecomment-2344477343
Thanks for confirming. I see the same issue
https://gist.github.com/kevinjqliu/c8310b6253beab52cce93391df03bfe4
And only for commits at and after `1971fcfe0875eeb200
dimas-b commented on code in PR #10722:
URL: https://github.com/apache/iceberg/pull/10722#discussion_r1755413798
##
open-api/rest-catalog-open-api.yaml:
##
@@ -3129,6 +3204,11 @@ components:
- `s3.secret-access-key`: secret for credentials that provide access
to data
slessard commented on code in PR #10953:
URL: https://github.com/apache/iceberg/pull/10953#discussion_r1720293723
##
arrow/src/test/java/org/apache/iceberg/arrow/vectorized/ArrowReaderTest.java:
##
@@ -262,6 +263,89 @@ public void testReadColumnFilter2() throws Exception {
dimas-b commented on code in PR #10722:
URL: https://github.com/apache/iceberg/pull/10722#discussion_r1755423596
##
open-api/rest-catalog-open-api.yaml:
##
@@ -3129,6 +3204,11 @@ components:
- `s3.secret-access-key`: secret for credentials that provide access
to data
kevinjqliu commented on PR #1155:
URL: https://github.com/apache/iceberg-python/pull/1155#issuecomment-2344509223
please do! we want to ensure that this change does not break new and
existing DB integrations.
--
This is an automated message from the Apache Git Service.
To respond to the
slessard commented on PR #10953:
URL: https://github.com/apache/iceberg/pull/10953#issuecomment-2344515203
> @sl255051 could you please update the PR title to reflect the changes that
are being done here?
I updated the PR title and description
--
This is an automated message from t
1 - 100 of 206 matches
Mail list logo
