Re: pgbench - allow to create partitioned tables
От | Fabien COELHO |
---|---|
Тема | Re: pgbench - allow to create partitioned tables |
Дата | |
Msg-id | alpine.DEB.2.21.1909262244440.7473@lancre обсуждение исходный текст |
Ответ на | Re: pgbench - allow to create partitioned tables (Alvaro Herrera <alvherre@2ndquadrant.com>) |
Ответы |
Re: pgbench - allow to create partitioned tables
|
Список | pgsql-hackers |
Hello Alvaro, > pgbench's main() is overly long already, and the new code being added > seems to pollute it even more. Can we split it out into a static > function that gets placed, say, just below disconnect_all() or maybe > after runInitSteps? I agree that refactoring is a good idea, but I do not think it belongs to this patch. The file is pretty long too, probably some functions could be moved to distinct files (eg expression evaluation, variable management, ...). > (Also, we seem to be afraid of function prototypes. Why not move the > append_fillfactor() to *below* the functions that use it?) Because we avoid one more line for the function prototype? I try to put functions in def/use order if possible, especially for small functions like this one. -- Fabien.
В списке pgsql-hackers по дате отправления: