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

[Core]Add filesystem catalog lock #2867

Closed
wants to merge 4 commits into from
Closed

Conversation

TaoZex
Copy link
Contributor

@TaoZex TaoZex commented Feb 10, 2024

Purpose

When a process in file system attempts to acquire a lock, it tries to create a specific file in the file system. The existence of this file represents the state of the lock. If the file does not exist, it indicates that the lock is available, and the process can successfully acquire the lock. If the file already exists, it indicates that the lock has been acquired by another process.

subtask of #2824

Tests

API and Format

Documentation

@FangYongs
Copy link
Contributor

@TaoZex In fact, what we want is not a filesystem lock, but to optimize the catalog lock interface and support configuring catalog lock in the filesystem catalog. This will be quite different from the implementation of this PR.

@TaoZex TaoZex closed this Feb 18, 2024
@TaoZex
Copy link
Contributor Author

TaoZex commented Feb 18, 2024

@TaoZex In fact, what we want is not a filesystem lock, but to optimize the catalog lock interface and support configuring catalog lock in the filesystem catalog. This will be quite different from the implementation of this PR.

Thanks for your reply, I will close this pr.

@FangYongs
Copy link
Contributor

@TaoZex You can help to review #2872 if you are interested in this feature

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.

2 participants