sguggilam commented on a change in pull request #2627:
URL: https://github.com/apache/hbase/pull/2627#discussion_r523198964
##########
File path:
hbase-common/src/main/java/org/apache/hadoop/hbase/util/UnsafeAvailChecker.java
##########
@@ -39,6 +42,10 @@
@Override
public Boolean run() {
try {
+ final Configuration conf = HBaseConfiguration.create();
Review comment:
@Apache9 That's a good point .How about having both with system
property (if defined ) taking precedence over the configuration property. That
allows clients to define a config variable in places where they can pass in
with the config file in the class path ?
----------------------------------------------------------------
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]
