impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) IMPALA-3253: Modify gen build to always output the right version
Date Wed, 29 Jun 2016 19:06:10 GMT
Sailesh Mukil has submitted this change and it was merged.

Change subject: IMPALA-3253: Modify to always output the right version

IMPALA-3253: Modify to always output the right version previously had a --noclean option which did not
overwrite the version information if it was already populated. Since
--noclean was the default option, it always never updated the version

This patch modifies to generate a
common/ instead of a common/version.h. Now,
common/version.h will be a part of the git repo and will not need to
be modified on every build. It declares the functions that will return
the build information. These functions will be defined in
common/ and the build information will change on every new

Since only the .cc file changes on every build, we will not incur a
highly noticable change in build times.

Also changed the function names from GetImpalaBuild...() to
GetImpaladBuild...() so as to avoid naming confusion between the
Impala-lzo and the Impala functions.

There is an accompanying change in the Impala-lzo library too.

Change-Id: Ie461110b6f8ca545f04ea33b7b502aea550b8551
Reviewed-by: Sailesh Mukil <>
Tested-by: Sailesh Mukil <>
M be/.gitignore
M be/src/common/CMakeLists.txt
A be/src/common/version.h
M be/src/exec/
M be/src/exec/
M be/src/service/
M be/src/util/
M bin/
M bin/
9 files changed, 72 insertions(+), 38 deletions(-)

  Sailesh Mukil: Looks good to me, approved; Verified

To view, visit
To unsubscribe, visit

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie461110b6f8ca545f04ea33b7b502aea550b8551
Gerrit-PatchSet: 10
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Sailesh Mukil <>
Gerrit-Reviewer: Casey Ching <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Sailesh Mukil <>

View raw message