[jira] [Commented] (HBASE-4687) regionserver may miss zk-heartbeats to master when replaying edits at region open

2011-10-31 Thread Hudson (Commented) (JIRA)

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

Hudson commented on HBASE-4687:
---

Integrated in HBase-0.92 #90 (See 
[https://builds.apache.org/job/HBase-0.92/90/])
HBASE-4687 regionserver may miss zk-heartbeats to master when replaying 
edits at region open (prakash via jgray)

jgray : 
Files : 
* /hbase/branches/0.92/CHANGES.txt
* 
/hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java


> regionserver may miss zk-heartbeats to master when replaying edits at region 
> open
> -
>
> Key: HBASE-4687
> URL: https://issues.apache.org/jira/browse/HBASE-4687
> Project: HBase
>  Issue Type: Bug
>Reporter: Prakash Khemani
>Assignee: Prakash Khemani
> Fix For: 0.92.0
>
> Attachments: 
> 0001-HBASE-4687-regionserver-may-miss-zk-heartbeats-to-ma.patch
>
>
> replayRecoveredEdits() should do another reporter.progress() before returning.

--
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




[jira] [Commented] (HBASE-4687) regionserver may miss zk-heartbeats to master when replaying edits at region open

2011-10-28 Thread Hudson (Commented) (JIRA)

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

Hudson commented on HBASE-4687:
---

Integrated in HBase-TRUNK #2380 (See 
[https://builds.apache.org/job/HBase-TRUNK/2380/])
HBASE-4687  regionserver may miss zk-heartbeats to master when replaying 
edits at region open (prakash via jgray)

jgray : 
Files : 
* /hbase/trunk/CHANGES.txt
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java


> regionserver may miss zk-heartbeats to master when replaying edits at region 
> open
> -
>
> Key: HBASE-4687
> URL: https://issues.apache.org/jira/browse/HBASE-4687
> Project: HBase
>  Issue Type: Bug
>Reporter: Prakash Khemani
>Assignee: Prakash Khemani
> Fix For: 0.92.0
>
> Attachments: 
> 0001-HBASE-4687-regionserver-may-miss-zk-heartbeats-to-ma.patch
>
>
> replayRecoveredEdits() should do another reporter.progress() before returning.

--
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




[jira] [Commented] (HBASE-4687) regionserver may miss zk-heartbeats to master when replaying edits at region open

2011-10-28 Thread Jonathan Gray (Commented) (JIRA)

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

Jonathan Gray commented on HBASE-4687:
--

Thanks Prakash!

> regionserver may miss zk-heartbeats to master when replaying edits at region 
> open
> -
>
> Key: HBASE-4687
> URL: https://issues.apache.org/jira/browse/HBASE-4687
> Project: HBase
>  Issue Type: Bug
>Reporter: Prakash Khemani
>Assignee: Prakash Khemani
> Fix For: 0.92.0
>
> Attachments: 
> 0001-HBASE-4687-regionserver-may-miss-zk-heartbeats-to-ma.patch
>
>
> replayRecoveredEdits() should do another reporter.progress() before returning.

--
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




[jira] [Commented] (HBASE-4687) regionserver may miss zk-heartbeats to master when replaying edits at region open

2011-10-28 Thread jirapos...@reviews.apache.org (Commented) (JIRA)

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

jirapos...@reviews.apache.org commented on HBASE-4687:
--


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2585/#review2916
---

Ship it!


looks good to me

- Jonathan


On 2011-10-27 19:32:54, Prakash Khemani wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/2585/
bq.  ---
bq.  
bq.  (Updated 2011-10-27 19:32:54)
bq.  
bq.  
bq.  Review request for hbase, Dhruba Borthakur, Jonathan Gray, and Nicolas 
Spiegelberg.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  must report.progress() at least once per edits file when replaying edits
bq.  
bq.  
bq.  This addresses bug HBASE-4687.
bq.  https://issues.apache.org/jira/browse/HBASE-4687
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 5243b58 
bq.  
bq.  Diff: https://reviews.apache.org/r/2585/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  small change and not tested.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Prakash
bq.  
bq.



> regionserver may miss zk-heartbeats to master when replaying edits at region 
> open
> -
>
> Key: HBASE-4687
> URL: https://issues.apache.org/jira/browse/HBASE-4687
> Project: HBase
>  Issue Type: Bug
>Reporter: Prakash Khemani
>Assignee: Prakash Khemani
>
> replayRecoveredEdits() should do another reporter.progress() before returning.

--
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




[jira] [Commented] (HBASE-4687) regionserver may miss zk-heartbeats to master when replaying edits at region open

2011-10-27 Thread Prakash Khemani (Commented) (JIRA)

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

Prakash Khemani commented on HBASE-4687:


The progress reporter in this case heartbeats to the master via zk and
therefore I had called them zk-heartbeats. I know realize that calling
them zk-heartbeats easily confuses matters with the region server's "I am
Alive" zk-heartbeat.

In this case, master has put up the zk node /hbase/UNASSIGNED/ and
is waiting for the rs to open the region. The rs, while opening the
region, keeps updating that zk node to let the master know of that region
open is progressing.

On 10/27/11 12:46 PM, "jirapos...@reviews.apache.org (Commented) (JIRA)"




> regionserver may miss zk-heartbeats to master when replaying edits at region 
> open
> -
>
> Key: HBASE-4687
> URL: https://issues.apache.org/jira/browse/HBASE-4687
> Project: HBase
>  Issue Type: Bug
>Reporter: Prakash Khemani
>Assignee: Prakash Khemani
>
> replayRecoveredEdits() should do another reporter.progress() before returning.

--
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




[jira] [Commented] (HBASE-4687) regionserver may miss zk-heartbeats to master when replaying edits at region open

2011-10-27 Thread jirapos...@reviews.apache.org (Commented) (JIRA)

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

jirapos...@reviews.apache.org commented on HBASE-4687:
--


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2585/#review2889
---


Looks good to me.

Could you explain what the ramifications of not doing this are? I.e. what does 
the progress reporter have to do with ZK heartbeats?
Thanks!

- Lars


On 2011-10-27 19:32:54, Prakash Khemani wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/2585/
bq.  ---
bq.  
bq.  (Updated 2011-10-27 19:32:54)
bq.  
bq.  
bq.  Review request for hbase, Dhruba Borthakur, Jonathan Gray, and Nicolas 
Spiegelberg.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  must report.progress() at least once per edits file when replaying edits
bq.  
bq.  
bq.  This addresses bug HBASE-4687.
bq.  https://issues.apache.org/jira/browse/HBASE-4687
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 5243b58 
bq.  
bq.  Diff: https://reviews.apache.org/r/2585/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  small change and not tested.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Prakash
bq.  
bq.



> regionserver may miss zk-heartbeats to master when replaying edits at region 
> open
> -
>
> Key: HBASE-4687
> URL: https://issues.apache.org/jira/browse/HBASE-4687
> Project: HBase
>  Issue Type: Bug
>Reporter: Prakash Khemani
>Assignee: Prakash Khemani
>
> replayRecoveredEdits() should do another reporter.progress() before returning.

--
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




[jira] [Commented] (HBASE-4687) regionserver may miss zk-heartbeats to master when replaying edits at region open

2011-10-27 Thread jirapos...@reviews.apache.org (Commented) (JIRA)

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

jirapos...@reviews.apache.org commented on HBASE-4687:
--


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2585/
---

Review request for hbase, Dhruba Borthakur, Jonathan Gray, and Nicolas 
Spiegelberg.


Summary
---

must report.progress() at least once per edits file when replaying edits


This addresses bug HBASE-4687.
https://issues.apache.org/jira/browse/HBASE-4687


Diffs
-

  src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 5243b58 

Diff: https://reviews.apache.org/r/2585/diff


Testing
---

small change and not tested.


Thanks,

Prakash



> regionserver may miss zk-heartbeats to master when replaying edits at region 
> open
> -
>
> Key: HBASE-4687
> URL: https://issues.apache.org/jira/browse/HBASE-4687
> Project: HBase
>  Issue Type: Bug
>Reporter: Prakash Khemani
>Assignee: Prakash Khemani
>
> replayRecoveredEdits() should do another reporter.progress() before returning.

--
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