Skip to content

Clarify file-lock being a distributed name lock#184

Closed
kwin wants to merge 1 commit intoapache:masterfrom
kwin:patch-2
Closed

Clarify file-lock being a distributed name lock#184
kwin wants to merge 1 commit intoapache:masterfrom
kwin:patch-2

Conversation

@kwin
Copy link
Copy Markdown
Member

@kwin kwin commented Jun 22, 2022

No description provided.

@cstamas
Copy link
Copy Markdown
Member

cstamas commented Jun 22, 2022

Well, it is not "distributed" as is not for multi hosts case. I'd rather distinguish like ST, MT, MP/SH and MH (single thread, multi thread, multi process/single-host and multi-host).

File is MP, redisson is MH in this manner.

@kwin
Copy link
Copy Markdown
Member Author

kwin commented Jun 22, 2022

Why is it not MH if you have e.g. a NFS network share mounted?

@cstamas
Copy link
Copy Markdown
Member

cstamas commented Jun 22, 2022

Unsure about NFS file locking... @michael-o ?

@michael-o
Copy link
Copy Markdown
Member

Looking at https://www.freebsd.org/cgi/man.cgi?query=nfsv4&sektion=4&manpath=FreeBSD+9.0-RELEASE NFSv4 has advisory locks. Forget 3 or older. Now there many impls of it and I don't know its quality. I don't have a v4 server at hand. Expect it to work, but your kilometrage may vary. So I am not rejecting it on v4.

@michael-o
Copy link
Copy Markdown
Member

Ceph and friends, no idea.

@cstamas
Copy link
Copy Markdown
Member

cstamas commented Jun 22, 2022

That was my thought as well: only NFSv4 w/ all the services needed (so not "bare" nfs server, but all rpc and whistles and bells needed)

@michael-o
Copy link
Copy Markdown
Member

That was my thought as well: only NFSv4 w/ all the services needed (so not "bare" nfs server, but all rpc and whistles and bells needed)

Correct, no blanket v4.

@cstamas
Copy link
Copy Markdown
Member

cstamas commented Jun 22, 2022

In short: if your FS correctly support locking, should work. Local FS usually is okay. For NFS your mileage may vary.

@michael-o
Copy link
Copy Markdown
Member

This should also go to the docs.

@cstamas
Copy link
Copy Markdown
Member

cstamas commented Jun 22, 2022

Superseded by #185

@cstamas cstamas closed this Jun 22, 2022
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.

3 participants