[GitHub] [airflow] mik-laj commented on a change in pull request #6594: [AIRFLOW-6001][depends on AIRFLOW-6000] Lazy load CLI commands

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

mik-laj commented on a change in pull request #6594: [AIRFLOW-6001][depends on AIRFLOW-6000] Lazy load CLI commands
URL: https://github.com/apache/airflow/pull/6594#discussion_r347296205
 
 

 ##########
 File path: airflow/bin/cli.py
 ##########
 @@ -20,1703 +20,38 @@
 """Command-line interface"""
 
 import argparse
-import errno
-import functools
-import getpass
-import importlib
-import json
-import logging
 import os
-import random
-import re
-import reprlib
-import signal
-import string
-import subprocess
-import sys
 import textwrap
-import threading
-import time
-import traceback
 from argparse import RawTextHelpFormatter
-from importlib import import_module
-from typing import Any
-from urllib.parse import urlunparse
+from typing import Callable
 
-import daemon
-import psutil
-from daemon.pidfile import TimeoutPIDLockFile
-from sqlalchemy.orm import exc
-from tabulate import tabulate, tabulate_formats
+from tabulate import tabulate_formats
 
 Review comment:
   https://github.com/apache/airflow/blob/911910e6feb2c04d7a0a69692832e0cc7e184c3e/airflow/bin/cli.py#L121-L128
   It is the ways to display the table

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


With regards,
Apache Git Services