good catch

Thanks @merlimat — you're right, the URL scheme should be pulsar:// to match the (corrected) key. Applied your suggestion in 7836370 and force-pushed; commit message updated to credit the scheme change to the review. Thanks @dao-jun for the 👍 on the original report.

Just to clarify, config.put("serviceUrl", "http://localhost:xxxx"); is also valid.

Thanks for the clarification @dao-jun — you're right, http://localhost:xxxx is also accepted. ClientConfigurationData#serviceUrl takes both the native binary scheme (pulsar://host:6650, pulsar+ssl://host:6651) and the HTTP lookup scheme (http://host:8080, https://host:8443), so both spellings are perfectly valid values for the serviceUrl key.

I kept the example in this PR as pulsar://localhost:6650 to match @merlimat's inline suggestion and because that pairs naturally with the default broker port shown in the snippet. Happy to extend it to something like

// Any scheme accepted by PulsarAdmin is fine; e.g.
//   config.put("serviceUrl", "pulsar://localhost:6650");
//   config.put("serviceUrl", "http://localhost:8080");

if you think documenting both schemes in the Javadoc would be clearer for users landing on loadConf. Otherwise this PR is scoped purely to fixing the servicUrl → serviceUrl typo and I'd rather keep the doc change minimal — LMK which you prefer and I'll push a follow-up in the same PR.

Thanks for the clarification @dao-jun — you're right, http://localhost:xxxx is also accepted. ClientConfigurationData#serviceUrl takes both the native binary scheme (pulsar://host:6650, pulsar+ssl://host:6651) and the HTTP lookup scheme (http://host:8080, https://host:8443), so both spellings are perfectly valid values for the serviceUrl key.

I kept the example in this PR as pulsar://localhost:6650 to match @merlimat's inline suggestion and because that pairs naturally with the default broker port shown in the snippet. Happy to extend it to something like

// Any scheme accepted by PulsarAdmin is fine; e.g.
//   config.put("serviceUrl", "pulsar://localhost:6650");
//   config.put("serviceUrl", "http://localhost:8080");

if you think documenting both schemes in the Javadoc would be clearer for users landing on loadConf. Otherwise this PR is scoped purely to fixing the servicUrl → serviceUrl typo and I'd rather keep the doc change minimal — LMK which you prefer and I'll push a follow-up in the same PR.

No need to keep them both, this is just an example.