Re: [I] Fails to Read SnowFlake Iceberg after Interoperating as Hudi and Delta in Spark [incubator-xtable]

2024-08-02 Thread via GitHub
sagarlakshmipathy commented on issue #503: URL: https://github.com/apache/incubator-xtable/issues/503#issuecomment-2266355191 Were you able to query the iceberg table from spark? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [I] Fails to Read SnowFlake Iceberg after Interoperating as Hudi and Delta in Spark [incubator-xtable]

2024-08-02 Thread via GitHub
soumilshah1995 commented on issue #503: URL: https://github.com/apache/incubator-xtable/issues/503#issuecomment-2266093688 Reregistering tables via Glue Crawlers works, allowing me to read them in Athena. Via Crawlers (Works) ![image](https://github.com/user-attachments/assets/aa

[I] Fails to Read SnowFlake Iceberg after Interoperating as Hudi and Delta in Spark [incubator-xtable]

2024-08-02 Thread via GitHub
soumilshah1995 opened a new issue, #503: URL: https://github.com/apache/incubator-xtable/issues/503 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-xtable/issues?q=is%3Aissue) and found no similar issues. ### Please describ

(incubator-xtable) branch 297-properties-based-config-2 updated (9c317ca2 -> 02bff5cf)

2024-08-02 Thread timbrown
This is an automated email from the ASF dual-hosted git repository. timbrown pushed a change to branch 297-properties-based-config-2 in repository https://gitbox.apache.org/repos/asf/incubator-xtable.git from 9c317ca2 fix compile issues add 02bff5cf fix test setup No new revisions were

(incubator-xtable) branch 297-properties-based-config-2 updated (437cc155 -> 9c317ca2)

2024-08-02 Thread timbrown
This is an automated email from the ASF dual-hosted git repository. timbrown pushed a change to branch 297-properties-based-config-2 in repository https://gitbox.apache.org/repos/asf/incubator-xtable.git from 437cc155 lint add 9c317ca2 fix compile issues No new revisions were added by

Re: [PR] [297] Refactor config classes [incubator-xtable]

2024-08-02 Thread via GitHub
the-other-tim-brown commented on code in PR #480: URL: https://github.com/apache/incubator-xtable/pull/480#discussion_r1702200344 ## xtable-api/src/main/java/org/apache/xtable/conversion/TableSyncConfig.java: ## @@ -0,0 +1,58 @@ +/* + * Licensed to the Apache Software Foundation

(incubator-xtable) branch 297-properties-based-config-2 updated: lint

2024-08-02 Thread timbrown
This is an automated email from the ASF dual-hosted git repository. timbrown pushed a commit to branch 297-properties-based-config-2 in repository https://gitbox.apache.org/repos/asf/incubator-xtable.git The following commit(s) were added to refs/heads/297-properties-based-config-2 by this push

(incubator-xtable) branch 297-properties-based-config-2 updated (08c31514 -> a1f0bcd0)

2024-08-02 Thread timbrown
This is an automated email from the ASF dual-hosted git repository. timbrown pushed a change to branch 297-properties-based-config-2 in repository https://gitbox.apache.org/repos/asf/incubator-xtable.git from 08c31514 PR feedback add a1f0bcd0 more refactoring for config naming No new r

(incubator-xtable) branch 297-properties-based-config-2 updated (6d37fb6c -> 08c31514)

2024-08-02 Thread timbrown
This is an automated email from the ASF dual-hosted git repository. timbrown pushed a change to branch 297-properties-based-config-2 in repository https://gitbox.apache.org/repos/asf/incubator-xtable.git from 6d37fb6c lint add bd627b50 Bump braces from 3.0.2 to 3.0.3 in /website ad

(incubator-xtable) 02/02: PR feedback

2024-08-02 Thread timbrown
This is an automated email from the ASF dual-hosted git repository. timbrown pushed a commit to branch 297-properties-based-config-2 in repository https://gitbox.apache.org/repos/asf/incubator-xtable.git commit 08c31514de18643bf0b1908acef889d276dc27a7 Author: Timothy Brown AuthorDate: Fri Aug 2

(incubator-xtable) 01/02: Merge remote-tracking branch 'origin/main' into 297-properties-based-config-2

2024-08-02 Thread timbrown
This is an automated email from the ASF dual-hosted git repository. timbrown pushed a commit to branch 297-properties-based-config-2 in repository https://gitbox.apache.org/repos/asf/incubator-xtable.git commit f5fc448c8ad387751672e678f21f54137240 Merge: 6d37fb6c 5baf117c Author: Timothy Brow

Re: [PR] [297] Refactor config classes [incubator-xtable]

2024-08-02 Thread via GitHub
the-other-tim-brown commented on code in PR #480: URL: https://github.com/apache/incubator-xtable/pull/480#discussion_r1702188124 ## xtable-api/src/main/java/org/apache/xtable/conversion/TableSyncConfig.java: ## @@ -0,0 +1,58 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [297] Refactor config classes [incubator-xtable]

2024-08-02 Thread via GitHub
the-other-tim-brown commented on code in PR #480: URL: https://github.com/apache/incubator-xtable/pull/480#discussion_r1702175178 ## xtable-api/src/main/java/org/apache/xtable/conversion/ExternalTable.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (

Re: [PR] [297] Refactor config classes [incubator-xtable]

2024-08-02 Thread via GitHub
the-other-tim-brown commented on code in PR #480: URL: https://github.com/apache/incubator-xtable/pull/480#discussion_r1702175178 ## xtable-api/src/main/java/org/apache/xtable/conversion/ExternalTable.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (

Re: [PR] [297] Refactor config classes [incubator-xtable]

2024-08-02 Thread via GitHub
ashvina commented on code in PR #480: URL: https://github.com/apache/incubator-xtable/pull/480#discussion_r1702149420 ## xtable-api/src/main/java/org/apache/xtable/conversion/TableSyncConfig.java: ## @@ -0,0 +1,58 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [297] Refactor config classes [incubator-xtable]

2024-08-02 Thread via GitHub
ashvina commented on code in PR #480: URL: https://github.com/apache/incubator-xtable/pull/480#discussion_r1702109326 ## xtable-api/src/main/java/org/apache/xtable/conversion/TableSyncConfig.java: ## @@ -0,0 +1,58 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [297] Refactor config classes [incubator-xtable]

2024-08-02 Thread via GitHub
ashvina commented on code in PR #480: URL: https://github.com/apache/incubator-xtable/pull/480#discussion_r1702107246 ## xtable-api/src/main/java/org/apache/xtable/conversion/ExternalTable.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] [297] Refactor config classes [incubator-xtable]

2024-08-02 Thread via GitHub
ashvina commented on code in PR #480: URL: https://github.com/apache/incubator-xtable/pull/480#discussion_r1702069326 ## xtable-api/src/main/java/org/apache/xtable/conversion/ExternalTable.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o