2018-10-04 21:30:30 UTC - Sijie Guo: @Rajan Dhabalia - also any reason about this change : <https://github.com/apache/pulsar/pull/1946> ---- 2018-10-04 21:31:53 UTC - Sijie Guo: if the advertisedAddress is set to be allow external connections to broker, function can not use advertisedAddress to talk to itself though. ---- 2018-10-04 21:32:26 UTC - Sijie Guo: at least if people has set the urls, we should use those urls? ---- 2018-10-04 21:38:34 UTC - Rajan Dhabalia: trying to recollect this change.. yes, if worker is starting with in broker process then it should not connect to advertised address.. ---- 2018-10-04 21:38:36 UTC - Rajan Dhabalia: let me fix it ---- 2018-10-04 22:01:18 UTC - Sijie Guo: cool thank you ---- 2018-10-04 22:11:22 UTC - Rajan Dhabalia: we had added that change because: if broker-tls-authentication is enabled then function should use broker's tls service url to connect and authenticate. but it should not use advertised address. so, fixed it: <https://github.com/apache/pulsar/pull/2722> ---- 2018-10-05 02:47:09 UTC - Sijie Guo: @Joe Francis @Rajan Dhabalia: <https://github.com/apache/pulsar/pull/2730> please take a look at this schema change. it is related to the BC concern that @Joe Francis found. ----
