Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] feat: basic implementation of dynamodb catalog #223

Closed
wants to merge 3 commits into from

Conversation

odysa
Copy link
Contributor

@odysa odysa commented Feb 27, 2024

todo

  • Implement list namespaces
  • Use environment variables for configs.
  • unit tests
  • integration tests

@Xuanwo
Copy link
Member

Xuanwo commented Feb 28, 2024

There's a discussion about Deprecate DynamodbCatalog on [email protected]. Maybe we should join in?

LINK: https://lists.apache.org/thread/b92zwcvy7917bhr5t00b07f7r8qqwxqj

@Xuanwo
Copy link
Member

Xuanwo commented Feb 28, 2024

Based on the discussion, it appears that few people utilize this catalog. Do you have a specific use case for it?

@odysa
Copy link
Contributor Author

odysa commented Feb 28, 2024

They suggest to use Glue. Let me figure out does it work for me. cc @Xuanwo

@Xuanwo
Copy link
Member

Xuanwo commented Feb 28, 2024

They suggest to use Glue. Let me figure out does it work for me. cc @Xuanwo

Thanks! Feel free to implement the DynamoDB catalog in iceberg-rust if you still want it.

I just wanted to update you on the current situation within the community: there is a disscusion to deprecate it. And it's highly possible for us to deprecate it too someday.

@odysa
Copy link
Contributor Author

odysa commented Feb 28, 2024

Update: We decide to use Postgres, cuz none of us know Glue😢. I will start working on the SQL Catalog which I think it may be more widely used.
I will close this PR and move this implementation to my personal repo.

@odysa odysa closed this Feb 28, 2024
@liurenjie1024
Copy link
Collaborator

Update: We decide to use Postgres, cuz none of us know Glue😢. I will start working on the SQL Catalog which I think it may be more widely used. I will close this PR and move this implementation to my personal repo.

I remember @JanKaul mentioned that he already has an implementation for this, maybe we can help to port it?

@JanKaul
Copy link
Collaborator

JanKaul commented Feb 29, 2024

Yes, sorry I didn't have time to implement it. I'll try to work on it today.

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.

4 participants