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

Tsz Wo Nicholas Sze commented on RATIS-176:
-------------------------------------------

Thanks Shashi for working on this.  Some comments on the patch:
- The check should be in LogAppender.addEntry(..)
- It should throw RaftLogIOException instead of StateMachineException.
- Let's move the new test to ratis-server module instead of ratis-example.  
Indeed, we should move out all the non-example tests out from the ratis-example.


> Log Appender should throw an Exception in case  append entry size exceeds the 
> maxBufferSize configured
> ------------------------------------------------------------------------------------------------------
>
>                 Key: RATIS-176
>                 URL: https://issues.apache.org/jira/browse/RATIS-176
>             Project: Ratis
>          Issue Type: Bug
>            Reporter: Shashikant Banerjee
>            Assignee: Shashikant Banerjee
>            Priority: Minor
>         Attachments: RATIS-176.001.patch
>
>
> LogAppender while adding append entry in LogEntryBuffer, checks whether the 
> total allocated for all entries does not exceed the maxBufferSize allocated. 
> In case, the size exceeds the limit ,entries are not added to the buffer but 
> no exception is thrown . This case needs to be handled.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to