gcsio.py use RuntimeError with exc chaining instead of overwriting pr…

…evious exception (#28470) * use RuntimeError with exc chaining instead of overwriting previous exception * update changes
apache · Sep 15, 2023 · 108680a · 108680a
1 parent f09955e
commit 108680a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -80,7 +80,8 @@
 
 ## Bugfixes
 
-* Fixed X (Java/Python) ([#X](https://github.com/apache/beam/issues/X)).
+* Fixed exception chaining issue in GCS connector (Python) ([#26769](https://github.com/apache/beam/issues/26769#issuecomment-1700422615)).
+
 
 ## Security Fixes
 * Python containers updated, fixing [CVE-2021-30474](https://nvd.nist.gov/vuln/detail/CVE-2021-30474), [CVE-2021-30475](https://nvd.nist.gov/vuln/detail/CVE-2021-30475), [CVE-2021-30473](https://nvd.nist.gov/vuln/detail/CVE-2021-30473), [CVE-2020-36133](https://nvd.nist.gov/vuln/detail/CVE-2020-36133), [CVE-2020-36131](https://nvd.nist.gov/vuln/detail/CVE-2020-36131), [CVE-2020-36130](https://nvd.nist.gov/vuln/detail/CVE-2020-36130), and [CVE-2020-36135](https://nvd.nist.gov/vuln/detail/CVE-2020-36135)

diff --git a/sdks/python/apache_beam/io/gcp/gcsio.py b/sdks/python/apache_beam/io/gcp/gcsio.py
@@ -825,5 +825,4 @@ def finish(self):
     # Check for exception since the last put() call.
     if self._upload_thread.last_error is not None:
       e = self._upload_thread.last_error
-      raise type(self._upload_thread.last_error)(
-          "Error while uploading file %s" % self._path) from e  # pylint: disable=raising-bad-type
+      raise RuntimeError("Error while uploading file %s" % self._path) from e