Code0x58 commented on pull request #3619: URL: https://github.com/apache/incubator-heron/pull/3619#issuecomment-696372987
does setting `--base_url=` work as expected as I feel it should? If so, I'm thinking it would be cleaner to use something better than `default` when setting the url, and to instead use something like `value = config_value if config_value is not None else config_value` - that would avoid the new coupled variable currently introduced by the PR. ---------------------------------------------------------------- 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: [email protected]
