oqgraph-dev team mailing list archive

[Andrew McDonnell <bugs@xxxxxxxxxxxxxxxxxxx>]

OK I worked out how to do that

You also have to recode from latch back to varchar as well in fill_record()

Will push working prototype shortly

On 05/03/13 07:37, Antony T Curtis wrote:
> Ahh yeah. if you change the type, you must fix the index key decode.
> 
> Antony T Curtis
> On Mar 4, 2013 1:04 PM, "Arjen Lentz" <arjen@xxxxxxxxxxxxx
> <mailto:arjen@xxxxxxxxxxxxx>> wrote:
>>
>> I think we need Antony for this. Antony?
>>
>> Andrew McDonnell <bugs@xxxxxxxxxxxxxxxxxxx
> <mailto:bugs@xxxxxxxxxxxxxxxxxxx>> wrote:
>>
>> >Hi Arjen, Antony
>> >
>> >I have pushed my work so far to
>> >
>> >http://bazaar.launchpad.net/~andymc73/maria/10.0-oqgraph3-varchar
>> >
>> >I thought things were flying along according to plan... then I hit how to
>> >interpret the keys (O_o) which from a mysql internals is a bit beyond my
>> >understanding still.
>> >
>> >
>> >To date, I have:
>> >
>> >* changed the latch to VARCHAR
>> >* retained backwards compatbility (with an #ifdef) allowing latch SHORTINT to
>> >still be created, but reporting a deprecation warning
>> >* added warnings to create() and a better message than 'table crashed' if the
>> >OQGRAPH table created incorrectly.  So the user can go show warnings and see
>> >which column they mucked up.
>> >* implemented string -> int latch parsing in search(), which should work, but
>> >I havent been able to test it as yet, because
>> >
>> >when doing
>> >
>> >SELECT * FROM graph2 WHERE latch = 'breadth_first' AND origid = 1 AND
> weight = 1;
>> >
>> >aka the first select in basic.test, I get an assertion on key store_length
>> >inside ha_oqgraph::records_in_range(); at this point I realised I am a little
>> >stuck, and need some explaining as to how the keys are supposed to work in
>> >this case...
>> >
>> >FWIW the existing basic.test still works with the backward compat #ifdef
>> >turned on.  (Except I cant flush at the end, and I get a server memory leak
>> >assertion thereafter.  But I doubt that is related to my changes so far)
>> >
>> >
>> >
>> >--Andrew
>> >
>> >--
>> >Mailing list: https://launchpad.net/~oqgraph-dev
>> >Post to     : oqgraph-dev@xxxxxxxxxxxxxxxxxxx
> <mailto:oqgraph-dev@xxxxxxxxxxxxxxxxxxx>
>> >Unsubscribe : https://launchpad.net/~oqgraph-dev
>> >More help   : https://help.launchpad.net/ListHelp
>> --
>> Mailing list: https://launchpad.net/~oqgraph-dev
>> Post to     : oqgraph-dev@xxxxxxxxxxxxxxxxxxx
> <mailto:oqgraph-dev@xxxxxxxxxxxxxxxxxxx>
>> Unsubscribe : https://launchpad.net/~oqgraph-dev
>> More help   : https://help.launchpad.net/ListHelp
>