pengmq1 commented on a change in pull request #1730:
URL: https://github.com/apache/hbase/pull/1730#discussion_r443387870
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileWriter.java
##########
@@ -547,6 +553,20 @@ public StoreFileWriter build() throws IOException {
CommonFSUtils.setStoragePolicy(this.fs, dir, policyName);
if (filePath == null) {
+ // The stored file and related blocks will used the directory based
StoragePolicy.
+ // Because HDFS DistributedFileSystem does not support create files
with storage policy
+ // before version 3.3.0 (See HDFS-13209). Use child dir here is to
make stored files
+ // satisfy the specific storage policy when writing. So as to avoid
later data movement.
+ // We don't want to change whole temp dir to 'fileStoragePolicy'.
+ if (fileStoragePolicy != null && !fileStoragePolicy.isEmpty()) {
Review comment:
OK
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
