Hi,

There seems to be a naming conflict between the Thrift and Hypertable 
headers.

If cwd is /opt/hypertable/0.9.6.3, there here are the conflicting 
definitions:

./include/thrift/config.h:324:#define VERSION "0.8.0" 
./include/ThriftBroker/SerializedCellsFlag.h:39:      VERSION        = 0x01

The result of building something 
(e.g. http://hypertable.com/documentation/code_examples/cpp/) is:

/opt/hypertable/0.9.6.3/include/ThriftBroker/SerializedCellsFlag.h:39:7: 
error: expected identifier before string constant
/opt/hypertable/0.9.6.3/include/ThriftBroker/SerializedCellsFlag.h:39:7: 
error: expected ‘}’ before string constant
/opt/hypertable/0.9.6.3/include/ThriftBroker/SerializedCellsFlag.h:39:7: 
error: expected unqualified-id before string constant
/opt/hypertable/0.9.6.3/include/ThriftBroker/SerializedCellsFlag.h:44:1: 
error: expected declaration before ‘}’ token
make: *** [client_test.o] Error 1

I am using the latest release 0.9.6.3, but I think this has existed 
previously.

In my code I just changed SerializedCellsVersion::VERSION to 
SerializedCellsVersion::SVERSION and seemed to work ok.

Separate from that, in the C++ example 
 http://hypertable.com/documentation/code_examples/cpp 
it seems there are several typos that make it unable to compile as is.

E.g. line 182:   
    std::vector<hypertable::thriftgen::cell> cells;
needs to be
    std::vector<Hypertable::ThriftGen::Cell> cells;

There are several other lines like that. If you want I'll send through my 
cleaned up
for compile version, but I used things like the C++11 range-based for 
instead of 
your foreach macro so not sure you want it.

Thanks 

Joel



-- 
You received this message because you are subscribed to the Google Groups 
"Hypertable Development" group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/hypertable-dev/-/kHFzRwjojFcJ.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/hypertable-dev?hl=en.

Reply via email to