cf_atomic.h is not compatible with POWERPC about aerospike-client-c

[BrianNichols]

aerospike-client-c does not allow direct branching. You will need to fork the repository and then make a pull request (Click "Fork" icon in upper right hand corner on aerospike-client-c github page).

Note that "aerospike/ck" is just a copy of the concurrencykit repository include files with some minor changes.

ConcurrencyKit

I suggest submitting a pull request to the ConcurrencyKit too. That way, both repos can stay in sync.

[BrianNichols]

Also, which version of aerospike-client-c are you using? The assembly code in cf_atomic.h was replaced with ConcurrenyKit methods over 6 months ago. See:

https://github.com/aerospike/aerospike-common/blob/bc285243acc5333fc4abd85f47c66e85cfff2f03/src/include/citrusleaf/cf_atomic.h

I think the Aerospike code in PerfKitBenchmarker is out-of-date.

[mlpesant]

@BrianNichols, thanks for your response. It took me a little while to get this figured out, but I committed a change to aerospike-client-c/Makefile yesterday and then pushed it with the command
git push origin ppc64
I then tried to do a pull request, but it says there's nothing to compare. Will my change make it in or is there something else I need to do? Thanks a lot for your help. I'm still learning how to work with git and the open source community.

[BrianNichols]

The pull requests were all received. The aerospike-client-c and aerospike-mod-lua pull requests had the correct Makefile change. The aerospike-common pull request had lots of changes that clearly was crafted from a very old version. I modified aerospike-common to only include the Makefile change.

[BrianNichols]

C client 4.0.4 has been released.