DM-42005: HTTP-based Worker Replication server#883
Open
iagaponenko wants to merge 14 commits intomainfrom
Open
DM-42005: HTTP-based Worker Replication server#883iagaponenko wants to merge 14 commits intomainfrom
iagaponenko wants to merge 14 commits intomainfrom
Conversation
91b3c7e to
b5b2f7f
Compare
e90255c to
ea7962d
Compare
013e4f5 to
021a07a
Compare
c8f58ae to
9973d08
Compare
cae38b9 to
da02c98
Compare
bf0a008 to
24c5495
Compare
154426a to
91c03f7
Compare
b581b6a to
124acba
Compare
5485b61 to
1af3b73
Compare
6833b15 to
2575b5b
Compare
1aab3fd to
ae1a11c
Compare
5548365 to
dc93085
Compare
…uests This function that was provisioned in the original design of the Replication System has shown no benefits. Besides, the "duplicate" requests can't be reliably detected since the detection algorithm is time dependent.
…ation classes The class is now obsolete.
…service Extended Replication Worker's config page on the Web Dashbnoard to display connection parameters of the HTTP-based worker management services.
The eliminated code appears to be some left over experiment that was never finished neither it matured into any reasonable idea. Another problem with the removed code was that it was making the package qana depending on the Qserv frontend. This was not right from the dependency management standpoint.
Added REDESIGN comments in the relevant code to indicate the code that has to be changed or replaced. Added NEW comments for the newely added or refactored code. Using pre-processor macros to switch in between those as needed.
This is meant to pevent bottlecks during query analysis stage.
This experiment is meant to see if there would be any improvements in the query performance.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.