Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

fix: remove unneeded llmq-qvvec-sync setting #397

Closed
wants to merge 2 commits into from
Closed

fix: remove unneeded llmq-qvvec-sync setting #397

wants to merge 2 commits into from

Conversation

strophy
Copy link
Contributor

@strophy strophy commented Jul 6, 2021

Issue being fixed or feature implemented

llmq-qvvec-sync var is no longer required

What was done?

Remove llmq-qvvec-sync logic

How Has This Been Tested?

Untested, how do I test this?

Breaking Changes

Unknown

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@strophy strophy added this to the v0.20 milestone Jul 6, 2021
@strophy strophy changed the title @strophy fix: remove unneeded llmq-qvvec-sync setting fix: remove unneeded llmq-qvvec-sync setting Jul 6, 2021
@strophy strophy requested a review from shumkov July 6, 2021 09:43
@shumkov
Copy link
Member

shumkov commented Jul 6, 2021

It's done already in #395

@strophy
Copy link
Contributor Author

strophy commented Jul 6, 2021

Closed in favor of #395

@strophy strophy closed this Jul 6, 2021
@strophy strophy deleted the remove-llmq-qvvec-sync branch July 6, 2021 12:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@shumkov shumkov Awaiting requested review from shumkov

Assignees
No one assigned
Labels
None yet
Milestone
v0.20
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@strophy @shumkov