[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