4paradigm / 4paradigm/OpenMLDB

apiserver needs full sql test, pay attention to NsClient::db_

Open
#2,212 0 comments 0 reactions 1 assignee Claimed by @vagetablechicken View on GitHub
api-server enhancement
Dominant language
C++
Stars
1.7k
Forks
331
Avg merge
12d 12h
Merged PRs (30d)
1

Description

Each apiserver request has its own db, but NsClient used to cache db. So we may get no db bug when we request apiserver and it calls NsClient.

Contributor guide

Open the contributing guide

Research direction

The issue mentions apiserver, NsClient::db_, and a potential 'no db bug'. Look for apiserver request handling code and the NsClient class to understand how database connections are managed and cached. Write a full SQL test that reproduces the scenario where a request triggers NsClient and encounters a missing database. Verify the fix ensures each request has its own db accessible.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, sql
Domain
backend-api-design, databases, testing-qa
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.