[query] fix NPE introduced in #12555 (#12711)

Not all IOExceptions have a cause.
hail-is · Feb 17, 2023 · 5f7e4be · 5f7e4be
1 parent dbfd553
commit 5f7e4be
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/hail/src/main/scala/is/hail/io/fs/GoogleStorageFS.scala b/hail/src/main/scala/is/hail/io/fs/GoogleStorageFS.scala
@@ -136,7 +136,7 @@ class GoogleStorageFS(
     makeUserProjectOption: String => U,
     bucket: String
   ): T = exc match {
-    case exc: IOException =>
+    case exc: IOException if exc.getCause() != null =>
       retryIfRequesterPays(exc.getCause(), makeRequest, makeUserProjectOption, bucket)
     case exc: StorageException =>
       retryIfRequesterPays(exc, exc.getMessage(), exc.getCode(), makeRequest, makeUserProjectOption, bucket)
@@ -332,7 +332,7 @@ class GoogleStorageFS(
     }
 
     def discoverExceptionThenRetryCopyIfRequesterPays(exc: Throwable): Unit = exc match {
-      case exc: IOException =>
+      case exc: IOException if exc.getCause() != null =>
         discoverExceptionThenRetryCopyIfRequesterPays(exc.getCause())
       case exc: StorageException =>
         retryCopyIfRequesterPays(exc, exc.getMessage(), exc.getCode())