BjornPrime commented on code in PR #25965:
URL: https://github.com/apache/beam/pull/25965#discussion_r1245601310
##########
sdks/python/apache_beam/io/gcp/gcsio.py:
##########
@@ -59,20 +48,10 @@
_LOGGER = logging.getLogger(__name__)
-# Issue a friendlier error message if the storage library is not available.
-# TODO(silviuc): Remove this guard when storage is available everywhere.
try:
- # pylint: disable=wrong-import-order, wrong-import-position
- # pylint: disable=ungrouped-imports
- from apitools.base.py.batch import BatchApiRequest
- from apitools.base.py.exceptions import HttpError
- from apitools.base.py import transfer
from apache_beam.internal.gcp import auth
- from apache_beam.io.gcp.internal.clients import storage
except ImportError:
- raise ImportError(
- 'Google Cloud Storage I/O not supported for this execution environment '
- '(could not import storage API client).')
+ raise ImportError('Internal auth library not found')
Review Comment:
Okay. Removed the exception handling.
--
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]
