Hi pá 11. 9. 2020 v 17:00 odesílatel 曾文旌 <wenjing....@alibaba-inc.com> napsal:
> I have written the README for the GTT, which contains the GTT requirements > and design. > I found that compared to my first email a year ago, many GTT Limitations > are now gone. > Now, I'm adding comments to some of the necessary functions. > There are problems with patching. Please, can you rebase your patch? Regards Pavel > > Wenjing > > > > > > > 2020年7月31日 上午4:57,Robert Haas <robertmh...@gmail.com> 写道: > > > > On Thu, Jul 30, 2020 at 8:09 AM wenjing zeng <wjzeng2...@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. > > > > -- > > Robert Haas > > EnterpriseDB: http://www.enterprisedb.com > > The Enterprise PostgreSQL Company > >