Hello Lars Volker, Michael Brown, Nithya Janarthanan, Philip Zeyliger, Joe 
McDonnell, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/11257

to look at the new patch set (#9).

Change subject: IMPALA-7399: Emit a junit xml report when trapping errors
......................................................................

IMPALA-7399: Emit a junit xml report when trapping errors

This patch will cause a junitxml file to be emitted in the case of
errors in build scripts. Instead of simply echoing a message to the
console, we set up a trap function that also writes out the the
junit xml report that can be consumed by jenkins.impala.io.

Main things to pay attention to:

- New file that gets sourced by all bash scripts when trapping
  within bash scripts:

  https://gerrit.cloudera.org/c/11257/1/bin/report_build_error.sh

- Installation of the python lib into impala-python venv for use
  from within python files:

  https://gerrit.cloudera.org/c/11257/1/bin/impala-python-common.sh

- Change to the generate_junitxml.py file itself, for ease of
  
https://gerrit.cloudera.org/c/11257/1/lib/python/impala_py_lib/jenkins/generate_junitxml.py

Most of the other changes are to source the new report_build_error.sh
script to set up the trap function.

Change-Id: Idd62045bb43357abc2b89a78afff499149d3c3fc
---
M bin/clean-cmake.sh
M bin/clean.sh
M bin/create-test-configuration.sh
M bin/create_testdata.sh
M bin/distcc/distcc_server_setup.sh
M bin/impala-python-common.sh
M bin/jenkins/all-tests.sh
M bin/jenkins/build-all-flag-combinations.sh
M bin/jenkins/build-only.sh
M bin/make_impala.sh
A bin/report_build_error.sh
M bin/run-all-tests.sh
M bin/run-backend-tests.sh
M bin/start-catalogd.sh
M bin/start-impalad.sh
M bin/start-statestored.sh
M buildall.sh
M infra/python/bootstrap_virtualenv.py
M lib/python/impala_py_lib/jenkins/generate_junitxml.py
M shell/make_shell_tarball.sh
M testdata/bin/check-schema-diff.sh
M testdata/bin/compute-table-stats.sh
M testdata/bin/copy-data-sources.sh
M testdata/bin/copy-udfs-udas.sh
M testdata/bin/create-load-data.sh
M testdata/bin/create-table-many-blocks.sh
M testdata/bin/generate-load-nested.sh
M testdata/bin/kill-all.sh
M testdata/bin/kill-hbase.sh
M testdata/bin/kill-hive-server.sh
M testdata/bin/kill-java-service.sh
M testdata/bin/kill-sentry-service.sh
M testdata/bin/load-hive-builtins.sh
M testdata/bin/load-metastore-snapshot.sh
M testdata/bin/load-test-warehouse-snapshot.sh
M testdata/bin/run-all.sh
M testdata/bin/run-hbase.sh
M testdata/bin/run-hive-server.sh
M testdata/bin/run-mini-dfs.sh
M testdata/bin/run-sentry-service.sh
M testdata/bin/setup-hdfs-env.sh
M testdata/cluster/admin
M testdata/datasets/tpcds/preload
M testdata/datasets/tpch/preload
M tests/run-custom-cluster-tests.sh
M tests/run-process-failure-tests.sh
46 files changed, 159 insertions(+), 72 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/57/11257/9
--
To view, visit http://gerrit.cloudera.org:8080/11257
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idd62045bb43357abc2b89a78afff499149d3c3fc
Gerrit-Change-Number: 11257
Gerrit-PatchSet: 9
Gerrit-Owner: David Knupp <dkn...@cloudera.com>
Gerrit-Reviewer: David Knupp <dkn...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Michael Brown <mi...@cloudera.com>
Gerrit-Reviewer: Nithya Janarthanan <njanartha...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>

Reply via email to