pgbench --partitions=0

Поиск
Список
Период
Сортировка
От Amit Langote
Тема pgbench --partitions=0
Дата
Msg-id CA+HiwqGAGobiiHR8nH382HJxqm1mzZs8=3oKPXnXivWoFSZmNA@mail.gmail.com
обсуждение исходный текст
Ответы Re: pgbench --partitions=0  (Michael Paquier <michael@paquier.xyz>)
Список pgsql-hackers
Hi,

I noticed that $subject causes an error in HEAD:

$ pgbench -i --partitions=0
pgbench: error: --partitions must be in range 1..2147483647

However, it works in v13 and v14, assuming no partitions.

I think the commit 6f164e6d17 may have unintentionally broken it, by
introducing this hunk:

@@ -6135,12 +6116,9 @@ main(int argc, char **argv)
                break;
            case 11:            /* partitions */
                initialization_option_set = true;
-               partitions = atoi(optarg);
-               if (partitions < 0)
-               {
-                   pg_log_fatal("invalid number of partitions:
\"%s\"", optarg);
+               if (!option_parse_int(optarg, "--partitions", 1, INT_MAX,
+                                     &partitions))
                    exit(1);
-               }

Attached a patch to fix with a test added.  cc'ing Michael who
authored that commit.

-- 
Thanks, Amit Langote
EDB: http://www.enterprisedb.com

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Amit Kapila
Дата:
Сообщение: Re: list of TransactionIds
Следующее
От: Amit Kapila
Дата:
Сообщение: Re: PostgreSQL 15 Beta 1 release announcement draft