ifesdjeen commented on code in PR #4210:
URL: https://github.com/apache/cassandra/pull/4210#discussion_r2164583358
##########
test/unit/org/apache/cassandra/utils/RangeTreeTest.java:
##########
@@ -363,39 +376,89 @@ private static Model create(ModelType modelType)
{
switch (modelType)
{
- case List:
case SearchableRangeList:
return new SearchableRangeListModel();
case RTree:
- return new RangeTreeModel(new RTree<>(COMPARATOR,
ALL_INCLUSIVE));
+ return new RangeTreeModel();
case IntervalTree:
return new IntervalTreeModel();
case IntervalBTree:
- return new RangeTreeModel(new RTree<>(COMPARATOR,
ALL_INCLUSIVE));
+ return new IntervalBTreeModel();
default:
throw new AssertionError("Unknown type: " + modelType);
}
}
- private static Model createModel(ModelType modelType)
+ private static Model createOracleForValidating(ModelType modelType)
{
- switch (modelType)
+// if (modelType == ModelType.IntervalTree)
Review Comment:
Nit: probably worth to clean this up
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
