Skip to content

[enhancement](paimon)add obs filesystem for paimon #31061

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
Merged
yiguolei merged 2 commits into from
Feb 19, 2024
Merged

[enhancement](paimon)add obs filesystem for paimon #31061

yiguolei merged 2 commits into from
Feb 19, 2024

Conversation

@wuwenchi
Copy link
Contributor

@wuwenchi wuwenchi commented Feb 18, 2024

Proposed changes

add obs filesystem, like:

CREATE CATALOG `paimon_obs` PROPERTIES (
    "type" = "paimon",
    "warehouse" = "obs://<bucket>/paimon",
    "obs.endpoint"="obs.cn-north-4.myhuaweicloud.com",
    "obs.access_key"="ak",
    "obs.secret_key"="sk"
);

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@doris-robot
Copy link

doris-robot commented Feb 18, 2024

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

@wuwenchi
Copy link
Contributor Author

wuwenchi commented Feb 18, 2024

run buildall

morningman
morningman approved these changes Feb 18, 2024
View reviewed changes
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Feb 18, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Feb 18, 2024

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 18, 2024

PR approved by anyone and no changes requested.

wsjz
wsjz approved these changes Feb 18, 2024
View reviewed changes
Copy link
Contributor

@wsjz wsjz left a comment

Choose a reason for hiding this comment

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

LGTM

@yiguolei yiguolei merged commit a076d49 into apache:master Feb 19, 2024
@wuwenchi wuwenchi mentioned this pull request Feb 19, 2024
Merged
yiguolei pushed a commit that referenced this pull request Feb 19, 2024
@wuwenchi @Doris-Extras
[enhancement](paimon)add obs filesystem for paimon (#31061)
bda2042
xiaokang pushed a commit that referenced this pull request Feb 19, 2024
@wuwenchi
[enhancement](paimon)add obs filesystem for paimon for 2.0 #31061 (#3…
d90f7db 
…1089)
mongo360 pushed a commit to mongo360/doris that referenced this pull request Aug 16, 2024
@wuwenchi
[enhancement](paimon)add obs filesystem for paimon for 2.0 apache#31061…
4ba659b 
… (apache#31089)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morningman morningman morningman approved these changes

@yiguolei yiguolei yiguolei approved these changes

+1 more reviewer

@wsjz wsjz wsjz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by one committer. dev/2.0.5-merged reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@wuwenchi @doris-robot @morningman @yiguolei @wsjz @xiaokang