-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
not compatible with latest avro library #18
Comments
That's breaking the semver rules, minor version bumps must be backwards compatible. :( |
how can I get the avro version that works with current libserdes? It would be wise if your readme file has links to the latest compatible avro version and not the latest stable. |
avro-cpp v1.8.0 |
I opened #17 a while ago in an attempt to address this. (Of course, you could then argue that requiring C++11 in libserdes also warrants a major version bump...) |
will this fix be available any time soon? I am also facing the same issue. Serdes.cpp:285:30: error: conversion from ‘avro::InputStreamPtr {aka std::unique_ptravro::InputStream}’ to non-scalar type ‘std::auto_ptravro::InputStream’ requested |
Solution to Anwesha25 : To resolve above error in Serdes.cpp:285:30, update below : |
liserdes cannot be compiled because of the following error:
Serdes.cpp:224:71: error: conversion from 'avro::OutputStreamPtr {aka std::unique_ptravro::OutputStream}' to non-scalar type 'std::auto_ptravro::OutputStream' requested
std::auto_ptravro::OutputStream bin_os = avro::memoryOutputStream();
Serdes.cpp:242:5: error: no match for 'operator=' (operand types are 'boost::shared_ptr<std::vector >' and 'std::shared_ptr<std::vector >')
v = avro::snapshot(*bin_os.get());
^
....
It seems in the latest stable avro library (v1.9.0) auto_ptr have been replaced with unique_ptr & std::move.
Commits on Dec 4, 2018
AVRO-1542 replacing auto_ptr by unique_ptr & std::move
apache/avro@0e6df48#diff-7c029b447b91573980d87596d48331be
The text was updated successfully, but these errors were encountered: