[ 
https://issues.apache.org/jira/browse/HADOOP-11245?focusedWorklogId=584924&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-584924
 ]

ASF GitHub Bot logged work on HADOOP-11245:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 19/Apr/21 07:02
            Start Date: 19/Apr/21 07:02
    Worklog Time Spent: 10m 
      Work Description: jojochuang commented on a change in pull request #2832:
URL: https://github.com/apache/hadoop/pull/2832#discussion_r615587865



##########
File path: 
hadoop-common-project/hadoop-nfs/src/main/java/org/apache/hadoop/oncrpc/SimpleTcpClientHandler.java
##########
@@ -39,27 +37,27 @@ public SimpleTcpClientHandler(XDR request) {
   }
 
   @Override
-  public void channelConnected(ChannelHandlerContext ctx, ChannelStateEvent e) 
{
+  public void channelActive(ChannelHandlerContext ctx) throws Exception {
     // Send the request
     if (LOG.isDebugEnabled()) {
       LOG.debug("sending PRC request");
     }
-    ChannelBuffer outBuf = XDR.writeMessageTcp(request, true);
-    e.getChannel().write(outBuf);
+    ByteBuf outBuf = XDR.writeMessageTcp(request, true);
+    ctx.channel().writeAndFlush(outBuf);
   }
 
   /**
    * Shutdown connection by default. Subclass can override this method to do
    * more interaction with the server.
    */
   @Override
-  public void messageReceived(ChannelHandlerContext ctx, MessageEvent e) {
-    e.getChannel().close();
+  public void channelRead(ChannelHandlerContext ctx, Object msg) {
+    ctx.channel().closeFuture().awaitUninterruptibly();

Review comment:
       I'll rewrite this part according to the official doc.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 584924)
    Time Spent: 2h 20m  (was: 2h 10m)

> Update NFS gateway to use Netty4
> --------------------------------
>
>                 Key: HADOOP-11245
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11245
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: nfs
>            Reporter: Brandon Li
>            Assignee: Wei-Chiu Chuang
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 2h 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to