[GitHub] [airflow] potiuk commented on a change in pull request #12740: Fix the exception that the port is empty when using db shell

[GitBox]

potiuk commented on a change in pull request #12740:
URL: https://github.com/apache/airflow/pull/12740#discussion_r533572104



##########
File path: airflow/cli/commands/db_command.py
##########
@@ -67,7 +67,7 @@ def shell(args):
                 host     = {url.host}
                 user     = {url.username}
                 password = {url.password or ""}
-                port     = {url.port or ""}

Review comment:
       I think it should depend on the database: Mysql = 3306, Postgres = 5432




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