PARQUET-789: Catch/translate ParquetExceptions in parquet::arrow::FileReader

I ran into this issue when debugging ARROW-406. I also changed the translated status type to `IOError` if that makes sense.

Author: Wes McKinney <wes.mckinney@twosigma.com>

Closes apache#201 from wesm/PARQUET-789 and squashes the following commits:

67a8c58 [Wes McKinney] Catch ParquetException in parquet::arrow FileReader::ReadFlatColumn/ReadFlatTable

Change-Id: Iedec3d68d1aa391ca10374ef97fffef7069b8374

Author: wesm

cpp/src/parquet/arrow/reader.cc

@@ -201,11 +201,19 @@ Status FileReader::GetFlatColumn(int i, std::unique_ptr<FlatColumnReader>* out)
 }
 
 Status FileReader::ReadFlatColumn(int i, std::shared_ptr<Array>* out) {
-  return impl_->ReadFlatColumn(i, out);
+  try {
+    return impl_->ReadFlatColumn(i, out);
+  } catch (const ::parquet::ParquetException& e) {
+    return ::arrow::Status::IOError(e.what());
+  }
 }
 
 Status FileReader::ReadFlatTable(std::shared_ptr<Table>* out) {
-  return impl_->ReadFlatTable(out);
+  try {
+    return impl_->ReadFlatTable(out);
+  } catch (const ::parquet::ParquetException& e) {
+    return ::arrow::Status::IOError(e.what());
+  }
 }
 
 const ParquetFileReader* FileReader::parquet_reader() const {

cpp/src/parquet/arrow/utils.h

@@ -26,11 +26,11 @@
 namespace parquet {
 namespace arrow {
 
-#define PARQUET_CATCH_NOT_OK(s)                    \
-  try {                                            \
-    (s);                                           \
-  } catch (const ::parquet::ParquetException& e) { \
-    return ::arrow::Status::Invalid(e.what());     \
+#define PARQUET_CATCH_NOT_OK(s)                     \
+  try {                                             \
+    (s);                                            \
+  } catch (const ::parquet::ParquetException& e) {  \
+    return ::arrow::Status::IOError(e.what());      \
   }
 
 #define PARQUET_IGNORE_NOT_OK(s) \