You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by mm...@apache.org on 2018/02/21 16:37:18 UTC
[incubator-pulsar] branch master updated: Fixed Lookup redirect
logic on Proxy side (#1265)
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 3dbfb68 Fixed Lookup redirect logic on Proxy side (#1265)
3dbfb68 is described below
commit 3dbfb686bc699cb7a9a7c2eaa1cd364db3cd5c51
Author: Jai Asher <ja...@ccs.neu.edu>
AuthorDate: Wed Feb 21 08:37:15 2018 -0800
Fixed Lookup redirect logic on Proxy side (#1265)
---
.../main/java/org/apache/pulsar/proxy/server/LookupProxyHandler.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/LookupProxyHandler.java b/pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/LookupProxyHandler.java
index 5c6951a..abf80e8 100644
--- a/pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/LookupProxyHandler.java
+++ b/pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/LookupProxyHandler.java
@@ -126,7 +126,7 @@ public class LookupProxyHandler {
String brokerUrl = connectWithTLS ? result.brokerUrlTls : result.brokerUrl;
if (result.redirect) {
// Need to try the lookup again on a different broker
- performLookup(clientRequestId, topic, brokerUrl, authoritative, numberOfRetries - 1);
+ performLookup(clientRequestId, topic, brokerUrl, result.authoritative, numberOfRetries - 1);
} else {
// Reply the same address for both TLS non-TLS. The reason is that whether we use TLS
// between proxy
--
To stop receiving notification emails like this one, please contact
mmerli@apache.org.