Improve import time

.coveragerc

@@ -0,0 +1,4 @@
+[run]
+branch = True
+source = aiohttp, tests
+omit = site-packages

CHANGES/6591.misc

@@ -0,0 +1 @@
+Changed importing Gunicorn to happen on-demand, decreasing import time by ~53% -- :user:`Dreamsorcerer`.

aiohttp/__init__.py

@@ -1,6 +1,6 @@
 __version__ = "4.0.0a1"
 
-from typing import Tuple
+from typing import TYPE_CHECKING, Tuple
 
 from . import hdrs as hdrs
 from .client import (
@@ -103,6 +103,13 @@
     TraceResponseChunkReceivedParams as TraceResponseChunkReceivedParams,
 )
 
+if TYPE_CHECKING:
+    # At runtime these are lazy-loaded at the bottom of the file.
+    from .worker import (
+        GunicornUVLoopWebWorker as GunicornUVLoopWebWorker,
+        GunicornWebWorker as GunicornWebWorker,
+    )
+
 __all__: Tuple[str, ...] = (
     "hdrs",
     # client
@@ -203,11 +210,24 @@
     "TraceRequestRedirectParams",
     "TraceRequestStartParams",
     "TraceResponseChunkReceivedParams",
+    # workers (imported lazily with __getattr__)
+    "GunicornUVLoopWebWorker",
+    "GunicornWebWorker",
 )
 
-try:
-    from .worker import GunicornUVLoopWebWorker, GunicornWebWorker
 
-    __all__ += ("GunicornWebWorker", "GunicornUVLoopWebWorker")
-except ImportError:  # pragma: no cover
-    pass
+def __getattr__(name: str) -> object:
+    global GunicornUVLoopWebWorker, GunicornWebWorker
+
+    # Importing gunicorn takes a long time (>100ms), so only import if actually needed.
+    if name in ("GunicornUVLoopWebWorker", "GunicornWebWorker"):
+        try:
+            from .worker import GunicornUVLoopWebWorker as guv, GunicornWebWorker as gw
+        except ImportError:
+            return None
+
+        GunicornUVLoopWebWorker = guv  # type: ignore[misc]
+        GunicornWebWorker = gw  # type: ignore[misc]
+        return guv if name == "GunicornUVLoopWebWorker" else gw
+
+    raise AttributeError(f"module {__name__} has no attribute {name}")

setup.cfg

@@ -104,11 +104,6 @@ exclude_lines =
     @abc.abstractmethod
     @abstractmethod
 
-[coverage:run]
-branch = True
-source = aiohttp, tests
-omit = site-packages
-
 [tool:pytest]
 addopts =
     # show 10 slowest invocations:

tests/test_imports.py

@@ -0,0 +1,45 @@
+import platform
+import sys
+
+import pytest
+
+
+def test___all__(pytester: pytest.Pytester) -> None:
+    """See https://github.com/aio-libs/aiohttp/issues/6197"""
+    pytester.makepyfile(
+        test_a="""
+            from aiohttp import *
+            assert 'GunicornWebWorker' in globals()
+        """
+    )
+    result = pytester.runpytest("-vv")
+    if sys.platform == "win32":
+        print(result.outlines)
+    result.assert_outcomes(passed=0, errors=0)
+
+
+def test_web___all__(pytester: pytest.Pytester) -> None:
+    pytester.makepyfile(
+        test_b="""
+            from aiohttp.web import *
+        """
+    )
+    result = pytester.runpytest("-vv")
+    result.assert_outcomes(passed=0, errors=0)
+
+
+@pytest.mark.skipif(
+    not sys.platform.startswith("linux") or platform.python_implementation() == "PyPy",
+    reason="Timing is less reliable on other platforms",
+)
+def test_import_time(pytester: pytest.Pytester) -> None:
+    """Check that importing aiohttp doesn't take too long.
+
+    Obviously, the time may vary on different machines and may need to be adjusted
+    from time to time, but this should provide an early warning if something is
+    added that significantly increases import time.
+    """
+    r = pytester.run(sys.executable, "-We", "-c", "import aiohttp", timeout=0.45)
+
+    assert not r.stdout.str()
+    assert not r.stderr.str()