duhengforever closed pull request #509: [ISSUE #461]Purge filterserv related
code in client
URL: https://github.com/apache/rocketmq/pull/509
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git
a/client/src/main/java/org/apache/rocketmq/client/consumer/MQPushConsumer.java
b/client/src/main/java/org/apache/rocketmq/client/consumer/MQPushConsumer.java
index d56075c60..bc6d328c4 100644
---
a/client/src/main/java/org/apache/rocketmq/client/consumer/MQPushConsumer.java
+++
b/client/src/main/java/org/apache/rocketmq/client/consumer/MQPushConsumer.java
@@ -55,11 +55,15 @@
void subscribe(final String topic, final String subExpression) throws
MQClientException;
/**
+ * This method will be removed in the version 5.0.0,because filterServer
was removed,and method <code>subscribe(final String topic, final
MessageSelector messageSelector)</code>
+ * is recommended.
+ *
* Subscribe some topic
*
* @param fullClassName full class name,must extend
org.apache.rocketmq.common.filter. MessageFilter
* @param filterClassSource class source code,used UTF-8 file
encoding,must be responsible for your code safety
*/
+ @Deprecated
void subscribe(final String topic, final String fullClassName,
final String filterClassSource) throws MQClientException;
diff --git
a/client/src/main/java/org/apache/rocketmq/client/impl/factory/MQClientInstance.java
b/client/src/main/java/org/apache/rocketmq/client/impl/factory/MQClientInstance.java
index 1b3885629..9ffaed0a4 100644
---
a/client/src/main/java/org/apache/rocketmq/client/impl/factory/MQClientInstance.java
+++
b/client/src/main/java/org/apache/rocketmq/client/impl/factory/MQClientInstance.java
@@ -720,7 +720,11 @@ private boolean isBrokerInNameServer(final String
brokerAddr) {
return false;
}
-
+ /**
+ * This method will be removed in the version 5.0.0,because filterServer
was removed,and method <code>subscribe(final String topic, final
MessageSelector messageSelector)</code>
+ * is recommended.
+ */
+ @Deprecated
private void uploadFilterClassToAllFilterServer(final String
consumerGroup, final String fullClassName,
final String topic,
final String filterClassSource) throws UnsupportedEncodingException {
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
