[Kde-pim] Review Request: Increase the column size for the queryString to 32768

Christian Mollekopf chrigi_1 at fastmail.fm
Thu May 10 00:51:00 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104902/
-----------------------------------------------------------

Review request for Akonadi and Volker Krause.


Description
-------

Make the query string column wide enough for Sparql queries.
    
Ideally we would be using BLOB, but that is a bit of a pain to add (would require to alter the whole mapping of Qt-Datatypes, or use some magic value), and 32000 should be big enough for any sane query.
As an extra there is now a safety check for oversized queries, so we don't get truncated queries executed.


Diffs
-----

  server/src/search/nepomuksearchengine.cpp 341c2d9be770896ba6c4739d59be6e4a1a87a17c 
  server/src/search/searchmanager.cpp 334edf5ed14c580a3e4045b2af241c1c0f627937 
  server/src/storage/akonadidb.xml 7d9e089ae3bce20690ab50485f0649d8c2da32eb 
  server/src/storage/dbupdate.xml 7991de8d4d7e0d134bdafb4637aa84e1e7238649 
  server/tests/unittest/dbinitializertest_data/mysql_ct_collection c1ec3a45fd43207664e6b8412365d045ac99c17b 
  server/tests/unittest/dbinitializertest_data/odbc_ct_collection 30e52b9eaddf3dd8d1d23005a7b1df83d78fe628 
  server/tests/unittest/dbupdatertest.cpp 1cdc04ff6b885330f785c0352d2b443569338838 

Diff: http://git.reviewboard.kde.org/r/104902/diff/


Testing
-------

unittests, tried it with kmail.


Thanks,

Christian Mollekopf

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list