Skip to content

Conversation

@lubo212
Copy link
Contributor

@lubo212 lubo212 commented Nov 17, 2021

What is the purpose of the pull request
Resolve integration issues with GooseFS and HUDI

Brief change log
Goosefs does not support appending and needs to be marked in StorageSchemes

Verify this pull request
(Please pick either of the following options)

This pull request is a trivial rework / code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.
Committer checklist
Has a corresponding JIRA in PR title & commit

Commit message is descriptive of the change

CI is green

Necessary doc changes done or have another open PR

For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@lubo212 lubo212 changed the title Asf site Add the GooseFS integration document Nov 17, 2021
Copy link
Contributor

@leesf leesf left a comment

Choose a reason for hiding this comment

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

you should not put the goosefs_hoodie.md in older version dir. you can put it under /website/docs/, just Eupdate cloud.md and add goosefs_hoodie.md file.

@lubo212
Copy link
Contributor Author

lubo212 commented Nov 19, 2021

你不应该把 goosefs_hoodie.md 放在旧版本的目录中。你可以把它放在/website/docs/,只是 Eupdatecloud.md并添加 goosefs_hoodie.md 文件。

Thank you, I have modified

@leesf leesf self-assigned this Nov 21, 2021
last_modified_at: 2021-11-17T12:50:50-10:00
language: cn
---
这个页面描述了如何让你的Hudi spark任务使用Tencent Cloud GooseFS。
Copy link
Contributor

Choose a reason for hiding this comment

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

would you use english please? and here not just spark job?

@leesf
Copy link
Contributor

leesf commented Dec 7, 2021

@lubo212 any updates here?

@nsivabalan
Copy link
Contributor

@lubo212 : can you address the feedback please.

@xushiyan xushiyan added the docs label Apr 20, 2022
@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Feb 26, 2024
@yihua yihua changed the title Add the GooseFS integration document [HUDI-8226][DOCS] Add the GooseFS integration document Sep 21, 2024
Copy link
Contributor

@yihua yihua left a comment

Choose a reason for hiding this comment

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

Closing this PR due to inactivity. @lubo212 feel free to reopen if needed. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S PR with lines of changes in (10, 100]

Projects

Status: 🚧 Needs Repro
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

5 participants