[Improvement] Two classes of the same name

[justinmclean]

What would you like to be improved?

We have two classes with the same name - this could be confusing:
./clients/client-java/src/main/java/com/datastrato/gravitino/client/FilesetCatalog.java
./api/src/main/java/com/datastrato/gravitino/file/FilesetCatalog.java

How should we improve?

Rename one or more of the classes.

[jerryshao]

I think it is on purpose, we don't have a better name to separate the two files.

[justinmclean]

Give the description of one is "The FilesetCatalog interface defines the public API for managing fileset objects in a schema." could it be renamed FilesetSchema?

[jerryshao]

I think it's not proper to rename to "FilesetSchema", Fileset and Schema are two concepts in Gravitino. Previously we chose "GravitinoFilesetCatalog", it's also a bit weird. I think we still have time to figure out a good name before release.

CC @coolderli to take a look at it.

[coolderli]

@jerryshao Here are some new names I can think: GenericFilesetCatalog, CommonFilesetCatalog, FileCatalog, ClientFilesetCatalog, SimpleFilesetCatalogand DefaultFilesetCatalog. Which do you prefer? Or any other suggestion?

[coolderli]

@jerryshao Here are some new names I can think: GenericFilesetCatalog, CommonFilesetCatalog, FileCatalog, ClientFilesetCatalog, SimpleFilesetCatalogand DefaultFilesetCatalog. Which do you prefer? Or any other suggestion?

@jerryshao How about GenericFilesetCatalog?

[qqqttt123]

Actually I don't think this is an issue which need to solve.

[Jash2606]

Hey @jerryshao , I would like to work on this issue. please could you assign me this issue?