keith-turner commented on a change in pull request #163:
URL: https://github.com/apache/accumulo-testing/pull/163#discussion_r720304767
##########
File path:
src/main/java/org/apache/accumulo/testing/performance/tests/ScanExecutorPT.java
##########
@@ -89,8 +95,9 @@ public Report runTest(Environment env) throws Exception {
String tableName = "scept";
Map<String,String> props = new HashMap<>();
- props.put(Property.TABLE_SCAN_DISPATCHER_OPTS.getKey() + "executor",
"se1");
- props.put(Property.TABLE_SCAN_DISPATCHER_OPTS.getKey() + "heed_hints",
"true");
+ props.put(Property.TABLE_SCAN_DISPATCHER_OPTS.getKey() + "executor.se1p1",
"se1");
+ props.put(Property.TABLE_SCAN_DISPATCHER_OPTS.getKey() + "executor.se1p2",
"se1");
+ props.put(Property.TABLE_SCAN_DISPATCHER_OPTS.getKey() + "executor.se2p1",
"se2");
Review comment:
Yeah you do need two scan types. This per table dispatcher does not
care about prio, it just dispatches scans to the executor se1 or se2. The
executor se1 has further config that sets the prio based on scan type, which is
where the two diff scan types matter. So we have it dispatch everything to
executor se1 unless the scan type is dedicated in which case it dispatches to
se2.
--
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]
