task.result can throw (#837)

* `task.result` can raise exception

* Remove redundant `except`

* Fix doc spell.  We need to fix it well, currently strategy feels dumb and PITA

* Move common words within `spelling_wordlist.txt`

Author: abhinavsingh

Makefile

@@ -29,7 +29,7 @@ endif
 .PHONY: all https-certificates sign-https-certificates ca-certificates
 .PHONY: lib-check lib-clean lib-test lib-package lib-coverage lib-lint lib-pytest
 .PHONY: lib-release-test lib-release lib-profile lib-doc
-.PHONY: lib-dep lib-flake8 lib-mypy
+.PHONY: lib-dep lib-flake8 lib-mypy lib-speedscope
 .PHONY: container container-run container-release container-build container-buildx
 .PHONY: devtools dashboard dashboard-clean
 
@@ -148,6 +148,23 @@ lib-profile:
 			--open-file-limit 65536 \
 			--log-file /dev/null
 
+lib-speedscope:
+	ulimit -n 65536 && \
+	sudo py-spy record \
+		-o profile.speedscope.json \
+		-f speedscope \
+		-t -F -s -- \
+		python -m proxy \
+			--hostname 127.0.0.1 \
+			--num-acceptors 1 \
+			--num-workers 1 \
+			--enable-web-server \
+			--plugin proxy.plugin.WebServerPlugin \
+			--local-executor \
+			--backlog 65536 \
+			--open-file-limit 65536 \
+			--log-file /dev/null
+
 devtools:
 	pushd dashboard && npm run devtools && popd
 

benchmark/compare.sh

@@ -84,7 +84,7 @@ benchmark_proxy_py() {
     --enable-web-server \
     --plugin proxy.plugin.WebServerPlugin \
     --disable-http-proxy \
-    --num-acceptors 1 \
+    --num-acceptors 10 \
     --local-executor \
     --log-file /dev/null > /dev/null 2>&1 &
   local SERVER_PID=$!

docs/spelling_wordlist.txt

@@ -1,13 +1,27 @@
 IPv
 Nginx
 Pluggable
-scm
+Submodules
+Subpackages
 Threadless
-threadless
-youtube
-socio
-sexualized
+WebSocket
+Websocket
+acceptor
+acceptors
+eventing
+faq
+html
+http
 https
+iterable
+readables
+scm
+sexualized
+socio
+tcp
+teardown
+threadless
+websocket
+writables
 www
-html
-faq
+youtube

proxy/__init__.py

@@ -7,12 +7,6 @@
 
     :copyright: (c) 2013-present by Abhinav Singh and contributors.
     :license: BSD, see LICENSE for more details.
-
-    .. spelling::
-
-       eventing
-       Submodules
-       Subpackages
 """
 from .proxy import entry_point, main, Proxy
 from .testing import TestCase

proxy/common/__init__.py

@@ -7,8 +7,4 @@
 
     :copyright: (c) 2013-present by Abhinav Singh and contributors.
     :license: BSD, see LICENSE for more details.
-
-    .. spelling::
-
-       Submodules
 """

proxy/common/utils.py

@@ -11,9 +11,6 @@
     .. spelling::
 
        utils
-       websocket
-       Websocket
-       WebSocket
 """
 import sys
 import ssl

proxy/core/acceptor/__init__.py

@@ -10,10 +10,7 @@
 
     .. spelling::
 
-       acceptor
-       acceptors
        pre
-       Submodules
 """
 from .acceptor import Acceptor
 from .pool import AcceptorPool

proxy/core/acceptor/acceptor.py

@@ -10,7 +10,6 @@
 
     .. spelling::
 
-       acceptor
        pre
 """
 import socket

proxy/core/acceptor/executors.py

@@ -7,10 +7,6 @@
 
     :copyright: (c) 2013-present by Abhinav Singh and contributors.
     :license: BSD, see LICENSE for more details.
-
-    .. spelling::
-
-       acceptor
 """
 import socket
 import logging

proxy/core/acceptor/local.py

@@ -7,11 +7,6 @@
 
     :copyright: (c) 2013-present by Abhinav Singh and contributors.
     :license: BSD, see LICENSE for more details.
-
-    .. spelling::
-
-        acceptor
-        teardown
 """
 import queue
 import logging
@@ -38,7 +33,7 @@ def __init__(self, *args: Any, **kwargs: Any) -> None:
     @property
     def loop(self) -> Optional[asyncio.AbstractEventLoop]:
         if self._loop is None:
-            self._loop = asyncio.new_event_loop()
+            self._loop = asyncio.get_event_loop_policy().new_event_loop()
         return self._loop
 
     def work_queue_fileno(self) -> Optional[int]: