Manno15 commented on a change in pull request #149:
URL: https://github.com/apache/accumulo-testing/pull/149#discussion_r693142298
##########
File path: conf/accumulo-testing.properties
##########
@@ -39,12 +39,15 @@ test.ci.common.accumulo.num.tablets=20
# Optional authorizations that if specified will be randomly selected by
scanners and walkers
# Format: a,b|a,b,c|c
test.ci.common.auths=
+# Accumulo tserver properties to set when creating a table
+test.ci.common.accumulo.server.props=\
+tserver.compaction.major.service.cs1.planner=org.apache.accumulo.core.spi.compaction.DefaultCompactionPlanner
\
+tserver.compaction.major.service.cs1.planner.opts.executors=[{"name":"small","type":"internal","maxSize":"32M","numThreads":4},{"name":"medium","type":"internal","maxSize":"128M","numThreads":2},{"name":"large","type":"external","queue":"DCQ1"}]
Review comment:
After looking into this more, we should not include an external queue as
the default case unless we also include starting the other compaction services
(compactor and compaction coordinator) by default. I think the best case is to
have the configuration only be internal opts and then the user can decide on if
they want external compactions and then start the other services.
--
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]
