cloudberrydb / cloudberrydb-site

Website sources for Cloudberry Database
http://cloudberrydb.org/
Apache License 2.0
10 stars 13 forks source link

CBDB Build on AWS/Rocky Linux 8 - make failure #167

Closed pjvldb closed 1 week ago

pjvldb commented 1 week ago

I'm following the install guide at https://cloudberrydb.org/docs/cbdb-linux-compile using an EC2 instance running Rocky Linux 8.

AMI is ami-020c6cfb9f8b61b53.

All is well until the 'make' step:

make[4]: Entering directory '/home/rocky/cloudberrydb/src/backend/access/appendonly' gcc -Wall -Wmissing-prototypes -Wpointer-arith -Werror=vla -Wendif-labels -Wmissing-format-attribute -Wimplicit-fallthrough=3 -Wcast-function-type -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -Wno-unused-but-set-variable -Werror=implicit-fallthrough=3 -Wno-format-truncation -Wno-stringop-truncation -O3 -fPIC -DUSE_INTERNAL_FTS=1 -Werror=uninitialized -Werror=implicit-function-declaration -Werror -I../../../../src/interfaces/libpq -I../../../../src/include -D_GNU_SOURCE -c -o appendonlyam.o appendonlyam.c appendonlyam.c: In function ‘appendonly_insert’: appendonlyam.c:3052:9: error: unused variable ‘firstSequence’ [-Werror=unused-variable] int64 firstSequence = GetFastSequences(aoInsertDesc->segrelid, ^~~~~ cc1: all warnings being treated as errors

github-actions[bot] commented 1 week ago

Hey, @pjvldb welcome!🎊 Thanks for taking the time to point this out.🙌

pjvldb commented 1 week ago

Also, FYI, the guide refers to yum for Rocky/RHEL v8 but this has been replaced by dnf.

edespino commented 1 week ago

@pjvldb Thank you for reporting this issue. I will try and reproduce it using the AMI you specified (or equivalent in my RegionZone).

pjvldb commented 1 week ago

Same error with RHEL9.

edespino commented 1 week ago

@pjvldb - It is quite easy to duplicate the issue on:

NAME="Rocky Linux"
VERSION="8.9 (Green Obsidian)"
edespino commented 1 week ago

@pjvldb This is a code issue and has nothing to do with the documentation. I am able to build Cloudberry 1.6.0 and go back a few to commits to 781c43b63b and Cloudberry builds without issue. But the current HEAD of main has problems.

I will raise this with the engineering team. Sorry for the inconvenience. In the meantime you can perform a git checkout 781c43b63b and your build should be happier.

edespino commented 1 week ago

@pjvldb I have filed https://github.com/cloudberrydb/cloudberrydb/issues/668 for the Cloudberrydb build issue. I will be closing this issue.

edespino commented 1 week ago

This is a source issue and not an issue with the documented build steps. I have filed https://github.com/cloudberrydb/cloudberrydb/issues/668 to address the source issue.