feature: add a new api ngx.shared.DICT.cas#583
feature: add a new api ngx.shared.DICT.cas#583doujiang24 wants to merge 2 commits intoopenresty:masterfrom
Conversation
|
+1 |
agentzh
force-pushed
the
master
branch
2 times, most recently
from
e7ac10c to
cfd4f90
Compare
There was a problem hiding this comment.
Need to add the newly added contexts recently, like balancer_by_lua* and ssl_certificate_by_lua*.
|
Please rebase to the latest git master. Thank you :) |
doujiang24
force-pushed
the
shdict_cas
branch
from
d8529f7 to
e6a2d68
Compare
doujiang24
force-pushed
the
shdict_cas
branch
from
e6a2d68 to
f58feb2
Compare
doujiang24
force-pushed
the
shdict_cas
branch
from
f58feb2 to
632f95f
Compare
|
@agentzh Thanks, I have rebased to the master and fixed the confusing descriptions. |
|
+1 I'll be merging this into my fork (with incr ttl). It would be good if there was a FFI (resty core) PR for this too |
|
I was wondering, is there any reason why this wasn't merged yet? Having |
|
@dolik-rce Just the lack of time and higher priority items. I think @thibaultcha will take care of this :) |
|
This pull request is now in conflict :( |
zhuizhuhaomeng
force-pushed
the
master
branch
from
f924579 to
fef2581
Compare
|
This pull request is now in conflict :( |
|
This pull request is now in conflict :( |
|
This pull request is now in conflict :( |
|
Any progress on this? This is the last missing function to make the shared dict truly atomic |
|
This pull request is now in conflict :( |
|
This pull request is now in conflict :( |
|
This pull request is now in conflict :( |
6 participants
Just like the replace method, but only stores the key-value pair into the dictionary ngx.shared.DICT if the old_value argument and old_flags argument does match the value and flags in the dictionary ngx.shared.DICT (or one of them).