On Mon, 08 Mar 2021 at 12:28, Bharath Rupireddy <bharath.rupireddyforpostg...@gmail.com> wrote: > On Sun, Mar 7, 2021 at 10:13 PM Zhihong Yu <z...@yugabyte.com> wrote: >> Hi, >> >> + * EXPLAIN ANALYZE CREATE TABLE AS or REFRESH MATERIALIZED VIEW >> + * WITH NO DATA is weird. >> >> Maybe it is clearer to spell out WITH NO DATA for both statements, instead >> of sharing it. > > Done that way. > >> - if (!stmt->skipData) >> + if (!stmt->skipData && !explainInfo) >> ... >> + else if (explainInfo) >> >> It would be cleaner to put the 'if (explainInfo)' as the first check. That >> way, the check for skipData can be simplified. > > Changed. > > Thanks for review comments. Attaching v7 patch set with changes only > in 0002 patch. Please have a look. >
The v7 patch looks good to me, and there is no other advice, so I change the status to "Ready for Committer". -- Regrads, Japin Li. ChengDu WenWu Information Technology Co.,Ltd.