Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1m2p4x-00022p-VB for pgsql-hackers@arkaria.postgresql.org; Mon, 12 Jul 2021 06:06:08 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1m2p4w-00088x-Vz for pgsql-hackers@arkaria.postgresql.org; Mon, 12 Jul 2021 06:06:06 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1m2p4w-00088p-MA for pgsql-hackers@lists.postgresql.org; Mon, 12 Jul 2021 06:06:06 +0000 Received: from mail-lj1-x22b.google.com ([2a00:1450:4864:20::22b]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1m2p4u-0007On-7P for pgsql-hackers@postgresql.org; Mon, 12 Jul 2021 06:06:06 +0000 Received: by mail-lj1-x22b.google.com with SMTP id s18so22281983ljg.7 for ; Sun, 11 Jul 2021 23:06:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding:content-language; bh=hphD7SMnd9rhQLJWMKiO/S1gzxsJQW9xJoGgKs5yloY=; b=JVvoDO/syKYoaI71e/ClPtCMPqJjp/cToktJ5sKgQQVbnHvyrGoyS9osLJ3tmvgKNZ sGtl4lk4sD5EQdPEC3rR23y7N/9F7t/ZEsdny5VWhMBtQnHnHvb/3IgtxP7O4I3CcNCw wsgEW2PjwyLjNqaDeM/aTR/CMpmGD6UPe2N6ZEGtG/F4Nqo7/FM/XSvMByNn0eToTj7/ qIxCLFWLSqr+E6yCOE2ty5+L0hwM83ANcstFdDNaKdkO/zmY1vM+/h7N4moq4oXIqEHy R7V2JXa+3PXrUBcXBAc5yBF7fXNSy6r+BjQ9xi4/1MJ6um5SfyrRmcO2CFkDxagK6j9K HDwA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding :content-language; bh=hphD7SMnd9rhQLJWMKiO/S1gzxsJQW9xJoGgKs5yloY=; b=thEl/z8hin1DxZpoTU+5vxVptvqwd0yOk604Ofw223VAveAsFZm65g5NREgWzem+Fc o2xRYA+Hv0HxNhdhO6HNVfRYMArsmSMI56Pudw+ZkesqEU9eMcVTbG+FWU3Cp/Ex5UVB uDwGMdambwPz4osJOsVIqBUvYQGt8EKUbZi/BauR6PVRGKcZHDEMd1NFbzK0PbZhAVmF cMgNz587FtelbQtozNi3QkYYhULk+fFNTdQ9ULRnITwtySNrTlSwLfNGmDuVIRrAcLyX gYVh28Rfsv7WPcuACYMEffuhCVUOtYRH9oxlEFm+vxiB8Ee6ouk0ujkFxWtPphJg+vrr 0tAw== X-Gm-Message-State: AOAM533VpDRw9D1iCIdcyQOVeYT6ZDOS5cHSh6LLglyDExc5/ooXmBCS ki9AibB5FYi1tzPBT814rcY= X-Google-Smtp-Source: ABdhPJyF4V9yIqTkoojo0ZAI0heuk7eq7upaEVg1igOz4E7pgbG+feHwq09zlVpT1paG+10fYofimA== X-Received: by 2002:a2e:b5a3:: with SMTP id f3mr30665221ljn.406.1626069963063; Sun, 11 Jul 2021 23:06:03 -0700 (PDT) Received: from [1.0.0.7] ([178.155.6.217]) by smtp.gmail.com with ESMTPSA id y7sm338417lfg.256.2021.07.11.23.06.02 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 11 Jul 2021 23:06:02 -0700 (PDT) Subject: Re: More time spending with "delete pending" To: Michael Paquier Cc: pgsql-hackers , Tom Lane , Justin Pryzby References: <20729794-e2bf-bd7a-f01b-d132f89fd0d7@gmail.com> <78fa6a17-8e71-7c99-62bd-8a11a47fe894@gmail.com> <2aa7ce7a-d73a-fa36-4fb5-3affe1860c58@gmail.com> From: Alexander Lakhin Message-ID: Date: Mon, 12 Jul 2021 09:06:01 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Content-Language: ru-RU List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Hello Michael, 12.07.2021 08:52, Michael Paquier wrote: > On Mon, Jul 12, 2021 at 02:09:41PM +0900, Michael Paquier wrote: >> And fairywren, that uses MinGW, is unhappy: >> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=fairywren&dt=2021-07-12%2004%3A47%3A13 >> Looking at it now. > I am not sure what to do here to cool down MinGW, so the patch has > been reverted for now: > - Plain stat() is not enough to do a proper detection of the files > pending for deletion on MSVC, so reverting only the part with > microsoft_native_stat() is not going to help. > - We are going to need an evaluation of the stat() implementation in > MinGW and check if is it possible to rely on it more. If yes, we > could get away with more tweaks based on __MINGW32__/__MINGW64__. I'll take care of it and will prepare mingw-compatible patch tomorrow. Best regards, Alexander