You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2018/02/19 05:11:29 UTC

[GitHub] jai1 commented on issue #1248: Start Proxy in TLS only mode.

jai1 commented on issue #1248: Start Proxy in TLS only mode.
URL: https://github.com/apache/incubator-pulsar/pull/1248#issuecomment-366590381
 
 
   @merlimat - have addressed your review comments
   Had to change the ports data type to integer else you get 
   ```
   java.lang.IllegalArgumentException: failed to initialize servicePort field while setting value 
   	at org.apache.pulsar.common.util.FieldParser.lambda$update$0(FieldParser.java:126)
   	at java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:948)
   	at java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:580)
   	at org.apache.pulsar.common.util.FieldParser.update(FieldParser.java:120)
   	at org.apache.pulsar.common.configuration.PulsarConfigurationLoader.create(PulsarConfigurationLoader.java:89)
   	at org.apache.pulsar.common.configuration.PulsarConfigurationLoader.create(PulsarConfigurationLoader.java:73)
   	at org.apache.pulsar.common.configuration.PulsarConfigurationLoader.create(PulsarConfigurationLoader.java:54)
   	at org.apache.pulsar.proxy.server.ProxyServiceStarter.<init>(ProxyServiceStarter.java:79)
   	at org.apache.pulsar.proxy.server.ProxyServiceStarter.main(ProxyServiceStarter.java:127)
   Caused by: java.lang.IllegalArgumentException: Can not set int field org.apache.pulsar.proxy.server.ProxyConfiguration.servicePort to null value
   	at sun.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(UnsafeFieldAccessorImpl.java:167)
   	at sun.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(UnsafeFieldAccessorImpl.java:171)
   	at sun.reflect.UnsafeIntegerFieldAccessorImpl.set(UnsafeIntegerFieldAccessorImpl.java:80)
   	at java.lang.reflect.Field.set(Field.java:764)
   	at org.apache.pulsar.common.util.FieldParser.lambda$update$0(FieldParser.java:124)
   ```
   
   I will make changes to the broker side and Discovery Configuration as a part of  a separate PR 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services