Add test for realtime queue data buildup on sink clusters #545

Merged
merged 2 commits into from Feb 28, 2014

Projects

None yet

3 participants

@kellymclaughlin

No description provided.

@kellymclaughlin

This test is associated with this PR.

@lordnull

The rt_cascading test is not named in the same manner as the rest of the repl tests, but it wasn't caught before it was merged in. The only other test that uses rt_ at the beginning of its name is one for riak_test. To avoid a similar fate of confusion for this test, I'd recommend renaming it to 'repl_rt_cascading_rtq'.

@kellymclaughlin

The rt_cascading test is not named in the same manner as the rest of the repl tests, but it wasn't caught before it was merged in. The only other test that uses rt_ at the beginning of its name is one for riak_test. To avoid a similar fate of confusion for this test, I'd recommend renaming it to 'repl_rt_cascading_rtq'.

Great point, will make that change.

@andrewjstone
  • code inspection
  • test fails without riak_repl#547
  • test passes with riak_repl#547
@kellymclaughlin kellymclaughlin merged commit bfb35d5 into master Feb 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment