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.

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

[DOCS] New blog Concurrency Control #12724

Merged
merged 3 commits into from
Jan 30, 2025

Conversation

dipankarmazumdar
Copy link

Change Logs

Adds a new blog on Concurrency Control in Lakehouse

Impact

None, Site update

Risk level (write none, low medium or high below)

None

Documentation Update

None

Contributor's checklist

@bhasudha
Copy link
Contributor

bhasudha commented Jan 28, 2025

@dipankarmazumdar One minor comment. Can you move the images from [content/assets/images/blog/concurrency_control/..] to [website/static/assets/images/blog/concurrency_control/..] instead ? The images under content/ will automatically be populated after the PR is merged.

@bhasudha bhasudha self-requested a review January 28, 2025 19:56
@github-actions github-actions bot added docs size:M PR with lines of changes in (100, 300] labels Jan 28, 2025
Copy link
Contributor

@bhasudha bhasudha left a comment

Choose a reason for hiding this comment

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

LGTM except for a minor comment

@dipankarmazumdar
Copy link
Author

LGTM except for a minor comment

thanks. PTAL now!

Copy link
Contributor

@bhasudha bhasudha left a comment

Choose a reason for hiding this comment

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

LGTM

@bhasudha bhasudha merged commit c190a60 into apache:asf-site Jan 30, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs size:M PR with lines of changes in (100, 300]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants