search

apache / gravitino

World's most powerful open data catalog for building a high-performance, geo-distributed and federated metadata lake.
https://gravitino.apache.org
Apache License 2.0
997 stars 311 forks source link

[Improvement] re-organize the code architecture of lakehouse Iceberg #1725

Closed FANNG1 closed 4 months ago

FANNG1 commented 8 months ago

What would you like to be improved?

the code of IcebergRESTcatalog and IcebergCatalog are mixed, we'd better re-organize the code architecture to make it cleaner.

How should we improve?

No response

FANNG1 commented 8 months ago

I want to put 3 packages under com.datastrato.gravitino.catalog.lakehouse.iceberg:

  1. catalog contains Gravitino Iceberg Catalog specified code
  2. rest.service contains Iceberg REST catalog service specified code
  3. common, the code reused by both.

@jerryshao @Clearvive any advice? especially for the package name.

jerryshao commented 8 months ago

I'm going to remove this out of 0.4.0, we can fix it in the next version.