[Tarantool-patches] [PATCH v3 3/5] sql: introduce SET statement
Vladislav Shpilevoy
v.shpilevoy at tarantool.org
Thu Nov 28 02:07:34 MSK 2019
No, stop. It is not LGTM.
1) Why on the branch I see a commit
"sql: refactor PRAGMA-related code"
But don't see it in this thread?
2) In a previous email I wrote this:
- Parts of the message before '@Tarantoolbot document' and
after are in some ideas different, in some they are totally
the same. I propose you to remove the part before doc
request, and write the doc request more accurate. And check
again that all deleted PRAGMAs really don't work (in the last
patch), all new SETs really work, with exactly the same names
as in the commit message;
This is still actual. You wrote lots of text before docrequest,
and just a few lines after. The doc team will see the small part.
Once again, I propose you to drop the part before doc request,
and write everything after @Tarantoolbot document.
More information about the Tarantool-patches
mailing list