FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(redis): do not kill the keep-alive Overview socket when opening a DB · Issue #812 · QueryaHub/Querya-Desktop · GitHub

fix(redis): do not kill the keep-alive Overview socket when opening a DB #812

Description

Summary

RedisService stores one RedisConnection per connection id. createConnection disconnects the previous socket (fire-and-forget, not awaited).

workspace_panel _homeObjectMorph keeps Overview RedisView mounted under RedisExplorerView. Both call createConnection:

  1. Open the connection → Overview owns the socket.
  2. Click db0 → Explorer createConnection kills Overview. Home still holds the dead object (_ownsConnection == true).
  3. Back to Home, tap Refresh → Overview createConnection kills Explorer while Explorer stays keep-alive. Next SCAN/GET throws.

The sidebar already documents this (Use a temporary connection so we don't kill the main view's connection) and uses id: -1 — the workspace does not.

Scope

  • Reuse getConnection when already connected, or pool by role (stats vs explorer).
  • Await disconnect only on a real reconnect / user Disconnect.
  • Home Refresh must not steal Explorer’s socket.

Out of scope

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingconnectionsDatabase connections, URI parsing, poolsstabilityTheme parser epic label: stability

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL