[Kde-pim] Review Request 112976: Map QString to text instead of Bytea with PostgreSQL backend

[Dan Vrátil]

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



server/src/storage/dbinitializer_p.cpp
<http://git.reviewboard.kde.org/r/112976/#comment30070>

    Could you elaborate the FIXME a bit more? :-)
    
    I remember you said we could/should use QSqlDriver::formatValue() so maybe you could resolve the FIXME right away and replace the code with it.setValue(driver.formatValue(....)) :-)



server/src/storage/dbupdate.xml
<http://git.reviewboard.kde.org/r/112976/#comment30072>

    How long does the conversion take on a reasonable-sized database? If it's over a minute, we need to tell clients we are upgrading the database so that they know why their Akonadi does not work :-)
    
    Reporting of upgrading is currently broken in master. The fix is in parttable-optimization branch though, so we should either merge it (I'm not sure I want to do it yet), or at least cherry-pick the commit, if it's possible (I'll have to check, I did quite some changes to the upgrade mechanism)



server/src/storage/dbupdate.xml
<http://git.reviewboard.kde.org/r/112976/#comment30071>

    This is just a rant, but could you keep the CamelCase convention for table and column names, as used above?


- Dan Vrátil


On Sept. 28, 2013, 1:16 p.m., Cédric  Villemain wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112976/
> -----------------------------------------------------------
> 
> (Updated Sept. 28, 2013, 1:16 p.m.)
> 
> 
> Review request for Akonadi.
> 
> 
> Description
> -------
> 
> It's what suggested by QTSql.
> 
> It also happens to fix the recent issues raising around '\\SEEN' errors in SQL queries. It was visible because QTSql somewhat improve the postgresql driver, and so... akonadi revealed its own bug.
> 
> 
> Diffs
> -----
> 
>   server/src/storage/dbinitializer_p.cpp 55afaa9884df5cc1c3513907cbc3ac23785b0136 
>   server/src/storage/dbupdate.xml 2b1955e3e156b6dbe1a95759be1a6bbedd7c9732 
> 
> Diff: http://git.reviewboard.kde.org/r/112976/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Cédric  Villemain
> 
>

_______________________________________________
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/