[flink] FlinkCatalog#listTables returns both tables and views

[Sxnan]

Purpose

Linked issue: close #5781

According to Flink's Catalog#listTables method's Javadoc, the FlinkCatalog#listTables method should return all tables and views under a certain database.

Tests

API and Format

Documentation

[Sxnan]

Hi @yunfengzhou-hub, can you review this PR?

[yunfengzhou-hub]

Hi @Sxnan Thanks for the PR. It might be better to also check if other APIs have similar syntax gap and fix them altogether.

Besides, could you add some tests to verify the correctness of this change?

[Sxnan]

@yunfengzhou-hub I checked the other APIs and only listTable has the problem. I added the test to verify that the listTable returns views as well.

[yunfengzhou-hub]

@yunfengzhou-hub I checked the other APIs and only listTable has the problem. I added the test to verify that the listTable returns views as well.

It seems that FlinkCatalog#alterTable does not support alter Views yet. But different from listTables, this method could explicitly throw exception instead of returning a wrong response, so the support would be more adding a new feature than fixing a bug. Thus we can consider add such supports in future PRs.

[yunfengzhou-hub]

Thanks for the PR. LGTM.