Hi, Jose,

Thanks for taking the initiative to build a python interface for 
FastBit.  Since the C API of FastBit has not been touched for a long 
while, there are definitely room for some updates and we would be 
happy to hear your suggestions.

On the issue of handling string values, the main function to be added 
would be a function to retrieve the string values, likely something 
named fastbit_get_qualified_strings, with the following prototype

const char** fastbit_get_qualified_strings(FastBitQueryHandle  query, 
const char *cname);

For row-wise data accesses, there is already a function named 
fastbit_result_set_get_string.  It should work for any string valued 
columns.

Do you think this might be enough?  Would you have a different suggestion?

currently, the pointers returned by the fastbit_get_qualified_ttts and 
fastbit_result_set_get_string are NOT to be deleted by the user code. 
  Instead, the user must call fastbit_destroy_query in order to free 
the memory allocated for those pointers.  Does this work OK for 
python?  Any comments or suggestions to improve this?

John

On 2/26/2010 8:11 AM, Jose Nazario wrote:
> i know that this has been discussed before but i hadn't seen anyone
> provide them. thanks to Brent Pedersen for some key bugfixes and
> contributions, python bindings to FastBit are now in development (e.g.
> working but no release yet):
>
>       http://code.google.com/p/pyfastbit/
>
> built against fastbit-ibis1.1.6 and cython 0.12.1.
>
> they are based on the C API. their status:
>
>       - writes work
>       - queries work
>       - there's a thread lock in the destructors which causes a hang
>
> feedback and improvements most welcome. specifically my C++ skills are
> non-existent, but i'd like to add support for text columns (category,
> blob, text etc) to the C API and thus the Py interface.
>
> thanks to all of the developers and contributors for fastbit, i hope this
> contribution helps.
>
> ________
> jose nazario, ph.d.               http://monkey.org/~jose/
> _______________________________________________
> FastBit-users mailing list
> [email protected]
> https://hpcrdm.lbl.gov/cgi-bin/mailman/listinfo/fastbit-users
_______________________________________________
FastBit-users mailing list
[email protected]
https://hpcrdm.lbl.gov/cgi-bin/mailman/listinfo/fastbit-users

Reply via email to