You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Joe McDonnell (Code Review)" <ge...@cloudera.org> on 2022/06/22 01:30:04 UTC

[Impala-ASF-CR] IMPALA-11379: Remove kerberos.egg-info directory

Hello Impala Public Jenkins, 

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

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

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

Change subject: IMPALA-11379: Remove kerberos.egg-info directory
......................................................................

IMPALA-11379: Remove kerberos.egg-info directory

This directory is currently checked in, but it is
overwritten when building the shell. On some Linux
distributions, the output is different from what
is checked in. This causes problems for perf-AB-test
(based on bin/single_node_perf_run.py), which relies on
a build not causing any modifications.

This removes the kerberos.egg-info directory,
which does not need to be checked in.

This also adds checks to the GVO Jenkins jobs
to verify that the source tree is unmodified after
bootstrap_build.sh and boostrap_development.sh.
These checks are not included in those scripts
directly, because developers can run those scripts
in their development environments, which may have
modifications.

Tests:
 - Uploaded a change without removing the kerberos.egg-info
   directory and verified that the new checks fail
 - Verified that perf-AB-test gets past the current issue

Change-Id: I90b486bb6c1644fc18b56779d6c54e1e1b3c9aaa
---
M bin/jenkins/all-tests.sh
M bin/jenkins/build-only.sh
M bin/single_node_perf_run.py
D shell/ext-py/kerberos-1.3.1/kerberos.egg-info/PKG-INFO
D shell/ext-py/kerberos-1.3.1/kerberos.egg-info/SOURCES.txt
D shell/ext-py/kerberos-1.3.1/kerberos.egg-info/dependency_links.txt
D shell/ext-py/kerberos-1.3.1/kerberos.egg-info/top_level.txt
7 files changed, 29 insertions(+), 158 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/50/18650/2
-- 
To view, visit http://gerrit.cloudera.org:8080/18650
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I90b486bb6c1644fc18b56779d6c54e1e1b3c9aaa
Gerrit-Change-Number: 18650
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>