[jira] [Updated] (HDFS-3125) Add a service that enables JournalDaemon

2012-03-26 Thread Suresh Srinivas (Updated) (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-3125?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Suresh Srinivas updated HDFS-3125:
--

  Resolution: Fixed
Hadoop Flags: Reviewed
  Status: Resolved  (was: Patch Available)

I committed the patch.

> Add a service that enables JournalDaemon
> 
>
> Key: HDFS-3125
> URL: https://issues.apache.org/jira/browse/HDFS-3125
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>  Components: ha, name-node
>Reporter: Suresh Srinivas
>Assignee: Suresh Srinivas
> Attachments: HDFS-3125.patch, HDFS-3125.patch, HDFS-3125.patch, 
> HDFS-3125.patch
>
>
> In this subtask, I plan to add JournalService. It will provide the following 
> functionality:
> # Starts RPC server with JournalProtocolService or uses the RPC server 
> provided and add JournalProtocol service. 
> # Registers with the namenode.
> # Receives JournalProtocol related requests and hands it to over to a 
> listener.
>  

--
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] [Updated] (HDFS-3125) Add a service that enables JournalDaemon

2012-03-26 Thread Suresh Srinivas (Updated) (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-3125?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Suresh Srinivas updated HDFS-3125:
--

Attachment: HDFS-3125.patch

Updated patch with verifyRequest added to startLogSegment method

> Add a service that enables JournalDaemon
> 
>
> Key: HDFS-3125
> URL: https://issues.apache.org/jira/browse/HDFS-3125
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>  Components: ha, name-node
>Reporter: Suresh Srinivas
>Assignee: Suresh Srinivas
> Attachments: HDFS-3125.patch, HDFS-3125.patch, HDFS-3125.patch, 
> HDFS-3125.patch
>
>
> In this subtask, I plan to add JournalService. It will provide the following 
> functionality:
> # Starts RPC server with JournalProtocolService or uses the RPC server 
> provided and add JournalProtocol service. 
> # Registers with the namenode.
> # Receives JournalProtocol related requests and hands it to over to a 
> listener.
>  

--
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] [Updated] (HDFS-3125) Add a service that enables JournalDaemon

2012-03-23 Thread Suresh Srinivas (Updated) (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-3125?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Suresh Srinivas updated HDFS-3125:
--

Attachment: HDFS-3125.patch

bq. Question: why service is need in the methods of JournalListener?
Makes it convenient to know which listener is sending the call back. Then the 
listener can stop the service using the reference it gets.

bq. Does is make sense to support multiple listeners? If yes, we should 
implement the observer pattern.
I do not have a use case for multiple lister. If needed, we can add support for 
it later. What do you think?

Other comments addressed. I also changed exception handling - the exception 
from callbacks are thrown all the way to the client.


> Add a service that enables JournalDaemon
> 
>
> Key: HDFS-3125
> URL: https://issues.apache.org/jira/browse/HDFS-3125
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>  Components: ha, name-node
>Reporter: Suresh Srinivas
>Assignee: Suresh Srinivas
> Attachments: HDFS-3125.patch, HDFS-3125.patch, HDFS-3125.patch
>
>
> In this subtask, I plan to add JournalService. It will provide the following 
> functionality:
> # Starts RPC server with JournalProtocolService or uses the RPC server 
> provided and add JournalProtocol service. 
> # Registers with the namenode.
> # Receives JournalProtocol related requests and hands it to over to a 
> listener.
>  

--
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] [Updated] (HDFS-3125) Add a service that enables JournalDaemon

2012-03-23 Thread Suresh Srinivas (Updated) (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-3125?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Suresh Srinivas updated HDFS-3125:
--

Attachment: HDFS-3125.patch

Updated patch - ready for review.

> Add a service that enables JournalDaemon
> 
>
> Key: HDFS-3125
> URL: https://issues.apache.org/jira/browse/HDFS-3125
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>  Components: ha, name-node
>Reporter: Suresh Srinivas
>Assignee: Suresh Srinivas
> Attachments: HDFS-3125.patch, HDFS-3125.patch
>
>
> In this subtask, I plan to add JournalService. It will provide the following 
> functionality:
> # Starts RPC server with JournalProtocolService or uses the RPC server 
> provided and add JournalProtocol service. 
> # Registers with the namenode.
> # Receives JournalProtocol related requests and hands it to over to a 
> listener.
>  

--
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] [Updated] (HDFS-3125) Add a service that enables JournalDaemon

2012-03-22 Thread Suresh Srinivas (Updated) (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-3125?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Suresh Srinivas updated HDFS-3125:
--

Status: Patch Available  (was: Open)

> Add a service that enables JournalDaemon
> 
>
> Key: HDFS-3125
> URL: https://issues.apache.org/jira/browse/HDFS-3125
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>  Components: ha, name-node
>Reporter: Suresh Srinivas
>Assignee: Suresh Srinivas
> Attachments: HDFS-3125.patch
>
>
> In this subtask, I plan to add JournalService. It will provide the following 
> functionality:
> # Starts RPC server with JournalProtocolService or uses the RPC server 
> provided and add JournalProtocol service. 
> # Registers with the namenode.
> # Receives JournalProtocol related requests and hands it to over to a 
> listener.
>  

--
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] [Updated] (HDFS-3125) Add a service that enables JournalDaemon

2012-03-22 Thread Suresh Srinivas (Updated) (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-3125?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Suresh Srinivas updated HDFS-3125:
--

Summary: Add a service that enables JournalDaemon  (was: Add a service that 
enable JournalDaemon)

> Add a service that enables JournalDaemon
> 
>
> Key: HDFS-3125
> URL: https://issues.apache.org/jira/browse/HDFS-3125
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>  Components: ha, name-node
>Reporter: Suresh Srinivas
>Assignee: Suresh Srinivas
> Attachments: HDFS-3125.patch
>
>
> In this subtask, I plan to add JournalService. It will provide the following 
> functionality:
> # Starts RPC server with JournalProtocolService or uses the RPC server 
> provided and add JournalProtocol service. 
> # Registers with the namenode.
> # Receives JournalProtocol related requests and hands it to over to a 
> listener.
>  

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