Unverified Commit 1eb8fb6d authored by qiaozhanwei's avatar qiaozhanwei Committed by GitHub
Browse files

NettyDecoder class decode method switch case remove break (#3035)

parent d87d2d80

dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/codec/NettyDecoder.java

+0 −5
Original line number Diff line number Diff line
@@ -55,19 +55,15 @@ public class NettyDecoder extends ReplayingDecoder<NettyDecoder.State> { 
            case MAGIC:

                checkMagic(in.readByte());

                checkpoint(State.COMMAND);

                break;

            case COMMAND:

                commandHeader.setType(in.readByte());

                checkpoint(State.OPAQUE);

                break;

            case OPAQUE:

                commandHeader.setOpaque(in.readLong());

                checkpoint(State.BODY_LENGTH);

                break;

            case BODY_LENGTH:

                commandHeader.setBodyLength(in.readInt());

                checkpoint(State.BODY);

                break;

            case BODY:

                byte[] body = new byte[commandHeader.getBodyLength()];

                in.readBytes(body);
@@ -79,7 +75,6 @@ public class NettyDecoder extends ReplayingDecoder<NettyDecoder.State> { 
                out.add(packet);

                //

                checkpoint(State.MAGIC);

                break;

            default:

                logger.warn("unknown decoder state {}", state());

        }