Skip to content

Conversation

@zhoulii
Copy link
Member

@zhoulii zhoulii commented Jul 12, 2022

What is the purpose of the pull request

fix link rendering error in Docker Demo and some other typos.

Verify this pull request

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

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.

@zhoulii
Copy link
Member Author

zhoulii commented Jul 13, 2022

hi @danny0405 , can you help to review this, thanks.

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.

@zhoulii Thanks for your first contribution!


> #### hoodie.keep.min.commits
> Similar to hoodie.keep.max.commits, but controls the minimum number ofinstants to retain in the active timeline.<br></br>
> Similar to hoodie.keep.max.commits, but controls the minimum number of instants to retain in the active timeline.<br></br>
Copy link
Contributor

Choose a reason for hiding this comment

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

Please refrain from changing the configurations.md directly. This is automatically generated and updated based on the Hudi config classes.

Copy link
Contributor

Choose a reason for hiding this comment

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

I addressed it in #6542.

Copy link
Member Author

Choose a reason for hiding this comment

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

@yihua Thanks for reviewing.

@yihua yihua force-pushed the fix-link-rendering branch from 2b062eb to 90ac46d Compare August 30, 2022 03:25
@yihua yihua self-assigned this Aug 30, 2022
@zhoulii
Copy link
Member Author

zhoulii commented Aug 30, 2022

Hi @yihua, I add a new commit to solve the conflict (seems not work), and there are three commits for this PR,Do I need to squash these commits?

@zhoulii zhoulii force-pushed the fix-link-rendering branch from a10ced8 to 5c2b583 Compare August 30, 2022 03:57
@yihua
Copy link
Contributor

yihua commented Aug 30, 2022

Hi @yihua, I add a new commit to solve the conflict (seems not work), and there are three commits for this PR,Do I need to squash these commits?

Don't worry about it. I'll do the squash and merge so you don't have to squash the commits.

@yihua yihua merged commit b42a13a into apache:asf-site Aug 30, 2022
@zhoulii zhoulii deleted the fix-link-rendering branch August 30, 2022 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants