You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The IcebergREST has three impalement IT classes.
I think it better to add the registerIcebergCatalogConfig abstract interface in IcebergRESTServiceBaseIT.class.
How should we improve?
Use an abstract interface replace switch statement.
### What changes were proposed in this pull request?
1. make getCatalogConfig abstract, Hive&Jdbc&Memory implement these
interfaces.
2. remove `gravitino-docker-it` tag from IcebergRESTServiceIT.
### Why are the changes needed?
After #711, It takes too much time to test IcebergRESTServiceIT, there's
no need to start the docker container for the memory catalog
Fix: #874
### Does this PR introduce _any_ user-facing change?
no
### How was this patch tested?
existing UT
What would you like to be improved?
The IcebergREST has three impalement IT classes.
I think it better to add the registerIcebergCatalogConfig abstract interface in
IcebergRESTServiceBaseIT.class
.How should we improve?
Use an abstract interface replace switch statement.
com/datastrato/gravitino/integration/test/catalog/lakehouse/iceberg/IcebergRESTServiceBaseIT.java
The text was updated successfully, but these errors were encountered: