huaxiangsun commented on code in PR #4493:
URL: https://github.com/apache/hbase/pull/4493#discussion_r891797524
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/util/FSTableDescriptors.java:
##########
@@ -117,21 +118,22 @@ public FSTableDescriptors(final FileSystem fs, final Path
rootdir) {
public FSTableDescriptors(final FileSystem fs, final Path rootdir, final
boolean fsreadonly,
final boolean usecache) {
- this(fs, rootdir, fsreadonly, usecache, false, 0);
+ this(fs, rootdir, fsreadonly, usecache, 0);
}
public FSTableDescriptors(final FileSystem fs, final Path rootdir, final
boolean fsreadonly,
- final boolean usecache, final boolean tableDescriptorParallelLoadEnable,
- final int tableDescriptorParallelLoadThreads) {
+ final boolean usecache, final int tableDescriptorParallelLoadThreads) {
this.fs = fs;
this.rootdir = rootdir;
this.fsreadonly = fsreadonly;
this.usecache = usecache;
- this.tableDescriptorParallelLoadEnable = tableDescriptorParallelLoadEnable;
- if (tableDescriptorParallelLoadEnable) {
- executor =
Executors.newFixedThreadPool(tableDescriptorParallelLoadThreads,
+ if (tableDescriptorParallelLoadThreads > 0) {
+ tableDescriptorParallelLoadEnable = true;
Review Comment:
tableDescriptorParallelLoadEnable is not need anymore?
--
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]
