adelapena commented on a change in pull request #1130:
URL: https://github.com/apache/cassandra/pull/1130#discussion_r686736894
##########
File path: .circleci/config.yml
##########
@@ -706,8 +706,8 @@ jobs:
- DTEST_BRANCH: trunk
- CCM_MAX_HEAP_SIZE: 1024M
- CCM_HEAP_NEWSIZE: 256M
- - REPEATED_UTEST_TARGET: testsome
- - REPEATED_UTEST_CLASS: null
+ - REPEATED_UTEST_TARGET: test
+ - REPEATED_UTEST_CLASS: ViewTest
Review comment:
Nit: this runs both `org.apache.cassandra.cql3.ViewTest` and
`org.apache.cassandra.db.lifecycle.ViewTest`, when we only need the former. I
think this can be avoided by using `testsome` and the qualified class name:
```
REPEATED_UTEST_TARGET: testsome
REPEATED_UTEST_CLASS: org.apache.cassandra.cql3.ViewTest
```
There is nothing wrong with running more tests than the strictly needed, I'm
just telling in case we want to run this again, or in the other branches :)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
