[CASSANDRA-16515] Don't fail all ViewFilteringTest cases because one case timed out

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 4.0-rc1, 4.0
• Component/s: Test/unit
• Bug Category: Code
• Severity: Low
• Complexity: Low Hanging Fruit
• Discovered By: Unit Test
• Platform: All
• Since Version: 4.0-alpha1

Source Control Link

https://github.com/apache/cassandra/commit/7ae0fcecd207af14e35d171642b8d21feb6e543a

Test and Documentation Plan

run regression tests (just unit tests)

• Assignee: Jacek Lewandowski
• Reporter: Jacek Lewandowski

Authors

Jacek Lewandowski

Reviewers

Brandon Williams

• Created: 12/Mar/21 09:21
• Updated: 19/Nov/21 06:55
• Resolved: 12/Mar/21 20:00

Description

When ViewFilteringTest creates a view, it is added to the list of created views. Those views are removed after each test cases. If creating a view fail, it is not added to the list of views to be dropped because the assumption is that when the command fails, the view should not be created. This is ok in general however it is not a valid assumption when we get operation timed out because we cannot say anything about its real result. This way, it is possible that the view gets created, the driver throws time out and we do not remove the created view. All other test cases which tries to create that view will fail because the view already exists.

While it seems very improbable, we actually hit this a couple of times on our infrastructure

Issue Links

• links to: Github PR