ankitsinghal commented on a change in pull request #722: HBASE-23065 [hbtop]
Top-N heavy hitter user and client drill downs
URL: https://github.com/apache/hbase/pull/722#discussion_r338412238
##########
File path:
hbase-hbtop/src/main/java/org/apache/hadoop/hbase/hbtop/screen/top/TopScreenModel.java
##########
@@ -113,7 +116,7 @@ public void refreshMetricsData() {
}
private void refreshSummary(ClusterMetrics clusterMetrics) {
- String currentTime = DateFormatUtils.ISO_8601_EXTENDED_TIME_FORMAT
+ String currentTime = DateFormatUtils.ISO_DATE_FORMAT
Review comment:
Yes but don't know why ISO_8601 was throwing below error for me though there
are no conflicts of commons-lang3 .
```
Exception in thread "main" java.lang.NoSuchFieldError:
ISO_8601_EXTENDED_TIME_FORMAT
at
org.apache.hadoop.hbase.hbtop.screen.top.TopScreenModel.refreshSummary(TopScreenModel.java:119)
at
org.apache.hadoop.hbase.hbtop.screen.top.TopScreenModel.refreshMetricsData(TopScreenModel.java:114)
at
org.apache.hadoop.hbase.hbtop.screen.top.TopScreenPresenter.refresh(TopScreenPresenter.java:89)
at
org.apache.hadoop.hbase.hbtop.screen.top.TopScreenView.init(TopScreenView.java:65)
at org.apache.hadoop.hbase.hbtop.screen.Screen.run(Screen.java:77)
at org.apache.hadoop.hbase.hbtop.HBTop.run(HBTop.java:121)
at org.apache.hadoop.util.ToolRunner.run(ToolRunner.java:76)
```
Will debug into this more tomorrow.
----------------------------------------------------------------
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
