Skip to content

Allow _local doc writes to the replicator dbs#4416

Merged
nickva merged 1 commit intomainfrom
allow-local-docs-in-replicator-dbs
Feb 7, 2023
Merged

Allow _local doc writes to the replicator dbs#4416
nickva merged 1 commit intomainfrom
allow-local-docs-in-replicator-dbs

Conversation

@nickva
Copy link
Contributor

@nickva nickva commented Feb 6, 2023

During the VDU -> BDU update we inadvertently blocked _local doc writes to _replicator dbs. This commit rectifies that.

Add tests for _local, _design, and regular malformed docs for both the default _replicator db as well for for the prefixed version like $db/_replicator. For completeness, update the _scheduler/docs counts test also test both cases.

@nickva nickva requested a review from jaydoane February 6, 2023 20:33
During the VDU -> BDU update we inadvertently blocked _local doc writes to
_replicator dbs. This commit rectifies that.

Add tests for _local, _design, and regular malformed docs for both the default
_replicator db as well for for the prefixed version like $db/_replicator. For
completeness, update the _scheduler/docs counts test also test both cases.
@nickva nickva force-pushed the allow-local-docs-in-replicator-dbs branch from 9a3c0ac to 2d7038e Compare February 6, 2023 20:43
Copy link
Contributor

@jaydoane jaydoane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New tests look good 👍

module 'couch_replicator_scheduler_docs_tests'
  couch_replicator_scheduler_docs_tests:53: -scheduler_docs_test_main_db_test_/0-fun-0- (t_scheduler_docs_total_rows)...[5.227 s] ok
  couch_replicator_scheduler_docs_tests:63: -scheduler_docs_test_prefixed_db_test_/0-fun-0- (t_scheduler_docs_total_rows)...[5.083 s] ok
  couch_replicator_scheduler_docs_tests:97: -with/1-fun-0- (t_local_docs_can_be_written)...[0.016 s] ok
  couch_replicator_scheduler_docs_tests:97: -with/1-fun-0- (t_design_docs_can_be_written)...[0.014 s] ok
  couch_replicator_scheduler_docs_tests:97: -with/1-fun-0- (t_malformed_docs_are_rejected)...[0.010 s] ok
  couch_replicator_scheduler_docs_tests:97: -with/1-fun-0- (t_local_docs_can_be_written)...[0.029 s] ok
  couch_replicator_scheduler_docs_tests:97: -with/1-fun-0- (t_design_docs_can_be_written)...[0.125 s] ok
  couch_replicator_scheduler_docs_tests:97: -with/1-fun-0- (t_malformed_docs_are_rejected)...[0.012 s] ok
  [done in 11.746 s]

@nickva nickva merged commit 0fa5955 into main Feb 7, 2023
@nickva nickva deleted the allow-local-docs-in-replicator-dbs branch February 7, 2023 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants