modify error report in mdwrite/mdextend

Поиск
Список
Период
Сортировка
От 蔡梦娟(玊于)
Тема modify error report in mdwrite/mdextend
Дата
Msg-id 408346bc-04f2-48fe-a663-d0af2ba49190.mengjuan.cmj@alibaba-inc.com
обсуждение исходный текст
Ответы Re: modify error report in mdwrite/mdextend
Список pgsql-hackers

Hi, all
I noticed that the "else" is missing during the error report after FileWrite() of mdwrite()/mdextend(), short write error is supposed to be reported when written bytes is not less than 0.
I modified it in the attached patch:
diff --git a/src/backend/storage/smgr/md.c b/src/backend/storage/smgr/md.c
index b4bca7eed6..dd60479b65 100644
--- a/src/backend/storage/smgr/md.c
+++ b/src/backend/storage/smgr/md.c
@@ -450,13 +450,14 @@ mdextend(SMgrRelation reln, ForkNumber forknum, BlockNumber blocknum,
       errmsg("could not extend file \"%s\": %m",
        FilePathName(v->mdfd_vfd)),
       errhint("Check free disk space.")));
-  /* short write: complain appropriately */
-  ereport(ERROR,
-    (errcode(ERRCODE_DISK_FULL),
-     errmsg("could not extend file \"%s\": wrote only %d of %d bytes at block %u",
-      FilePathName(v->mdfd_vfd),
-      nbytes, BLCKSZ, blocknum),
-     errhint("Check free disk space.")));
+  else
+   /* short write: complain appropriately */
+   ereport(ERROR,
+     (errcode(ERRCODE_DISK_FULL),
+      errmsg("could not extend file \"%s\": wrote only %d of %d bytes at block %u",
+       FilePathName(v->mdfd_vfd),
+       nbytes, BLCKSZ, blocknum),
+      errhint("Check free disk space.")));
  }

Does this match your previous expectations? Hope to get your reply.
Thanks & Best Regard

Вложения

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

Предыдущее
От: Japin Li
Дата:
Сообщение: Re: [Bug] Logical Replication failing if the DateStyle is different in Publisher & Subscriber
Следующее
От: Bharath Rupireddy
Дата:
Сообщение: Re: modify error report in mdwrite/mdextend