nk1506 commented on code in PR #8804:
URL: https://github.com/apache/iceberg/pull/8804#discussion_r1372617190
##
aws/src/integration/java/org/apache/iceberg/aws/TestAssumeRoleAwsClientFactory.java:
##
@@ -189,7 +192,18 @@ public void testAssumeRoleS3FileIO() throws Exception {
nk1506 commented on code in PR #8804:
URL: https://github.com/apache/iceberg/pull/8804#discussion_r1372614898
##
core/src/test/java/org/apache/iceberg/hadoop/TestHadoopCommits.java:
##
@@ -435,13 +437,11 @@ public void testConcurrentFastAppends(@TempDir File dir)
throws Excepti
nk1506 commented on code in PR #8907:
URL: https://github.com/apache/iceberg/pull/8907#discussion_r1372578783
##
hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveViewCatalog.java:
##
@@ -0,0 +1,1358 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under on
nk1506 commented on code in PR #8907:
URL: https://github.com/apache/iceberg/pull/8907#discussion_r1372577053
##
api/src/main/java/org/apache/iceberg/exceptions/NoSuchIcebergViewException.java:
##
@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under on
stevenzwu commented on code in PR #8803:
URL: https://github.com/apache/iceberg/pull/8803#discussion_r1372568261
##
core/src/main/java/org/apache/iceberg/BaseFile.java:
##
@@ -504,6 +508,27 @@ private static Map
toReadableByteBufferMap(Map Map filterColumnsStats(
+ Map map
camper42 commented on issue #8863:
URL: https://github.com/apache/iceberg/issues/8863#issuecomment-1780377369
Can this caused by critical bug described in #8834 ? @bryanck
If it is, this issue could be closed
--
This is an automated message from the Apache Git Service.
To respond t
bitsondatadev commented on PR #272:
URL: https://github.com/apache/iceberg-docs/pull/272#issuecomment-1780366373
> Hey all, I'm looping back to this today.
There are lies, damned lies, and random promises I make on GitHub. Pardon my
🧠.
Just one open question but we can update t
jzhuge opened a new pull request, #8922:
URL: https://github.com/apache/iceberg/pull/8922
Honor Spark conf `spark.sql.files.maxPartitionBytes` in read split size,
right before the default value.
Here is the proposed order:
1. DataFrame read option `split-size`
2. Table prope
bitsondatadev commented on code in PR #272:
URL: https://github.com/apache/iceberg-docs/pull/272#discussion_r1372537159
##
landing-page/content/common/roadmap.md:
##
@@ -22,28 +22,36 @@ disableSidebar: true
# Roadmap Overview
-This roadmap outlines projects that the Iceberg
tomtongue closed pull request #6806: Spark 3.3: Change the default dest catalog
name for TableSnapshot
URL: https://github.com/apache/iceberg/pull/6806
--
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
aokolnychyi commented on PR #8803:
URL: https://github.com/apache/iceberg/pull/8803#issuecomment-1780243850
I took a quick look. My original guess was to do this in `ManifestReader`
but it seems Peter is right: it should be sufficient to do this in
`ManifestGroup` as we are using either `en
HonahX commented on code in PR #104:
URL: https://github.com/apache/iceberg-python/pull/104#discussion_r1372446210
##
pyiceberg/catalog/dynamodb.py:
##
@@ -80,7 +80,15 @@
class DynamoDbCatalog(Catalog):
def __init__(self, name: str, **properties: str):
super().__i
wypoon commented on code in PR #8805:
URL: https://github.com/apache/iceberg/pull/8805#discussion_r1372439543
##
core/src/test/java/org/apache/iceberg/deletes/TestPositionFilter.java:
##
@@ -282,6 +286,16 @@ public void testPositionSetRowFilter() {
@Test
public void test
github-actions[bot] closed issue #7351: UnsupportedOperationException when
using `IcebergGenerics.read` to read metadata tables
URL: https://github.com/apache/iceberg/issues/7351
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
github-actions[bot] commented on issue #7351:
URL: https://github.com/apache/iceberg/issues/7351#issuecomment-1780223574
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 issue #7455:
URL: https://github.com/apache/iceberg/issues/7455#issuecomment-1780223493
This issue has been automatically marked as stale because it has been open
for 180 days with no activity. It will be closed in next 14 days if no further
activity occurs.
github-actions[bot] commented on issue #7461:
URL: https://github.com/apache/iceberg/issues/7461#issuecomment-1780223461
This issue has been automatically marked as stale because it has been open
for 180 days with no activity. It will be closed in next 14 days if no further
activity occurs.
aokolnychyi commented on code in PR #6799:
URL: https://github.com/apache/iceberg/pull/6799#discussion_r1372337825
##
core/src/main/java/org/apache/iceberg/ManifestFiles.java:
##
@@ -157,11 +157,34 @@ public static ManifestWriter
write(PartitionSpec spec, OutputFile outp
*/
singhpk234 commented on issue #8902:
URL: https://github.com/apache/iceberg/issues/8902#issuecomment-1780043095
Ya sure take your time you are pretty close :) let me know if you need some
help here, apologies I am short on bandwidth too due to upcoming re:invent.
--
This is an automated m
singhpk234 commented on issue #8921:
URL: https://github.com/apache/iceberg/issues/8921#issuecomment-1780040874
I am not entirely sure still how would this give us or being us to snapshot
id not found as the offset returned here would be pointing to -1 snapshot id
but as you suggested earli
cccs-jc commented on issue #8902:
URL: https://github.com/apache/iceberg/issues/8902#issuecomment-1780032370
I did manage to reproduce the issue. So we'll see how it goes.
```java
@Test
public void testReadStreamWithSnapshotTypeReplaceIgnoresReplace() throws
Exception {
cccs-jc commented on issue #8902:
URL: https://github.com/apache/iceberg/issues/8902#issuecomment-1780029801
a CL, that's a pull request?
right now I'm trying to add unit tests but it's going slowly. I don't have
too many free cycles..
--
This is an automated message from the Apach
cccs-jc commented on issue #8921:
URL: https://github.com/apache/iceberg/issues/8921#issuecomment-1780023262
ya seems pretty clear that it should be picking up the offset from the
checkpoint when it exists.
could this come into the picture. There are two checks about the timestamp
he
singhpk234 commented on issue #8902:
URL: https://github.com/apache/iceberg/issues/8902#issuecomment-1780015926
yes @cccs-jc i think the right way to skip over snapshots (for the overwrite
ones) should be latestOffset Api so that we coud just say the new offset is the
one ignoring un-necess
singhpk234 commented on issue #8921:
URL: https://github.com/apache/iceberg/issues/8921#issuecomment-1780012237
I am not entirely sure why this would happen, i was under the impression
that initialOffset() is used from checkpoint file and the way it's calculated
is same as you stated above
Fokko merged PR #105:
URL: https://github.com/apache/iceberg-python/pull/105
--
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.
cccs-jc opened a new issue, #8921:
URL: https://github.com/apache/iceberg/issues/8921
### Apache Iceberg version
None
### Query engine
None
### Please describe the bug 🐞
@singhpk234 I suspect there is a bug in the Spark readStream. I'm not 100%
but we had s
dependabot[bot] opened a new pull request, #105:
URL: https://github.com/apache/iceberg-python/pull/105
Bumps [werkzeug](https://github.com/pallets/werkzeug) from 2.3.7 to 3.0.1.
Release notes
Sourced from https://github.com/pallets/werkzeug/releases";>werkzeug's
releases.
3
waifairer commented on PR #104:
URL: https://github.com/apache/iceberg-python/pull/104#issuecomment-1779931470
Done, thanks @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 go to the specific co
Fokko commented on PR #104:
URL: https://github.com/apache/iceberg-python/pull/104#issuecomment-1779912483
Thanks @waifairer for raising this! Could you also update the docs under
`mkdocs/docs/configuration.md`?
--
This is an automated message from the Apache Git Service.
To respond to th
waifairer opened a new pull request, #104:
URL: https://github.com/apache/iceberg-python/pull/104
Looks like Dynamo is lacking some properties which are documented for other
AWS-centric catalog types.
--
This is an automated message from the Apache Git Service.
To respond to the message,
Fokko commented on code in PR #61:
URL: https://github.com/apache/iceberg-python/pull/61#discussion_r1372141271
##
pyiceberg/table/snapshots.py:
##
@@ -116,3 +144,199 @@ class MetadataLogEntry(IcebergBaseModel):
class SnapshotLogEntry(IcebergBaseModel):
snapshot_id: int =
Fokko commented on code in PR #61:
URL: https://github.com/apache/iceberg-python/pull/61#discussion_r1372141271
##
pyiceberg/table/snapshots.py:
##
@@ -116,3 +144,199 @@ class MetadataLogEntry(IcebergBaseModel):
class SnapshotLogEntry(IcebergBaseModel):
snapshot_id: int =
Fokko commented on code in PR #61:
URL: https://github.com/apache/iceberg-python/pull/61#discussion_r1372144754
##
pyiceberg/table/snapshots.py:
##
@@ -116,3 +144,199 @@ class MetadataLogEntry(IcebergBaseModel):
class SnapshotLogEntry(IcebergBaseModel):
snapshot_id: int =
Fokko commented on code in PR #61:
URL: https://github.com/apache/iceberg-python/pull/61#discussion_r1372143737
##
pyiceberg/table/snapshots.py:
##
@@ -116,3 +144,199 @@ class MetadataLogEntry(IcebergBaseModel):
class SnapshotLogEntry(IcebergBaseModel):
snapshot_id: int =
Fokko commented on code in PR #61:
URL: https://github.com/apache/iceberg-python/pull/61#discussion_r1372141271
##
pyiceberg/table/snapshots.py:
##
@@ -116,3 +144,199 @@ class MetadataLogEntry(IcebergBaseModel):
class SnapshotLogEntry(IcebergBaseModel):
snapshot_id: int =
Fokko commented on code in PR #61:
URL: https://github.com/apache/iceberg-python/pull/61#discussion_r1372139436
##
pyiceberg/table/snapshots.py:
##
@@ -116,3 +144,199 @@ class MetadataLogEntry(IcebergBaseModel):
class SnapshotLogEntry(IcebergBaseModel):
snapshot_id: int =
Fokko commented on code in PR #61:
URL: https://github.com/apache/iceberg-python/pull/61#discussion_r1372139436
##
pyiceberg/table/snapshots.py:
##
@@ -116,3 +144,199 @@ class MetadataLogEntry(IcebergBaseModel):
class SnapshotLogEntry(IcebergBaseModel):
snapshot_id: int =
Fokko commented on code in PR #61:
URL: https://github.com/apache/iceberg-python/pull/61#discussion_r1372134598
##
pyiceberg/table/snapshots.py:
##
@@ -116,3 +144,199 @@ class MetadataLogEntry(IcebergBaseModel):
class SnapshotLogEntry(IcebergBaseModel):
snapshot_id: int =
rdblue merged PR #8912:
URL: https://github.com/apache/iceberg/pull/8912
--
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 opened a new pull request, #8920:
URL: https://github.com/apache/iceberg/pull/8920
The purpose of caching auth session for tables is mainly so that we can stop
refreshing the session when it isn't used anymore
--
This is an automated message from the Apache Git Service.
To respond
Fokko commented on PR #103:
URL: https://github.com/apache/iceberg-python/pull/103#issuecomment-1779742734
Thanks for the prompt review @nastra & @amogh-jahagirdar 🙌
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
Fokko merged PR #103:
URL: https://github.com/apache/iceberg-python/pull/103
--
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.
Fokko commented on PR #8906:
URL: https://github.com/apache/iceberg/pull/8906#issuecomment-1779741555
@ajantha-bhat thanks for the context, closing this one
--
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
Fokko closed pull request #8906: Built: Update revapi to 1.4.1
URL: https://github.com/apache/iceberg/pull/8906
--
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,
amogh-jahagirdar merged PR #8868:
URL: https://github.com/apache/iceberg/pull/8868
--
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...@ic
jackye1995 merged PR #8914:
URL: https://github.com/apache/iceberg/pull/8914
--
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.
jackye1995 commented on PR #8914:
URL: https://github.com/apache/iceberg/pull/8914#issuecomment-1779669087
thanks for the fix, and thanks for the reviews!
--
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 t
pvary commented on code in PR #8803:
URL: https://github.com/apache/iceberg/pull/8803#discussion_r1371956676
##
api/src/main/java/org/apache/iceberg/ContentFile.java:
##
@@ -165,6 +166,20 @@ default Long fileSequenceNumber() {
*/
F copyWithoutStats();
+ /**
+ * Copie
pvary commented on code in PR #8803:
URL: https://github.com/apache/iceberg/pull/8803#discussion_r1371955308
##
core/src/main/java/org/apache/iceberg/BaseFile.java:
##
@@ -174,8 +177,10 @@ public PartitionData copy() {
*
* @param toCopy a generic data file to copy.
*
pvary commented on code in PR #8803:
URL: https://github.com/apache/iceberg/pull/8803#discussion_r1371954933
##
api/src/main/java/org/apache/iceberg/ContentFile.java:
##
@@ -165,6 +166,20 @@ default Long fileSequenceNumber() {
*/
F copyWithoutStats();
+ /**
+ * Copie
pvary commented on code in PR #8803:
URL: https://github.com/apache/iceberg/pull/8803#discussion_r1371953259
##
core/src/main/java/org/apache/iceberg/BaseFile.java:
##
@@ -504,6 +508,27 @@ private static Map
toReadableByteBufferMap(Map Map filterColumnsStats(
+ Map map, Se
pvary commented on code in PR #8803:
URL: https://github.com/apache/iceberg/pull/8803#discussion_r1371948395
##
core/src/main/java/org/apache/iceberg/BaseFile.java:
##
@@ -504,6 +508,27 @@ private static Map
toReadableByteBufferMap(Map Map filterColumnsStats(
+ Map map, Se
pvary commented on code in PR #8803:
URL: https://github.com/apache/iceberg/pull/8803#discussion_r1371944613
##
core/src/main/java/org/apache/iceberg/BaseFile.java:
##
@@ -504,6 +508,27 @@ private static Map
toReadableByteBufferMap(Map Map filterColumnsStats(
+ Map map, Se
adutra commented on code in PR #8857:
URL: https://github.com/apache/iceberg/pull/8857#discussion_r1371727433
##
nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java:
##
@@ -181,133 +185,223 @@ public IcebergTable table(TableIdentifier
tableIdentifier) {
}
Fokko commented on PR #8878:
URL: https://github.com/apache/iceberg/pull/8878#issuecomment-1779220520
Thanks everyone for the review and 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
Fokko merged PR #8878:
URL: https://github.com/apache/iceberg/pull/8878
--
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 commented on PR #8878:
URL: https://github.com/apache/iceberg/pull/8878#issuecomment-1779218865
> I didn't find the rules to decide when to rebase and when to squash the PR
in the docs. Are they both used/needed, or only the squash?
The squash and merge is the normal way of merg
Fokko commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1371717288
##
.github/workflows/spark-ci.yml:
##
@@ -141,4 +142,4 @@ jobs:
with:
name: test logs
path: |
-**/build/testlogs
\ No newline a
amogh-jahagirdar commented on code in PR #8804:
URL: https://github.com/apache/iceberg/pull/8804#discussion_r1371705732
##
aws/src/integration/java/org/apache/iceberg/aws/lakeformation/LakeFormationTestBase.java:
##
@@ -417,7 +431,16 @@ private static void registerResource(Strin
nastra opened a new pull request, #291:
URL: https://github.com/apache/iceberg-docs/pull/291
(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-
nastra commented on code in PR #8804:
URL: https://github.com/apache/iceberg/pull/8804#discussion_r1371698377
##
aws/src/integration/java/org/apache/iceberg/aws/TestAssumeRoleAwsClientFactory.java:
##
@@ -189,7 +192,18 @@ public void testAssumeRoleS3FileIO() throws Exception {
nastra commented on code in PR #8804:
URL: https://github.com/apache/iceberg/pull/8804#discussion_r1371693544
##
aws/src/integration/java/org/apache/iceberg/aws/TestAssumeRoleAwsClientFactory.java:
##
@@ -189,7 +192,18 @@ public void testAssumeRoleS3FileIO() throws Exception {
Fokko commented on code in PR #8914:
URL: https://github.com/apache/iceberg/pull/8914#discussion_r1371690577
##
open-api/rest-catalog-open-api.yaml:
##
@@ -2322,13 +2322,22 @@ components:
description: Authorization scope for client credentials or token
exchange
Fokko merged PR #290:
URL: https://github.com/apache/iceberg-docs/pull/290
--
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
Fokko merged PR #288:
URL: https://github.com/apache/iceberg-docs/pull/288
--
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
nastra opened a new pull request, #290:
URL: https://github.com/apache/iceberg-docs/pull/290
(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-
bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1371640541
##
.github/workflows/spark-ci.yml:
##
@@ -141,4 +142,4 @@ jobs:
with:
name: test logs
path: |
-**/build/testlogs
\ No n
nastra merged PR #8896:
URL: https://github.com/apache/iceberg/pull/8896
--
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 #8893:
URL: https://github.com/apache/iceberg/pull/8893
--
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 #8897:
URL: https://github.com/apache/iceberg/pull/8897
--
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 #8879:
URL: https://github.com/apache/iceberg/pull/8879
--
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
cccs-jc commented on issue #8902:
URL: https://github.com/apache/iceberg/issues/8902#issuecomment-1779110745
@singhpk234 just a few quick questions. see above.
--
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
nastra merged PR #8900:
URL: https://github.com/apache/iceberg/pull/8900
--
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 commented on code in PR #8907:
URL: https://github.com/apache/iceberg/pull/8907#discussion_r1371621034
##
hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveViewCatalog.java:
##
@@ -0,0 +1,1358 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under on
nastra commented on code in PR #8907:
URL: https://github.com/apache/iceberg/pull/8907#discussion_r1371613818
##
api/src/main/java/org/apache/iceberg/exceptions/NoSuchIcebergViewException.java:
##
@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under on
nastra commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1371601349
##
.github/workflows/spark-ci.yml:
##
@@ -141,4 +142,4 @@ jobs:
with:
name: test logs
path: |
-**/build/testlogs
\ No newline
bitsondatadev opened a new pull request, #8919:
URL: https://github.com/apache/iceberg/pull/8919
Revert back to using the `site` directory which contains the monorepo of the
site and versioned docs. Also move back to using the asf infra standards
including the [asf-site](https://github.com/
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1371583467
##
hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalogNew.java:
##
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
ajantha-bhat commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1371580522
##
hive-metastore/src/test/java/org/apache/iceberg/hive/HiveMetastoreSetup.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1371579935
##
hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalogNew.java:
##
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
ajantha-bhat commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1371550325
##
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##
@@ -261,6 +261,12 @@ public void renameTable(TableIdentifier from,
TableIdentifier or
nastra commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1371576645
##
hive-metastore/src/test/java/org/apache/iceberg/hive/HiveMetastoreSetup.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
ismailsimsek commented on code in PR #8340:
URL: https://github.com/apache/iceberg/pull/8340#discussion_r1371571888
##
core/src/test/java/org/apache/iceberg/jdbc/TestJdbcCatalog.java:
##
@@ -601,15 +601,81 @@ public void testDropNamespace() {
public void testCreateNamespace()
ismailsimsek commented on code in PR #8340:
URL: https://github.com/apache/iceberg/pull/8340#discussion_r1371566535
##
core/src/test/java/org/apache/iceberg/jdbc/TestJdbcCatalog.java:
##
@@ -604,6 +604,27 @@ public void testCreateNamespace() {
// Test with no metadata
josgarmar14 commented on issue #8908:
URL: https://github.com/apache/iceberg/issues/8908#issuecomment-1778946237
I think everything is correct. Once the table is created I execute:
```
query = f"""ALTER TABLE
job_catalog.{args['glue_db']}.{args['glue_iceberg_tablename']} SET
TBLPR
okayhooni commented on PR #8870:
URL: https://github.com/apache/iceberg/pull/8870#issuecomment-1778670668
@amogh-jahagirdar
Sorry for the late response and thanks to handle this issue..!
Yes, there is a trailing slash on the database location, as you said..! so,
your commit wi
87 matches
Mail list logo
