From b540a2835ae68e68a3ea0cbc1555c8271e52fd2b Mon Sep 17 00:00:00 2001
From: Adam Lee <ali@pivotal.io>
Date: Wed, 2 Mar 2016 10:43:00 +0800
Subject: [PATCH] s3ext: report error once an exception is caught

---
 gpAux/extensions/gps3ext/src/S3ExtWrapper.cpp | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/gpAux/extensions/gps3ext/src/S3ExtWrapper.cpp b/gpAux/extensions/gps3ext/src/S3ExtWrapper.cpp
index 46abd10baa..70d429d398 100644
--- a/gpAux/extensions/gps3ext/src/S3ExtWrapper.cpp
+++ b/gpAux/extensions/gps3ext/src/S3ExtWrapper.cpp
@@ -15,6 +15,7 @@ S3ExtBase *CreateExtWrapper(const char *url) {
         S3ExtBase *ret = new S3Reader(url);
         return ret;
     } catch (...) {
+        S3ERROR("Caught an exception, aborting");
         return NULL;
     }
 }
@@ -100,6 +101,7 @@ bool S3Reader::Init(int segid, int segnum, int chunksize) {
         S3INFO("Got %d files to download", this->keylist->contents.size());
         this->getNextDownloader();
     } catch (...) {
+        S3ERROR("Caught an exception, aborting");
         return false;
     }
 
@@ -183,6 +185,7 @@ bool S3Reader::TransferData(char *data, uint64_t &len) {
         }
         len = buflen;
     } catch (...) {
+        S3ERROR("Caught an exception, aborting");
         return false;
     }
 
@@ -203,6 +206,7 @@ bool S3Reader::Destroy() {
             delete this->keylist;
         }
     } catch (...) {
+        S3ERROR("Caught an exception, aborting");
         return false;
     }
 
-- 
GitLab