mikewalch commented on a change in pull request #549: ACCUMULO-4074 Added
support for multiple scan executors
URL: https://github.com/apache/accumulo/pull/549#discussion_r200682697
##########
File path: core/src/main/java/org/apache/accumulo/core/conf/Property.java
##########
@@ -430,12 +433,33 @@
"When a tablet server's SimpleTimer thread triggers to check idle"
+ " sessions, this configurable option will be used to evaluate
update"
+ " sessions to determine if they can be closed due to inactivity"),
+ @Deprecated
TSERV_READ_AHEAD_MAXCONCURRENT("tserver.readahead.concurrent.max", "16",
PropertyType.COUNT,
- "The maximum number of concurrent read ahead that will execute. This
effectively"
- + " limits the number of long running scans that can run
concurrently per tserver."),
+ "This property is deprecated, use tserver.scan.executors.default.threads
instead."),
+ @Deprecated
TSERV_METADATA_READ_AHEAD_MAXCONCURRENT("tserver.metadata.readahead.concurrent.max",
"8",
PropertyType.COUNT,
- "The maximum number of concurrent metadata read ahead that will
execute."),
+ "This property is deprecated, use tserver.scan.executors.meta.threads
instead."),
+ TSERV_SCAN_EXECUTORS_PREFIX("tserver.scan.executors.", null,
PropertyType.PREFIX,
+ "Prefix for defining executors to service scans. For each executor the
number of threads, "
+ + "thread priority, and an optional prioritizer can be configured.
The prioritizer "
+ + "determines which scan an executor should run first and must
implement "
+ + ScanPrioritizer.class.getName() + ". Tables can select which
executor to use by setting"
+ + " table.scan.dispatcher. To configure a new executor, set "
+ + "tserver.scan.executors.<name>.threads=<number>. Optionally, can
also set "
+ + "tserver.scan.executors.<name>.priority=<number 1 to 10>, "
+ + "tserver.scan.executors.<name>.prioritizer=<class name>, and "
+ + "tserver.scan.executors.<name>.prioritizer.opts.<key>=<value>"),
+
TSERV_SCAN_EXECUTORS_DEFAULT_THREADS("tserver.scan.executors.default.threads",
"16",
+ PropertyType.COUNT,
+ "The number of threads for the scan executor that tables use by
default."),
+
TSERV_SCAN_EXECUTORS_DEFAULT_PRIORITIZER("tserver.scan.executors.default.prioritizer",
"",
+ PropertyType.STRING,
+ "Prioritizer for the default scan exector. Defaults to none which "
Review comment:
typo. should be `executor`
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
