On Thu, May 31, 2018 at 11:50 AM, Daniel Gustafsson <dan...@yesql.se> wrote: > When reading the JIT developer documentation, a few small wordsmithing issues > stood out (although this may be due to me not being a native english speaker). > The attached patch fixes these to what I think the sentences inteded to say.
Committed with a few adjustments. I found that "an SQL..." was about as common as "a SQL..." in the documentation. Either can be correct -- it depends on how you pronounce "SQL". I didn't see any point in those changes, perhaps because I always say "S-Q-L". I'm also pretty sure that Andres did in fact mean "...even for faster queries", since an LRU cache of JIT functions should be particularly useful for OLTP queries that are already individually fast enough to make per-execution JIT compilation overhead prohibitively expensive. That change was also left out. If I'm mistaken in how I interpreted the sentence, then Andres should follow up. Thanks -- Peter Geoghegan