[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15726559#comment-15726559
]
huaxiang sun commented on HBASE-17241:
--
Thanks Ted and Anoop for review!
> Avoid compacting already
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15722533#comment-15722533
]
huaxiang sun commented on HBASE-17241:
--
Hi [~anoop.hbase], per my understanding, this is the current
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15721564#comment-15721564
]
Anoop Sam John commented on HBASE-17241:
So some file paths are removed as per the new logic
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15721559#comment-15721559
]
Hudson commented on HBASE-17241:
SUCCESS: Integrated in Jenkins build HBase-Trunk_matrix #2075 (See
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15721158#comment-15721158
]
Ted Yu commented on HBASE-17241:
Test failure was not related to the patch.
> Avoid compacting already
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15720865#comment-15720865
]
Hadoop QA commented on HBASE-17241:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15720347#comment-15720347
]
Ted Yu commented on HBASE-17241:
+1
> Avoid compacting already compacted mob files with _del files
>
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15719235#comment-15719235
]
huaxiang sun commented on HBASE-17241:
--
Thanks Ted. Yeah, it should be getFileCount() == 1 to
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15718943#comment-15718943
]
Ted Yu commented on HBASE-17241:
{code}
203 if (entry.getValue().getFileCount() <= 1) {
204
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15718539#comment-15718539
]
Hadoop QA commented on HBASE-17241:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15716856#comment-15716856
]
huaxiang sun commented on HBASE-17241:
--
Thanks Ted. The failure is caused by the patch, need to
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15716847#comment-15716847
]
Ted Yu commented on HBASE-17241:
{code}
90 public int getFileNumbers () {
{code}
Rename the
[
https://issues.apache.org/jira/browse/HBASE-17241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15715810#comment-15715810
]
Hadoop QA commented on HBASE-17241:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
13 matches
Mail list logo