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 1opybp-0003Qh-NL for pgsql-hackers@arkaria.postgresql.org; Tue, 01 Nov 2022 21:15:45 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1opybo-0002MK-Cr for pgsql-hackers@arkaria.postgresql.org; Tue, 01 Nov 2022 21:15:44 +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 1opybo-0002MA-1N for pgsql-hackers@lists.postgresql.org; Tue, 01 Nov 2022 21:15:44 +0000 Received: from mail-qt1-x82a.google.com ([2607:f8b0:4864:20::82a]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1opybl-0000t8-P2 for pgsql-hackers@postgresql.org; Tue, 01 Nov 2022 21:15:43 +0000 Received: by mail-qt1-x82a.google.com with SMTP id c15so3208200qtw.8 for ; Tue, 01 Nov 2022 14:15:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=timescale.com; s=google; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=uSkpxKdEMfYmynqTCSCmzQUBFxc4V/1ro+I8/XgB3fo=; b=Mcdhyj8NsTdhk8De/G3srj7pYZ/9Rt5jyJymm4iVrDfrEj2XXiuPqQj36E5gRZsVK7 AakmXgpt33B9KiIYFIKJeno+0mRBO9sS9ucRaDDwZJfa7gTtYfAzhUImD11tq7xdBIVL BVj6E0STmsLsB2doe7TysCdPihQk+9emFgOjlzuF2Dh9RAAtMqJPwQAnxlx7OSGgEFrn tzldXa+1gKlnzdhRoZRmXcE3gpjl2e/q62jaxvMY8z1itUQjuhIyHADH+zppJB6z4mZd SSStYKXU+2Vh+L3tnOVnFwsmVZRyt3Mna1OkRCKWZialNSYJVeMuO5qqK+3I/TvC1qpk ZABQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=uSkpxKdEMfYmynqTCSCmzQUBFxc4V/1ro+I8/XgB3fo=; b=gyYJ7juMzctzNBQM3RR0IlnziMSmJupqBlpoB+KsBxisxGNFLBhMXQAys85UwsgPtd U3FRuCw4wvFBVn26Q9DDB6g5YwrK51pBI51tvekQhnKzCIoIbxhshS6noLuUOzEjMo/f d/2EcO7qeX/tNcT8XfefzCtkeT5TF/xOF4uN9SUIHdQzcfxnkNYm4d2bESaKRo9fvCZS XtUCHHoI2MkXIDW0iiuRdvj2hp7U4j1dH0kZiL1MJGmfCUWktKZLr7YidtP2OgRGTd+H ZcRCs1LLP+WUVz5bdHRGm/TCcWLuexEy2SO1YyXhhqc9HZ8I9qPE3O2Iw2oWM9KxuoL2 066w== X-Gm-Message-State: ACrzQf0x87rq7nJRYTtn2hKYzQ0iQoCcve3sQFMEvwu2YjrP5aRozVKL CePFbFE4KhtV/EwWKtUSUJkrhnLYNrI8gCTUWfQPuw== X-Google-Smtp-Source: AMsMyM76uHjcoJZ1WzE+/q6sTe6cGON0iWCw6d+MHJC4/7V1WpWGuOKdQLJ1IW6b8ylKwhtpnkxSSpWcvT+WHRwe9Jc= X-Received: by 2002:ac8:5348:0:b0:3a5:4193:e7a2 with SMTP id d8-20020ac85348000000b003a54193e7a2mr1491084qto.630.1667337339142; Tue, 01 Nov 2022 14:15:39 -0700 (PDT) MIME-Version: 1.0 References: <20220802153704.lhrqpqdrrlpandh7@awork3.anarazel.de> In-Reply-To: From: Aleksander Alekseev Date: Wed, 2 Nov 2022 00:15:27 +0300 Message-ID: Subject: Re: Pluggable toaster To: Nikita Malakhov Cc: pgsql-hackers , Jacob Champion , Andres Freund , Matthias van de Meent , Robert Haas , Greg Stark , Teodor Sigaev Content-Type: multipart/mixed; boundary="000000000000d2c7c805ec6f39f3" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk --000000000000d2c7c805ec6f39f3 Content-Type: text/plain; charset="UTF-8" Hi Nikita, > Please don't forget to run `pgindent` before formatting the patches > with `git format-patch` next time. There are also some compiler warnings, please see the attachment. > I'm going to submit a more detailed code review soon. -- Best regards, Aleksander Alekseev --000000000000d2c7c805ec6f39f3 Content-Type: text/plain; charset="UTF-8"; name="warnings.txt" Content-Disposition: attachment; filename="warnings.txt" Content-Transfer-Encoding: base64 Content-ID: X-Attachment-Id: f_l9ypl93k0 L2hvbWUvZWF4L3Byb2plY3RzL3Bnc2NyaXB0cy8uLi9wb3N0Z3Jlc3FsL3NyYy9iYWNrZW5kL2Fj Y2Vzcy9oZWFwL2hlYXB0b2FzdC5jOiBJbiBmdW5jdGlvbiDigJhoZWFwX2NvbXB1dGVfZGF0YV9z aXplX3dpdGhvdXRfYXR0cuKAmToKL2hvbWUvZWF4L3Byb2plY3RzL3Bnc2NyaXB0cy8uLi9wb3N0 Z3Jlc3FsL3NyYy9iYWNrZW5kL2FjY2Vzcy9oZWFwL2hlYXB0b2FzdC5jOjkwOjE5OiB3YXJuaW5n OiDigJh0bXDigJkgbWF5IGJlIHVzZWQgdW5pbml0aWFsaXplZCBbLVdtYXliZS11bmluaXRpYWxp emVkXQogICA5MCB8ICAgICAgICAgKnB2YWx1ZSA9IFBvaW50ZXJHZXREYXR1bSgmdG1wKTsKICAg ICAgfCAgICAgICAgICAgICAgICAgICBefn5+fn5+fn5+fn5+fn5+fn5+fn4KSW4gZmlsZSBpbmNs dWRlZCBmcm9tIC9ob21lL2VheC9wcm9qZWN0cy9wZ3NjcmlwdHMvLi4vcG9zdGdyZXNxbC9zcmMv YmFja2VuZC9hY2Nlc3MvaGVhcC9oZWFwdG9hc3QuYzoyNToKL2hvbWUvZWF4L3Byb2plY3RzL3Bn c2NyaXB0cy8uLi9wb3N0Z3Jlc3FsL3NyYy9pbmNsdWRlL3Bvc3RncmVzLmg6NzM1OjE6IG5vdGU6 IGJ5IGFyZ3VtZW50IDEgb2YgdHlwZSDigJhjb25zdCB2b2lkICrigJkgdG8g4oCYUG9pbnRlckdl dERhdHVt4oCZIGRlY2xhcmVkIGhlcmUKICA3MzUgfCBQb2ludGVyR2V0RGF0dW0oY29uc3Qgdm9p ZCAqWCkKICAgICAgfCBefn5+fn5+fn5+fn5+fn4KL2hvbWUvZWF4L3Byb2plY3RzL3Bnc2NyaXB0 cy8uLi9wb3N0Z3Jlc3FsL3NyYy9iYWNrZW5kL2FjY2Vzcy9oZWFwL2hlYXB0b2FzdC5jOjg1OjI0 OiBub3RlOiDigJh0bXDigJkgZGVjbGFyZWQgaGVyZQogICA4NSB8ICAgICAgICAgc3RydWN0IHZh cmxlbmEgdG1wOwogICAgICB8ICAgICAgICAgICAgICAgICAgICAgICAgXn5+ --000000000000d2c7c805ec6f39f3--