thempatel commented on a change in pull request #16961:
URL: https://github.com/apache/beam/pull/16961#discussion_r841095107
##########
File path: sdks/python/gen_protos.py
##########
@@ -15,40 +15,100 @@
# limitations under the License.
#
-"""Generates Python proto modules and grpc stubs for Beam protos."""
+"""
+Generates Python proto modules and grpc stubs for Beam protos.
+"""
+
import contextlib
import glob
import inspect
import logging
-import multiprocessing
import os
import platform
import re
import shutil
import subprocess
import sys
import time
-import warnings
+from collections import defaultdict
from importlib import import_module
import pkg_resources
+LOG = logging.getLogger()
+LOG.setLevel(logging.INFO)
+
+LICENSE_HEADER = """
Review comment:
If you're not strongly opposed, I suggest we keep these. My argument is
less related to the build system, and more related to messaging to consumers of
the SDK: if they are exploring the SDK, it is a useful notice to have so that
they know this is Beam's posture
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
