hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "jiraposter@reviews.apache.org (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-3939) Some crossports of Hadoop IPC fixes
Date Sat, 09 Jul 2011 17:56:02 GMT

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

jiraposter@reviews.apache.org commented on HBASE-3939:
------------------------------------------------------


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


Needs unit tests before any +1:

- Confirm protocol signature (new feature) match and mismatch is handled correctly.

- Confirm this sends back nicer error responses when client is using an out of date IPC version

- Andrew


On 2011-06-24 14:42:47, Ted Yu wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/951/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-06-24 14:42:47)
bq.  
bq.  
bq.  Review request for hbase and Todd Lipcon.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  A few fixes from Hadoop IPC that we should probably cross-port into our copy:
bq.  
bq.      * HADOOP-7227: remove the protocol version check at call time
bq.      * HADOOP-7146: fix a socket leak in server
bq.      * HADOOP-7121: fix behavior when response serialization throws an exception
bq.      * HADOOP-7346: send back nicer error response when client is using an out of date
IPC version
bq.  
bq.  
bq.  This addresses bug HBASE-3939.
bq.      https://issues.apache.org/jira/browse/HBASE-3939
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    /src/main/java/org/apache/hadoop/hbase/coprocessor/AggregateImplementation.java 1137262

bq.    /src/main/java/org/apache/hadoop/hbase/coprocessor/AggregateProtocol.java 1137262 
bq.    /src/main/java/org/apache/hadoop/hbase/coprocessor/BaseEndpointCoprocessor.java 1137262

bq.    /src/main/java/org/apache/hadoop/hbase/ipc/CoprocessorProtocol.java 1137280 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/HBaseRPC.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/HBaseRpcMetrics.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/HBaseServer.java 1139326 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/HMasterInterface.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/HMasterRegionInterface.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/HRegionInterface.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/Invocation.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/ProtocolSignature.java PRE-CREATION 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/RpcEngine.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/Status.java PRE-CREATION 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/VersionedProtocol.java PRE-CREATION 
bq.    /src/main/java/org/apache/hadoop/hbase/ipc/WritableRpcEngine.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1134732 
bq.    /src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 1134732 
bq.    /src/test/java/org/apache/hadoop/hbase/regionserver/TestServerCustomProtocol.java 1137280

bq.  
bq.  Diff: https://reviews.apache.org/r/951/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Test suite passed.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ted
bq.  
bq.



> Some crossports of Hadoop IPC fixes
> -----------------------------------
>
>                 Key: HBASE-3939
>                 URL: https://issues.apache.org/jira/browse/HBASE-3939
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 0.92.0
>            Reporter: Todd Lipcon
>            Assignee: Ted Yu
>             Fix For: 0.92.0
>
>         Attachments: 3939-v2.txt, 3939-v3.txt, 3939.txt
>
>
> A few fixes from Hadoop IPC that we should probably cross-port into our copy:
> - HADOOP-7227: remove the protocol version check at call time
> - HADOOP-7146: fix a socket leak in server
> - HADOOP-7121: fix behavior when response serialization throws an exception
> - HADOOP-7346: send back nicer error response when client is using an out of date IPC
version

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message