tisonkun commented on code in PR #17062:
URL: https://github.com/apache/pulsar/pull/17062#discussion_r959393863


##########
bin/pulsar:
##########
@@ -257,8 +257,8 @@ if [ -z "$PULSAR_LOG_CONF" ]; then
     PULSAR_LOG_CONF=$DEFAULT_LOG_CONF
 fi
 
-if [ -z "$PULSAR_PRESTO_CONF" ]; then
-    PULSAR_PRESTO_CONF=$DEFAULT_PULSAR_PRESTO_CONF
+if [ -z "$PULSAR_TRINO_CONF" ]; then

Review Comment:
   Oops. I think I misunderstand your suggestion, followup as 
362207d3e54369175bca2553609f10ba0855ca7d.
   
   Then a version upgrade should go well, while a downgrade needs a caveat. And 
anyway, PIP-200 publicly drop the previous configure so that we can drop it in 
the following versions. This is for a smoothly deprecating period.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to