gravitino icon indicating copy to clipboard operation
gravitino copied to clipboard

[FEATURE] Support `testConnection` Without Catalog Name in Gravitino

Open dataageek opened this issue 11 months ago • 2 comments

Describe the feature

Description:

The testConnection method currently requires a catalog name, as it depends on Metalake, which enforces this. However, there are cases where we need to validate the connection using only provider and properties before creating a catalog. This limitation makes it difficult to test connections early in the workflow.

Motivation

  • Enables connection testing before catalog creation.
  • Simplifies workflows and enhances flexibility.

Describe the solution

Allow testConnection to work with just provider and properties, making the catalog name optional.

Additional context

No response

dataageek avatar Jan 15 '25 12:01 dataageek

@mchades can you please take a look at this requirement?

jerryshao avatar Feb 06 '25 04:02 jerryshao

In actuality, the original intention behind the design of the testConnection interface was to validate the usability of the parameters in the createCatalog interface, plz see #4107.

Furthermore, to my knowledge, some users have already employed it as a preliminary validation interface for creating catalogs in the UI. If we were to make the catalog name optional, it would compromise the compatibility of this interface. Therefore, I do not recommend making such a modification.

I believe that utilizing a randomly generated catalog name when invoking the testConnection interface can still achieve the desired outcome you described, and there is no necessity for alterations to the interface.

mchades avatar Feb 06 '25 05:02 mchades