[jira] [Commented] (HBASE-15986) BackupServerUtil.getWALFilesOlderThan() should handle oldWALs properly

2016-06-08 Thread Vladimir Rodionov (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15321662#comment-15321662 ] Vladimir Rodionov commented on HBASE-15986: --- You are always welcome, Ted :) >

[jira] [Commented] (HBASE-15986) BackupServerUtil.getWALFilesOlderThan() should handle oldWALs properly

2016-06-08 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15321254#comment-15321254 ] Ted Yu commented on HBASE-15986: lgtm Can you upload a new patch where proper space is u

[jira] [Commented] (HBASE-15986) BackupServerUtil.getWALFilesOlderThan() should handle oldWALs properly

2016-06-08 Thread Vladimir Rodionov (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15321241#comment-15321241 ] Vladimir Rodionov commented on HBASE-15986: --- [~tedyu], are you OK with the patc

[jira] [Commented] (HBASE-15986) BackupServerUtil.getWALFilesOlderThan() should handle oldWALs properly

2016-06-07 Thread Vladimir Rodionov (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15319770#comment-15319770 ] Vladimir Rodionov commented on HBASE-15986: --- No, couple more places: parseHostN

[jira] [Commented] (HBASE-15986) BackupServerUtil.getWALFilesOlderThan() should handle oldWALs properly

2016-06-07 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15319686#comment-15319686 ] Ted Yu commented on HBASE-15986: The catch is only needed in parseHostFromOldLog(), right