ndimiduk commented on a change in pull request #1408: HBASE-24086 Disable 
output stream capability enforcement when running in standalone mode
URL: https://github.com/apache/hbase/pull/1408#discussion_r402015257
 
 

 ##########
 File path: 
hbase-common/src/main/java/org/apache/hadoop/hbase/util/CommonFSUtils.java
 ##########
 @@ -350,11 +349,19 @@ public static void setWALRootDir(final Configuration c, 
final Path root) {
   public static FileSystem getWALFileSystem(final Configuration c) throws 
IOException {
     Path p = getWALRootDir(c);
     FileSystem fs = p.getFileSystem(c);
-    // hadoop-core does fs caching, so need to propogate this if set
+    // hadoop-core does fs caching, so need to propagate this if set
     String enforceStreamCapability = c.get(UNSAFE_STREAM_CAPABILITY_ENFORCE);
     if (enforceStreamCapability != null) {
       fs.getConf().set(UNSAFE_STREAM_CAPABILITY_ENFORCE, 
enforceStreamCapability);
     }
+    if (!c.getBoolean(HConstants.CLUSTER_DISTRIBUTED, false)
 
 Review comment:
   You tell me :)
   
   Yes, for running out of the source tree, not modifying any configuration, 
this change permits the process to start, master and region server threads all 
come online. I performed some additional verification by running `ltt` against 
the process, a scant 100k rows.
   
   I don't know well when we choose `LocalFileSystem` vs. `RawLocalFIleSystem`, 
so there may be other issues lurking.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to