DonalEvans commented on a change in pull request #7521:
URL: https://github.com/apache/geode/pull/7521#discussion_r838937126
##########
File path: ci/scripts/execute_redis_tests.sh
##########
@@ -78,7 +78,10 @@ failCount=0
--single unit/type/zset \
--single unit/quit \
--single unit/pubsub \
---single unit/dump
+--single unit/dump \
+--single unit/type/list \
+#--single unit/type/list-2 \ TODO enable when GEODE-10116 merged
+#--single unit/type/list-3 TODO enable when GEODE-10160 is fixed
Review comment:
These comments are a little confusing, as this PR is for GEODE-10116, so
it's strange to have a comment saying effectively "enable once this PR is
merged", and GEODE-10160 is fixing bucket sizing, which seems like it shouldn't
have an impact on unit-level tests.
--
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: notifications-unsubscr...@geode.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
