Skip to content

Commit

Permalink
Merge pull request #2047, deprecate dubbo-rpc-thrift.
Browse files Browse the repository at this point in the history
  • Loading branch information
lovepoem authored and chickenlj committed Jul 9, 2018
1 parent d2df156 commit 026693c
Show file tree
Hide file tree
Showing 14 changed files with 48 additions and 12 deletions.
3 changes: 2 additions & 1 deletion .codecov.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,5 @@ coverage:
ignore:
- "dubbo-demo/.*"
- "dubbo-common/src/main/java/org/apache/dubbo/common/json/*.java" # internal JSON impl is deprecate, ignore test coverage for them
- "dubbo-config/dubbo-config-spring/src/main/java/org/apache/dubbo/config/spring/AnnotationBean.java" # Deprecated
- "dubbo-config/dubbo-config-spring/src/main/java/org/apache/dubbo/config/spring/AnnotationBean.java" # Deprecated
- "dubbo-rpc/dubbo-rpc-thrift/.*"
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,10 @@
package org.apache.dubbo.rpc.protocol.thrift;

import org.apache.dubbo.common.extension.SPI;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
@SPI(DubboClassNameGenerator.NAME)
public interface ClassNameGenerator {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,10 @@
* limitations under the License.
*/
package org.apache.dubbo.rpc.protocol.thrift;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public class DubboClassNameGenerator implements ClassNameGenerator {

public static final String NAME = "dubbo";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,10 @@
* limitations under the License.
*/
package org.apache.dubbo.rpc.protocol.thrift;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public class ThriftClassNameGenerator implements ClassNameGenerator {

public static final String NAME = "thrift";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,10 @@
* </ol>
* </p>
*/
/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public class ThriftCodec implements Codec2 {

public static final int MESSAGE_LENGTH_INDEX = 2;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,10 @@
* limitations under the License.
*/
package org.apache.dubbo.rpc.protocol.thrift;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public final class ThriftConstants {

public static final String THRIFT_PROTOCOL_KEY = "thrift.protocol";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,10 @@

import java.util.Set;
import java.util.concurrent.locks.ReentrantLock;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public class ThriftInvoker<T> extends AbstractInvoker<T> {

private final ExchangeClient[] clients;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,10 @@

import java.io.IOException;
import java.util.concurrent.atomic.AtomicInteger;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public class ThriftNativeCodec implements Codec2 {

private final AtomicInteger thriftSeq = new AtomicInteger(0);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,10 @@
import java.util.concurrent.CompletableFuture;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.ConcurrentMap;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public class ThriftProtocol extends AbstractProtocol {

public static final int DEFAULT_PORT = 40880;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,10 @@

import java.util.HashMap;
import java.util.Map;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public enum ThriftType {

BOOL, BYTE, I16, I32, I64, DOUBLE, STRING;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,10 @@
* limitations under the License.
*/
package org.apache.dubbo.rpc.protocol.thrift;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift instead
*/
@Deprecated
public class ThriftUtils {

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.ConcurrentMap;

/**
* @since 2.7.0, use https://github.com/dubbo/dubbo-rpc-native-thrift to instead of this module
*/
@Deprecated
public class MultiServiceProcessor implements TProcessor {

private static final Logger logger = LoggerFactory.getLogger(MultiServiceProcessor.class);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

import java.io.IOException;
import java.io.InputStream;

@Deprecated
public class InputStreamWrapper extends InputStream {

private InputStream is;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
import java.io.OutputStream;
import java.io.UnsupportedEncodingException;
import java.nio.ByteBuffer;

@Deprecated
public class RandomAccessByteArrayOutputStream extends OutputStream {

protected byte buffer[];
Expand Down

0 comments on commit 026693c

Please sign in to comment.