← Back to team overview

maria-developers team mailing list archive

OQGraph?

 

Hi Arjen,

I wanted to see if there was something I could do to help getting
oqgraph-in-mariadb moving again.

As I understand it, the issues are the following:

1. Current OQGraph autoconf test enables OQGraph by default in the build if a
too old Boost is installed on the machine. This means that a plain
./configure && make will not work on such machines. (In other words, OQGraph
must not be enabled by default on machines where it does not compile).

2. OQGraph is affected by a bug in Boost, so it actually requires a patched
Boost (until some Boost version is released with the fix included).

3. Even if OQGraph is included properly into the MariaDB source tree, we still
need to set up package building so that it will actually be able to build and
include OQGraph.

So one possible solution for issue 3 is to include the patched boost headers
inside the storage/oqgraph/ directory. I remember that including all of boost
was infeasible due to the sheer size of the library. But you mentioned that
just the graph part of Boost would do. How big is this subset (number of files
/ number of lines?).

Another solution would be to create an ourdelta package of the patched boost
development headers. We could then make this a build dependency of mariadb
.debs. But not sure if this is too much work to set up (and not sure about how
this would work for .deb either).

A third solution could be to just install the patched boost in
/usr/local/include on the build VMs. This should work, though it's not 100%
nice in terms of providing full source code (you could always release OQGraph
as GPL with a boost linking exception if we're really paranoid).

A fourth solution could be to include the graph part of boost (with bug fix)
as a patch in ourdelta/bakery/, just like the other patches already
there.

For issue 1 + 2, if you can point me to what the bug in boost is I could look
into making an autoconf test for this bug, and only enable OQGraph by default
if a boost without the bug is found. Or failing that, if you can supply me
with the minimum version of boost needed, I could try for an autoconf test
that does not enable OQGraph by defaults on hosts where it breaks the build.

What do you think?

 - Kristian.



Follow ups