)]}'
{
  "commit": "f973727a0bec75a6e352c0f51a7af2630ac3ec42",
  "tree": "9e1dcbe8e28b595eb3ecce07d425896c8c2c9c0a",
  "parents": [
    "26422f58c47cfd44b003248fe8bac05f1a65bb4d"
  ],
  "author": {
    "name": "Joshua Storck",
    "email": "joshua.storck@twosigma.com",
    "time": "Mon Apr 23 22:51:32 2018 +0200"
  },
  "committer": {
    "name": "Uwe L. Korn",
    "email": "uwe@apache.org",
    "time": "Mon Apr 23 22:51:32 2018 +0200"
  },
  "message": "PARQUET-1279: [C++] Adding use of ASSERT_NO_FATAL_FAILURE in unit tests when calling helper functions that call ASSERT_ macros\n\nThere were a number of helper functions in the unit tests that has ASSERT_ macros. However, the callers of these functions were not asserting if the helper functions had encountered an assertion. In those circumstances, the unit test will continue because the ASSERT_ macros only return in the block in which they are used. This commit adds the use of ASSERT_NO_FATAL_FAILURE to the calls of those helper functions at the scope of the unit test, which will cause the unit test to exit. This ensures that we won\u0027t encounter segfaults or unexpected behavior due to a unit test continuing execution past an asserting inside of a helper function.\n\nAuthor: Joshua Storck \u003cjoshua.storck@twosigma.com\u003e\n\nCloses #458 from joshuastorck/use_assert_no_fatal_failure and squashes the following commits:\n\nfa89f5d [Joshua Storck] There were a number of helper functions in the unit tests that has ASSERT_ macros. However, the callers of these functions were not asserting if the helper functions had encountered an assertion. In those circumstances, the unit test will continue because the ASSERT_ macros only return in the block in which they are used. This commit adds the use of ASSERT_NO_FATAL_FAILURE to the calls of those helper functions at the scope of the unit test, which will cause the unit test to exit. This ensures that we won\u0027t encounter segfaults or unexpected behavior due to a unit test continuing execution past an asserting inside of a helper function.\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "cb38b8fb40f5352faeb34d3c7bf9b291a6394179",
      "old_mode": 33188,
      "old_path": "src/parquet/arrow/arrow-reader-writer-test.cc",
      "new_id": "47226a3c672cb9c2d1eee706380d6b4b5e38f23c",
      "new_mode": 33188,
      "new_path": "src/parquet/arrow/arrow-reader-writer-test.cc"
    },
    {
      "type": "modify",
      "old_id": "da6af528c6e16dd2f4d2af69e879033d89caffad",
      "old_mode": 33188,
      "old_path": "src/parquet/arrow/arrow-schema-test.cc",
      "new_id": "5c16c044ff1b3a4c388c3d46d39d5bf8945ec380",
      "new_mode": 33188,
      "new_path": "src/parquet/arrow/arrow-schema-test.cc"
    },
    {
      "type": "modify",
      "old_id": "ebb70f1d0e0af95d8ffef1124946973e8c80461b",
      "old_mode": 33188,
      "old_path": "src/parquet/column_reader-test.cc",
      "new_id": "15ddc8bb6fdc3da2f1bd77512def17e3a2a64c4f",
      "new_mode": 33188,
      "new_path": "src/parquet/column_reader-test.cc"
    },
    {
      "type": "modify",
      "old_id": "1ebc71968a2f1052c6e9f19a727debd6336ab500",
      "old_mode": 33188,
      "old_path": "src/parquet/column_scanner-test.cc",
      "new_id": "d39b9103d7ddf4490562d7118f7924b4edc681d5",
      "new_mode": 33188,
      "new_path": "src/parquet/column_scanner-test.cc"
    },
    {
      "type": "modify",
      "old_id": "224a23d75a3dfee618974c8828d7f25f7e5b38e6",
      "old_mode": 33188,
      "old_path": "src/parquet/column_writer-test.cc",
      "new_id": "7a5f379ba3c5857cca905080635b7f54d59acee3",
      "new_mode": 33188,
      "new_path": "src/parquet/column_writer-test.cc"
    },
    {
      "type": "modify",
      "old_id": "0da32cf02fdca6e9610eaced9eea88bc5973b48e",
      "old_mode": 33188,
      "old_path": "src/parquet/encoding-test.cc",
      "new_id": "8d97bffbbef4b7eb7e19533f454fd86dc176ec05",
      "new_mode": 33188,
      "new_path": "src/parquet/encoding-test.cc"
    },
    {
      "type": "modify",
      "old_id": "5e1737555ce9362bdee0c86988f1a9acb3370b4f",
      "old_mode": 33188,
      "old_path": "src/parquet/file-deserialize-test.cc",
      "new_id": "6b01ac21bc767db6bc41d80b8a22c884a5f99ce9",
      "new_mode": 33188,
      "new_path": "src/parquet/file-deserialize-test.cc"
    },
    {
      "type": "modify",
      "old_id": "b4df77efbd30a3a712f4983d1a323772f7520e92",
      "old_mode": 33188,
      "old_path": "src/parquet/file-serialize-test.cc",
      "new_id": "16a7c4f381447be63ba3190aeb1ace0b932147dd",
      "new_mode": 33188,
      "new_path": "src/parquet/file-serialize-test.cc"
    },
    {
      "type": "modify",
      "old_id": "ec9aff42c2f4ee5c2a9b96ce513e807dfaf928dc",
      "old_mode": 33188,
      "old_path": "src/parquet/schema-test.cc",
      "new_id": "a734a926b4854fb3bc288a40ef89e7a71e5a32ff",
      "new_mode": 33188,
      "new_path": "src/parquet/schema-test.cc"
    },
    {
      "type": "modify",
      "old_id": "54740166fe179b471c8c3eae793c5cd470a4dd0a",
      "old_mode": 33188,
      "old_path": "src/parquet/statistics-test.cc",
      "new_id": "bf3d196898767fa63c2a350c7036421c10e799f3",
      "new_mode": 33188,
      "new_path": "src/parquet/statistics-test.cc"
    }
  ]
}