Avoid restarting the query server while there are running queries

Không có tệp hoặc bài kiểm thử nào được nêu. Hãy bắt đầu bằng cách xác định client của query-server, các callback cho những thao tác đang chờ của nó và luồng cài đặt query-server; sau đó truy vết cách các thay đổi cài đặt kích hoạt việc khởi động lại. Công việc được xem là hoàn tất khi, sau một thay đổi cài đặt trong lúc có các thao tác đang chờ, một popup xác nhận không-modal được hiển thị, đồng thời vẫn cho phép các truy vấn mới và cung cấp một nút để khởi động lại cũng như hủy các truy vấn hiện tại.

Currently, if someone changes any query server settings, the query server is immediately stopped. Any running queries will complete with an error.

It should be possible to avoid restarting the server while there are running queries. The way that the query-server client is implemented, there are callbacks registered for any query or operation that is currently running. When completed, the callback is removed. Every time, after a callback is removed, we can check if the list of open callbacks is empty, and only then restart the server.

There are several potential edge cases to consider:

1. If there is a zombie callback that is registered even though the thing it is registered for is no longer calling back, it will never be removed and the query server will never restart. This shouldn't happen, but we have nothing in place to ensure this.
2. Consider the following pattern:
   1. User starts query1
   2. User changes a query server setting
   3. Query server restart is registered and is waiting for query1 to complete
   4. User starts query2. Query server restart is now waiting for both query1 and query2 to start
      It may be confusing for a user that their next query is using the old settings even though they requested a change.

For (1), perhaps we can set a timeout where if there are no callbacks triggered after X seconds, we assume they are all zombies and restart anyway. Long running queries will still invoke the callbacks on a regular basis, so this is safe.

For (2), I am less certain. Perhaps we can prevent starting new queries if a server restart is requested.

EDIT- After discussion with @adityasharad the simplest implementation that will suit are purposes is:

After editing any of the query server settings and there are server operations outstanding, open a popup saying something like: "Settings will not come into effect until the query server is restarted. Restart now? (This will cancel any currently executing queries.)" And add a button on the popup that triggers the restart.

The popup should be non-modal, allowing a user to potentially run new queries under the old settings. This is OK because it is clear what is happening.