Re: [Proposal] Global temporary tables
От | 曾文旌 |
---|---|
Тема | Re: [Proposal] Global temporary tables |
Дата | |
Msg-id | 082FCFA7-B950-4D84-A9CC-E12E0DF75D7F@alibaba-inc.com обсуждение исходный текст |
Ответ на | Re: [Proposal] Global temporary tables (Robert Haas <robertmhaas@gmail.com>) |
Список | pgsql-hackers |
> 2020年7月31日 上午4:57,Robert Haas <robertmhaas@gmail.com> 写道: > > On Thu, Jul 30, 2020 at 8:09 AM wenjing zeng <wjzeng2012@gmail.com> wrote: >> Please continue to review the code. > > This patch is pretty light on comments. Many of the new functions have > no header comments, for example. There are comments here and there in > the body of the new functions that are added, and in places where > existing code is changed there are comments here and there, but > overall it's not a whole lot. There's no documentation and no README, > either. Since this adds a new feature and a bunch of new SQL-callable > functions that interact with that feature, the feature itself should > be documented, along with its limitations and the new SQL-callable > functions that interact with it. I think there should be either a > lengthy comment in some suitable file, or maybe various comments in > various files, or else a README file, that clearly sets out the major > design principles behind the patch, and explaining also what that > means in terms of features and limitations. Without that, it's really > hard for anyone to jump into reviewing this code, and it will be hard > for people who have to maintain it in the future to understand it, > either. Or for users, for that matter. Your suggestion is to the point. I do lack a lot of comments, as is necessary. I'll do this. Wenjing > > -- > Robert Haas > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company
Вложения
В списке pgsql-hackers по дате отправления: