sqlite patch mentioned in the paper

Question

sqlite patch mentioned in the paper

albertnetymk opened this issue 5 years ago · comments

Hello, I am quite amazed by the elegance of the fixes for sqlite. I wonder if such patch is accessible in public domain, some mailing list or ticket database. "We have submitted patches to the developers of all the applications we optimized." from the paper.

I went to sqlite ticket database, but can't find anything using the keyword "mutex".

Dmitry Oksenchuk · Answer 1 · Wed Oct 02 2019 23:43:57 GMT+0800 (China Standard Time)

Additional indirection is still there: https://github.com/mackyle/sqlite/blob/a419afd73a544e30df878db55f7faa17790c01bd/src/mutex_unix.c

I would be nice to know what sqlite devs think about the patch.

Albert · Answer 2 · Fri Oct 04 2019 17:59:22 GMT+0800 (China Standard Time)

I am curious as well, and I want to see the patch if accessible anywhere.

Charlie Curtsinger · Answer 3 · Tue Oct 08 2019 04:23:57 GMT+0800 (China Standard Time)

The problem with the SQLite patch is that it removes the option to configure SQLite's synchronization mode at compile time. The SQLite project could change their configuration options to use #ifdefs instead of the struct of function pointers referenced in the paper, but my change was much smaller than what would be required for the core project.

I am about to merge a pull request that includes the other patches.