[jira] [Comment Edited] (HBASE-23115) Unit change for StoreFileSize and MemStoreSize

2019-10-10 Thread Karthik Palanisamy (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-23115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16948885#comment-16948885
 ] 

Karthik Palanisamy edited comment on HBASE-23115 at 10/10/19 7:08 PM:
--

[~brfrn169] The change was conflict with HBASE-22543. Fixed it now. 

Please review the new PR #710 for branch-2.2. Thanks again [~brfrn169]


was (Author: kpalanisamy):
[~brfrn169] The change was conflict with HBASE-22543. Fixed it now. 

Please review the new PR #710. Thanks again [~brfrn169]

> Unit change for StoreFileSize and MemStoreSize
> --
>
> Key: HBASE-23115
> URL: https://issues.apache.org/jira/browse/HBASE-23115
> Project: HBase
>  Issue Type: Bug
>  Components: metrics, UI
>Affects Versions: 3.0.0
>Reporter: Karthik Palanisamy
>Assignee: Karthik Palanisamy
>Priority: Minor
> Fix For: 3.0.0, 2.3.0
>
> Attachments: Units.pdf
>
>
> StoreFileSize and MemstoreSize usually returned in MBs (link1, link2)  but 
> few jsp pages contain inaccurate unit. The reason is table.jsp (link3) use 
> org.apache.hadoop.util.StringUtils.byteDesc(long len), this will perform 
> longtostring conversion and returns its unit(B, KB, MB, GB, TB, PB) based on 
> length. The concern here (link4) is computation (ByteVal/1024/1024) will 
> output always lesser than 1 for store contains few bytes or few kbs.  Also, 
> typecast will not round up to its nearest value.
> I think the best option is changing unit in table.jsp instead of changing 
> code, otherwise we may end up doing many refactors from getMemStoreSizeMB, 
> setMemStoreSizeMB, hasMemStoreSizeMB, getStorefileSizeMB, 
> setStorefileSizeMB,..
>  
> Please find the attachment, a simple example is posted.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Comment Edited] (HBASE-23115) Unit change for StoreFileSize and MemStoreSize

2019-10-09 Thread Toshihiro Suzuki (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-23115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16947765#comment-16947765
 ] 

Toshihiro Suzuki edited comment on HBASE-23115 at 10/9/19 2:46 PM:
---

Pushed into master and branch-2. But I faced conflicts when committing the 
patch to branch-2.1 and branch-2.2. Can you please make patches for the 
branches? [~kpalanisamy] One patch is okay if the patch can be applied for both 
the branches.


was (Author: brfrn169):
Pushed into master, branch-2. But I faced conflicts when committing the patch 
to branch-2.1 and branch-2.2. Can you please make patches for the branches? 
[~kpalanisamy] One patch is okay if the patch can be applied for both the 
branches.

> Unit change for StoreFileSize and MemStoreSize
> --
>
> Key: HBASE-23115
> URL: https://issues.apache.org/jira/browse/HBASE-23115
> Project: HBase
>  Issue Type: Bug
>  Components: metrics, UI
>Affects Versions: 3.0.0
>Reporter: Karthik Palanisamy
>Assignee: Karthik Palanisamy
>Priority: Minor
> Fix For: 3.0.0, 2.3.0
>
> Attachments: Units.pdf
>
>
> StoreFileSize and MemstoreSize usually returned in MBs (link1, link2)  but 
> few jsp pages contain inaccurate unit. The reason is table.jsp (link3) use 
> org.apache.hadoop.util.StringUtils.byteDesc(long len), this will perform 
> longtostring conversion and returns its unit(B, KB, MB, GB, TB, PB) based on 
> length. The concern here (link4) is computation (ByteVal/1024/1024) will 
> output always lesser than 1 for store contains few bytes or few kbs.  Also, 
> typecast will not round up to its nearest value.
> I think the best option is changing unit in table.jsp instead of changing 
> code, otherwise we may end up doing many refactors from getMemStoreSizeMB, 
> setMemStoreSizeMB, hasMemStoreSizeMB, getStorefileSizeMB, 
> setStorefileSizeMB,..
>  
> Please find the attachment, a simple example is posted.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)