Skip to content

Remove iceberg call from create table statement#21

Merged
rampage644 merged 2 commits into
mainfrom
aosipov/remove_iceberg_calls_from_create_table
Nov 19, 2025
Merged

Remove iceberg call from create table statement#21
rampage644 merged 2 commits into
mainfrom
aosipov/remove_iceberg_calls_from_create_table

Conversation

@osipovartem
Copy link
Copy Markdown
Contributor

Closes #12

  • removed all iceberg catalog call from executore
  • removed IcebergCatalogResult struct
  • moved to native DF traits

@osipovartem osipovartem requested review from DanCodedThis, YaroslavLitvinov and rampage644 and removed request for rampage644 November 18, 2025 10:18
Copy link
Copy Markdown
Contributor

@DanCodedThis DanCodedThis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@osipovartem LGTM, with some questions

Comment thread crates/catalog/src/schema.rs
Comment thread crates/executor/src/query.rs
Comment thread crates/catalog/src/table.rs
@rampage644 rampage644 merged commit 600c44d into main Nov 19, 2025
5 checks passed
@rampage644 rampage644 deleted the aosipov/remove_iceberg_calls_from_create_table branch November 19, 2025 11:10
osipovartem added a commit that referenced this pull request Nov 21, 2025
Remove invalidate_cache

Init

Read tables from config

Read tables from config

Fix avro

Add lambda configuration and more logging/tracing (#23)

sessions: JWT (#19)

* jwt token sessions

* Requested changes

* clippy

* fmt

* clippy

Remove iceberg call from create table statement (#21)

* Remove iceberg call from create table

* Remove invalidate_cache

Snowdlake table wrapper

snowflake rest retries support (#24)

* replace oneshot by JoinHandle

* switch to different execution model with results waiting until task joined

* fix abort

* simplify executor interface

Register tables from config (#22)

* Remove iceberg call from create table

* Remove invalidate_cache

* Init

* Read tables from config

* Read tables from config

* Fix avro
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove iceberg catalog call from Executor

3 participants