Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/24272 )

Change subject: KUDU-3776 fix sporadic gperftools build failure
......................................................................

KUDU-3776 fix sporadic gperftools build failure

I had a build machine where KUDU-3776 was 100% reproducible starting with
a particular state of the 3rd-party workspace.  Without spending time on
the detailed RCA, I put together a fix that worked 100% for me at that
machine, while the build would fail 100% of the times if starting with
a particular state of the workspace without this patch.

One of the regular suspects is the incorrect tracking of transitive
dependencies, but I didn't see much sense in getting to the bottom of the
issue, given the new way of building gperftools is more consistent with
the build recipe provided in the top-level INSTALL file.

Change-Id: Ie8c26bcc3c2625741cd05b3cc459159434e9d60b
Reviewed-on: http://gerrit.cloudera.org:8080/24272
Tested-by: Kudu Jenkins
Reviewed-by: Zoltan Martonka <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
Reviewed-by: Marton Greber <[email protected]>
---
M thirdparty/build-definitions.sh
1 file changed, 10 insertions(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Zoltan Martonka: Looks good to me, but someone else must approve
  Abhishek Chennaka: Looks good to me, approved
  Marton Greber: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/24272
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie8c26bcc3c2625741cd05b3cc459159434e9d60b
Gerrit-Change-Number: 24272
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>

Reply via email to