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

Feat/redis interface findex v7 #74

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

HatemMn
Copy link

@HatemMn HatemMn commented Nov 26, 2024

In replacement of : #73

Dependent of findex PR : Cosmian/findex#92

What's implemented :

  • Instanciate a v7 findex redis interface
  • unit tests
  • save LUA script in redis cache to avoid sending it over network
  • Mutex on connection -> is that necessary ? TB discussed
  • connection manager instead of "connection" So far and for a reason that I don't get my hands on, using a connection manager causes infnite waiting time when performing an operation. To be checked out ( here is a branch with one commit that uses connection manager : ca3ab1c)

@HatemMn HatemMn self-assigned this Nov 26, 2024
@HatemMn HatemMn changed the base branch from main to develop November 26, 2024 16:23
@HatemMn HatemMn force-pushed the feat/redis_interface_findex_v7 branch from 5b61a34 to 8b7777b Compare November 26, 2024 16:25
@HatemMn HatemMn requested review from Manuthor and tbrezot November 26, 2024 16:49
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