JeongDaeKim commented on a change in pull request #751:
URL: https://github.com/apache/hbase/pull/751#discussion_r445271153
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
##########
@@ -912,6 +912,7 @@ long initialize(final CancelableProgressable reporter)
throws IOException {
}
MonitoredTask status = TaskMonitor.get().createStatus("Initializing region
" + this);
+ status.enableStatusJournal(true);
Review comment:
This will produce a journal entry with `null` status like
```
regionserver.HRegion: Region open journal:
null at 1592977559739
Running coprocessor pre-open hook at 1592977559739
Writing region info on filesystem at 1592977559739
...
```
I'm not sure this Is your intention. it looks weird for me. seems better to
have meaningful status string.
----------------------------------------------------------------
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]
