[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-23 Thread Hanisha Koneru (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17090785#comment-17090785 ] Hanisha Koneru commented on RATIS-850: -- Thank you [~ljain] and [~msingh] for the revi

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-23 Thread Lokesh Jain (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17090343#comment-17090343 ] Lokesh Jain commented on RATIS-850: --- [~hanishakoneru] Thanks for the contribution! [~msi

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-23 Thread Lokesh Jain (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17090324#comment-17090324 ] Lokesh Jain commented on RATIS-850: --- [~hanishakoneru] Thanks for updating the patch! The

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-22 Thread Hadoop QA (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17089915#comment-17089915 ] Hadoop QA commented on RATIS-850: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-22 Thread Hanisha Koneru (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17089870#comment-17089870 ] Hanisha Koneru commented on RATIS-850: -- Snapshots are a function of the StateMachine.

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-21 Thread Lokesh Jain (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17089316#comment-17089316 ] Lokesh Jain commented on RATIS-850: --- [~hanishakoneru] I agree. What I am wondering is if

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-21 Thread Hanisha Koneru (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17089114#comment-17089114 ] Hanisha Koneru commented on RATIS-850: -- On second thoughts, purging only upto snapsho

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-21 Thread Hanisha Koneru (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17088980#comment-17088980 ] Hanisha Koneru commented on RATIS-850: -- Thank you [~ljain] and [~msingh] for the revi

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-21 Thread Lokesh Jain (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17088494#comment-17088494 ] Lokesh Jain commented on RATIS-850: --- [~hanishakoneru] Thanks for working on this! The ch

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-21 Thread Mukul Kumar Singh (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17088433#comment-17088433 ] Mukul Kumar Singh commented on RATIS-850: - Thanks for updating the patch. +1, the

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-17 Thread Hanisha Koneru (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17085970#comment-17085970 ] Hanisha Koneru commented on RATIS-850: -- Unit tests pass locally. [~msingh] , [~loke

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-15 Thread Hadoop QA (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17084508#comment-17084508 ] Hadoop QA commented on RATIS-850: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-15 Thread Hanisha Koneru (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17084485#comment-17084485 ] Hanisha Koneru commented on RATIS-850: -- Fixed the checkstyle and one related failing

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-14 Thread Hadoop QA (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17083804#comment-17083804 ] Hadoop QA commented on RATIS-850: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (RATIS-850) Allow log purge up to snapshot index

2020-04-14 Thread Mukul Kumar Singh (Jira)
[ https://issues.apache.org/jira/browse/RATIS-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17083774#comment-17083774 ] Mukul Kumar Singh commented on RATIS-850: - [~hanishakoneru], This was done on Ozon