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

stack commented on HBASE-5782:
------------------------------

Not sure I follow but I do know this patch more ambitious than what I was at....

+ You remove the 'other' sequence numbering system, unflushedEntries?  That 
looks good.
+ Are asserts on by default?  We disabled them a while back I believe?  You run 
w/ asserts? (Yeah, thats a good thing to test -- should you use your guava test 
instead?)
+ Its ugly we call it hlogFlush but internal we do appends (thats not your 
change)
+ I agree that the reset of the the pending writes linked list needs to be done 
under the synchronization held by hlogFlush
+ I like how you do pushback of edits if we fail....in hlogFlush.
+ On  this thing: 

{code}
+      // TODO: restore metric syncBatchSize.addAndGet(doneUpto - 
this.syncedTillHere);
{code}

Its not used anywhere and it the math looked dodgy... then when you read it it 
gets set to zero so I'm not so sure it is of any use.

                
> Edits can be appended out of seqid order since HBASE-4487
> ---------------------------------------------------------
>
>                 Key: HBASE-5782
>                 URL: https://issues.apache.org/jira/browse/HBASE-5782
>             Project: HBase
>          Issue Type: Bug
>          Components: wal
>    Affects Versions: 0.94.0
>            Reporter: Gopinathan A
>            Assignee: ramkrishna.s.vasudevan
>            Priority: Blocker
>             Fix For: 0.94.0
>
>         Attachments: 5782-sketch.txt, 5782.txt, HBASE-5782.patch
>
>
> Create a table with 1000 splits, after the region assignemnt, kill the 
> regionserver wich contains META table.
> Here few regions are missing after the log splitting and region assigment. 
> HBCK report shows multiple region holes are got created.
> Same scenario was verified mulitple times in 0.92.1, no issues.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to