Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18840 )
Change subject: IMPALA-11469: (Addendum) Initialize BackendConfig ...................................................................... IMPALA-11469: (Addendum) Initialize BackendConfig Adds FrontendTestBase base class to FileMetadataLoaderTest to ensure BackendConfig is set. Also switches FileSystemUtilTest to use FrontendTestBase to be more consistent around initialization. Failure to initialize BackendConfig meant that test success would depend on their order. If FileSystemUtilTest or another test that initialized it were the first to use FileSystemUtil, then everything would pass. If not, the static declaration in FileSystemUtil would fail, causing the class to fail to be loaded. Later tests would fail with NoClassDefFound errors. Testing: - Ran frontend tests individually. Change-Id: Iaefea5ac982e42a8b0f3b693d16a8db10cc7de31 Reviewed-on: http://gerrit.cloudera.org:8080/18840 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/test/java/org/apache/impala/catalog/FileMetadataLoaderTest.java M fe/src/test/java/org/apache/impala/common/FileSystemUtilTest.java 2 files changed, 4 insertions(+), 11 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/18840 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iaefea5ac982e42a8b0f3b693d16a8db10cc7de31 Gerrit-Change-Number: 18840 Gerrit-PatchSet: 3 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]>
