[GitHub] [pulsar] eaba edited a comment on pull request #6757: [#6748][sql] Fix is misleading in presto configuration

[GitBox <gi...@apache.org>]

eaba edited a comment on pull request #6757:
URL: https://github.com/apache/pulsar/pull/6757#issuecomment-626096300


   How does the `pulsar.properties` get injected?
   How about this?:
   ````
   public void startPrestoWorker() {
           if (null == prestoWorkerContainer) {
               prestoWorkerContainer = new PrestoWorkerContainer(clusterName, PrestoWorkerContainer.NAME)
                       .withNetwork(network)
                       .withNetworkAliases(PrestoWorkerContainer.NAME)
                       .withEnv("clusterName", clusterName)
                       .withEnv("zkServers", ZKContainer.NAME)
                       .withEnv("zookeeperServers", ZKContainer.NAME + ":" + ZKContainer.ZK_PORT)
                       .withEnv("pulsar.zookeeper-uri", ZKContainer.NAME + ":" + ZKContainer.ZK_PORT)
                       .withEnv("pulsar.broker-service-url", "http://pulsar-broker-0:8080");
           }
           log.info("Starting Presto Worker");
           prestoWorkerContainer.start();
       }
   ````
   This has to do with connection to either pulsar broker or pulsar sql


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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