impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Impala Public Jenkins (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-6049: breakpad tests: skip all tests with local filesystem
Date Sat, 14 Oct 2017 01:02:08 GMT
Impala Public Jenkins has submitted this change and it was merged. (

Change subject: IMPALA-6049: breakpad tests: skip all tests with local filesystem

IMPALA-6049: breakpad tests: skip all tests with local filesystem

The breakpad tests were recently refactored to support inclusion of one
of them as a core test. In this refactor, we neglected to ensure
setup_class() called its parent. This means the skipping called in said
parent doesn't occur, and the test is executed in an unsupported
environment (local filesystem).

This patch fixes that by ensuring we call the parent setup_class() via


$ TARGET_FILESYSTEM="local" impala-py.test tests/custom_cluster/ \
      -k test_abort_writes_minidump


Change-Id: Ib4a3ff29dd85c79c4c3b3e3afb699861e408aa95
Reviewed-by: Sailesh Mukil <>
Reviewed-by: Lars Volker <>
Tested-by: Impala Public Jenkins
M tests/custom_cluster/
1 file changed, 1 insertion(+), 0 deletions(-)

  Sailesh Mukil: Looks good to me, but someone else must approve
  Lars Volker: Looks good to me, approved
  Impala Public Jenkins: Verified

To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib4a3ff29dd85c79c4c3b3e3afb699861e408aa95
Gerrit-Change-Number: 8272
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Brown <>
Gerrit-Reviewer: Bikramjeet Vig <>
Gerrit-Reviewer: David Knupp <>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Lars Volker <>
Gerrit-Reviewer: Michael Brown <>
Gerrit-Reviewer: Sailesh Mukil <>

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message