RussellSpitzer commented on code in PR #1646:
URL: https://github.com/apache/polaris/pull/1646#discussion_r2102746037
##########
getting-started/jdbc/docker-compose.yml:
##########
@@ -29,18 +29,22 @@ services:
# Optional, allows attaching a debugger to the Polaris JVM
- "5005:5005"
environment:
- - JAVA_DEBUG=true
Review Comment:
I'm not sure this is required ... but i'm also not sure why the old version
worked. Wouldn't the previous syntax have essentially been
LIST
[ "JAVA_DEBUG=true", "JAVA_DEBUG_PORT=*:5005", ...]
While this syntax is
OBJECT
{ JAVA_DEBUG => true, JAVA_DEBUG_PORT => *:5005 }
? YAML is always confusing to me
--
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: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
