Skip to content

Register tables from config#22

Merged
osipovartem merged 7 commits into
mainfrom
aosipov/iceberg_snowflake
Nov 19, 2025
Merged

Register tables from config#22
osipovartem merged 7 commits into
mainfrom
aosipov/iceberg_snowflake

Conversation

@osipovartem
Copy link
Copy Markdown
Contributor

No description provided.

@rampage644
Copy link
Copy Markdown
Contributor

It LGTM. Quick question if you tried to run any meaningful queries yet?

rampage644
rampage644 previously approved these changes Nov 18, 2025
@osipovartem
Copy link
Copy Markdown
Contributor Author

It LGTM. Quick question if you tried to run any meaningful queries yet?

all DDL tests are ok

DanCodedThis
DanCodedThis previously approved these changes Nov 18, 2025
@DanCodedThis
Copy link
Copy Markdown
Contributor

@osipovartem The merge is not into origin main:
image

Base automatically changed from aosipov/remove_iceberg_calls_from_create_table to main November 19, 2025 11:10
@rampage644 rampage644 dismissed stale reviews from DanCodedThis and themself November 19, 2025 11:10

The base branch was changed.

@osipovartem osipovartem merged commit 28ce068 into main Nov 19, 2025
3 checks passed
@osipovartem osipovartem deleted the aosipov/iceberg_snowflake branch November 19, 2025 14:30
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.

3 participants