Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[flink] add IT to flink when use RESTCatalog #4805

Merged
merged 29 commits into from
Jan 9, 2025
Merged

[flink] add IT to flink when use RESTCatalog #4805

merged 29 commits into from
Jan 9, 2025

Conversation

jerry-024
Copy link
Contributor

@jerry-024 jerry-024 commented Dec 31, 2024 •
edited
Loading

Purpose

Linked issue: #4540

  • update RESTCatalogTest to extend CatalogTestBase
  • add RESTCatalogITCase to support IT case

Tests

API and Format

Documentation

@jerry-024 jerry-024 marked this pull request as draft December 31, 2024 02:23
jerry-024 added 25 commits January 6, 2025 10:46
@jerry-024
add IT
90c9144
@jerry-024
update
f9d9803
@jerry-024
support use MockRESTCatalogServer mock RESTCatalog server
85d3f4a
@jerry-024
add get database IT
d00623a
@jerry-024
add IT for auth fail
f5037c4
@jerry-024
add create table in streaming mode IT
da4d47a
@jerry-024
move MockRESTCatalogServer to rest package and add RESTCatalogMockSer…
7675c93 
…verTest
@jerry-024
add IT RESTCatalogITCase
bbe60b3
@jerry-024
delete no need test
f3c1403
@jerry-024
fix fail ut in RESTCatalogMockServerTest
aafa477
@jerry-024
fix alter table case fail in RESTCatalogMockServerTest
d6039fb
@jerry-024
fix renameTable fail
685224e
@jerry-024
fix all fail case in RESTCatalogMockServerTest
10159cf
@jerry-024
Merge branch 'master' into rest-catalog
c56388c
@jerry-024
delete no need update
1df6389
@jerry-024
add partition test and auth token test
8e08f04
@jerry-024
add alter table IT case for RESTCatalog
9317d61
@jerry-024
format
df7ef44
@jerry-024
update rest uri prefix to paimon
554eb94
@jerry-024
fix check style fail
1fc3b37
@jerry-024
update alter database test and add alter column type in RESTCatalogIT…
feef4c2 
…Case
@jerry-024
Merge branch 'master' into rest-catalog
c2e604d
@jerry-024
delete mock in HttpClientTest
3b0b9ea
@jerry-024
fix compile error
dcd9d10
@jerry-024
fix list tables in RESTCatalog
94ef732
@jerry-024 jerry-024 marked this pull request as ready for review January 9, 2025 07:34
@jerry-024
add test for get table from system database
609ee96 
format

fix check style fail

format
JingsongLi
JingsongLi approved these changes Jan 9, 2025
View reviewed changes
Copy link
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@JingsongLi JingsongLi merged commit fa52896 into apache:master Jan 9, 2025
11 of 12 checks passed
@jerry-024 jerry-024 deleted the rest-catalog branch January 10, 2025 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JingsongLi JingsongLi JingsongLi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jerry-024 @JingsongLi