Remove extra spaces

2022-01-03 Thread Suraj Kharage
Hi, While browsing the code, noticed the extra spaces after the function name. Removed the same in the attached patch. -- -- Thanks & Regards, Suraj kharage, edbpostgres.com remove_space_from_numeric_function.patch Description: Binary data

extra semicolon in postgres_fdw test cases

2021-03-30 Thread Suraj Kharage
Hi, Noticed that an extra semicolon in a couple of test cases related to postgres_fdw. Removed that in the attached patch. It can be backported till v11 where we added those test cases. -- -- Thanks & Regards, Suraj kharage, edbpostgres.com remove_extra_semicolon_postgres_fdw.p

Re: CREATE SEQUENCE with RESTART option

2021-04-08 Thread Suraj Kharage
err_detail along with the error > message? > Patch looks good to me. Current error message looks ok to me. Do we need to add double quotes for RESTART word in the error message since it is an option? -- -- Thanks & Regards, Suraj kharage, edbpostgres.com

identity column behavior in WHEN condition for BEFORE EACH ROW trigger

2019-10-03 Thread Suraj Kharage
78049=#select * from foo; no | id +---- 1 | *10* (1 row) Thoughts? -- -- Thanks & Regards, Suraj kharage, EnterpriseDB Corporation, The Postgres Database Company.

Re: identity column behavior in WHEN condition for BEFORE EACH ROW trigger

2019-10-06 Thread Suraj Kharage
, I think it is breaking the OVERRIDING SYSTEM VALUE clause in INSERT statement. i.e: without this clause, can insert the modified value from trigger in identity column. I don't find any document reference for this behavior. Thoughts? -- -- Thanks & Regards, Suraj kharage, EnterpriseDB C

Re: pg_dump/pg_restore fail for TAR_DUMP and CUSTOM_DUMP from v94/v95/v96 to v11/master.

2019-03-01 Thread Suraj Kharage
Hi, The Commit 5955d934194c3888f30318209ade71b53d29777f has changed the logic to avoid dumping creation and comment commands for the public schema. >From v11 onwards, we are using the DUMP_COMPONENT_ infrastructure in selectDumpableNamespace() to skip the public schema creation. As reported by Pr

Issue in to_timestamp/to_date while handling the quoted literal string

2019-07-24 Thread Suraj Kharage
rong hour value. My suggestion is to match the exact characters from quoted literal string in input string and if doesn't match then throw an error. Attached is the POC patch which almost works for all scenarios except for whitespace - as a quote character. Suggestions? -- -- Thanks & R

Re: Issue in to_timestamp/to_date while handling the quoted literal string

2019-07-24 Thread Suraj Kharage
ormat that > is known to you in advance. > > I hope that helps. > > Cheers > Brendan > > On Wed, 24 Jul 2019 at 21:38, Suraj Kharage < > suraj.khar...@enterprisedb.com> wrote: > >> Hi, >> >> I noticed the issue in to_timestamp()/to_date() while hand

Re: backup manifests

2019-11-19 Thread Suraj Kharage
we >> can't detect as things stand? I think the only case is an empty >> directory. If it's not empty, we'd have some entries for the files in >> that directory, and those files won't be able to exist unless the >> directory does. But,

Re: backup manifests

2019-11-25 Thread Suraj Kharage
Hi Jeevan, I have incorporated all the comments in the attached patch. Please review and let me know your thoughts. On Thu, Nov 21, 2019 at 2:51 PM Jeevan Chalke < jeevan.cha...@enterprisedb.com> wrote: > > > On Wed, Nov 20, 2019 at 11:05 AM Suraj Kharage < > suraj.kha

Re: backup manifests

2019-12-10 Thread Suraj Kharage
backup by changing the contents of a file but > not the size, the modification is detected if checksums are used. > - Everything above still works if there is user-defined tablespace > that contains a table. > > -- > Robert Haas > EnterpriseDB: http://www.enterprisedb.com > The Enter

Re: backup manifests

2019-12-12 Thread Suraj Kharage
Thanks, Robert for the review. On Wed, Dec 11, 2019 at 1:10 AM Robert Haas wrote: > On Tue, Dec 10, 2019 at 6:40 AM Suraj Kharage > wrote: > > Please find attached patch for backup validator implementation (0004 > patch). This patch is based > > on Rushabh's latest

Re: backup manifests

2019-12-16 Thread Suraj Kharage
nges that you > proposed on the mentioned thread. > > I have implemented the simplehash in backup validator patch as Robert suggested. Please find attached 0002 patch for the same. kindly review and let me know your thoughts. Also attached the remaining patches. 0001 and 0003 are same

Re: backup manifests

2019-12-20 Thread Suraj Kharage
Thank you for review comments. On Thu, Dec 19, 2019 at 2:54 AM Robert Haas wrote: > On Tue, Dec 17, 2019 at 12:54 AM Suraj Kharage > wrote: > > I have implemented the simplehash in backup validator patch as Robert > suggested. Please find attached 0002 patch for the same. > &

Re: backup manifests

2019-12-20 Thread Suraj Kharage
Fixed some typos in attached v5-0002 patch. Please consider this patch for review. On Fri, Dec 20, 2019 at 6:54 PM Suraj Kharage < suraj.khar...@enterprisedb.com> wrote: > Thank you for review comments. > > On Thu, Dec 19, 2019 at 2:54 AM Robert Haas wrote: > >> On Tue,

Re: backup manifests

2019-12-24 Thread Suraj Kharage
Thank you for review comments. On Fri, Dec 20, 2019 at 9:14 PM Robert Haas wrote: > On Fri, Dec 20, 2019 at 8:24 AM Suraj Kharage > wrote: > > Thank you for review comments. > > Thanks for the new version. > > + --verify-backup > > Whitespace. > Corrected

Re: backup manifests

2020-01-03 Thread Suraj Kharage
Thank you for review comments. On Mon, Dec 30, 2019 at 11:53 PM Robert Haas wrote: > On Tue, Dec 24, 2019 at 5:42 AM Suraj Kharage > wrote: > > To examine the first word of each line, I am using below check: > > if (strncmp(line, "File", 4) == 0) > > { >

Re: Server crash on RHEL 9/s390x platform against PG16

2023-10-22 Thread Suraj Kharage
On Sat, Oct 21, 2023 at 5:17 AM Andres Freund wrote: > Hi, > > On 2023-09-12 15:27:21 +0530, Suraj Kharage wrote: > > *[edb@9428da9d2137 postgres]$ cat /etc/redhat-release AlmaLinux release > 9.2 > > (Turquoise Kodkod)[edb@9428da9d2137 postgres]$ lscpuArchitecture:

[Regression] Incorrect filename in test case comment

2023-09-05 Thread Suraj Kharage
cases. Did the same in the attached patch. -- -- Thanks & Regards, Suraj kharage, edbpostgres.com Fix_file_name_in_test_case.patch Description: Binary data

Re: [Regression] Incorrect filename in test case comment

2023-09-06 Thread Suraj Kharage
Thanks Daniel and Michael. On Wed, Sep 6, 2023 at 1:52 PM Daniel Gustafsson wrote: > > On 6 Sep 2023, at 10:19, Michael Paquier wrote: > > > > On Wed, Sep 06, 2023 at 10:48:32AM +0530, Suraj Kharage wrote: > >> While browsing the test cases, found that the inc

Server crash on RHEL 9/s390x platform against PG16

2023-09-12 Thread Suraj Kharage
13fec34 in PostmasterMain (argc=argc@entry=3, argv=argv@entry=0x19a59a0) at postmaster.c:1466 #16 0x01096faa in main (argc=, argv=0x19a59a0) at main.c:198 (gdb) p val $1 = 0 ``` Does anybody have any idea about this? -- -- Thanks & Regards, Suraj kharage, edbpostgres.com

Re: Server crash on RHEL 9/s390x platform against PG16

2023-09-17 Thread Suraj Kharage
nnerslot for merge join, the value for attnum 1 is not getting fetched correctly. On Tue, Sep 12, 2023 at 3:27 PM Suraj Kharage < suraj.khar...@enterprisedb.com> wrote: > Hi, > > Found server crash on RHEL 9/s390x platform with below test case - > > *Machine details:* >

Re: pg_upgrade --check fails to warn about abstime

2023-09-20 Thread Suraj Kharage
aro Herrera PostgreSQL Developer — > https://www.EnterpriseDB.com/ > "We’ve narrowed the problem down to the customer’s pants being in a > situation > of vigorous combustion" (Robert Haas, Postgres expert extraordinaire) > -- -- Thanks & Regards, Suraj kharage, edbpostgres.com

Re: pg_upgrade --check fails to warn about abstime

2023-09-26 Thread Suraj Kharage
lsite and none in the function. It > looked a bit schizoid otherwise. > yeah, looks good to me. > > -- > Álvaro Herrera 48°01'N 7°57'E — > https://www.EnterpriseDB.com/ > "Postgres is bloatware by design: it was built to house > PhD theses." (Joey Hellerstein, SIGMOD annual conference 2002) > -- -- Thanks & Regards, Suraj kharage, edbpostgres.com

Re: Server crash on RHEL 9/s390x platform against PG16

2023-10-08 Thread Suraj Kharage
|1 | 1 | 2 | row2 |2 | visible | f |2 | 1 | (4 rows) Any idea on this? On Mon, Sep 18, 2023 at 11:20 AM Suraj Kharage < suraj.khar...@enterprisedb.com> wrote: > Few more details on this: > > (gdb) p val > $1 = 0 > (gdb) p i >

Re: Server crash on RHEL 9/s390x platform against PG16

2023-10-12 Thread Suraj Kharage
Here is clang version: [edb@9428da9d2137]$ clang --version clang version 15.0.7 (Red Hat 15.0.7-2.el9) Target: s390x-ibm-linux-gnu Thread model: posix InstalledDir: /usr/bin Let me know if any further information is needed. On Mon, Oct 9, 2023 at 8:21 AM Suraj Kharage wrote: > It lo

Re: backup manifests

2020-03-15 Thread Suraj Kharage
27;t think this is committable just yet, but I think it's getting > fairly close, so if anyone has major objections please speak up soon. > > -- > Robert Haas > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > -- -- Thanks & Regards, Suraj kharage, EnterpriseDB Corporation, The Postgres Database Company.

Re: backup manifests

2020-03-15 Thread Suraj Kharage
ouple of very minor bugs which I discovered by writing those >> tests. >> >> - I added documentation, in part based on a draft Mark Dilger shared >> with me off-list. >> >> I don't think this is committable just yet, but I think it's getting >> fairly close, so if anyone has major objections please speak up soon. >> >> -- >> Robert Haas >> EnterpriseDB: http://www.enterprisedb.com >> The Enterprise PostgreSQL Company >> > > > -- > -- > > Thanks & Regards, > Suraj kharage, > EnterpriseDB Corporation, > The Postgres Database Company. > -- -- Thanks & Regards, Suraj kharage, EnterpriseDB Corporation, The Postgres Database Company.

Re: Query about time zone patterns in to_char

2021-05-19 Thread Suraj Kharage
> If there are some others that happen to work, we probably shouldn't > > remove them for fear of breaking poorly-written apps ... but that does > > not imply that we need to support every case-variant. > > > > regards, tom lane > > > -- -- Thanks & Regards, Suraj kharage, edbpostgres.com

Re: [PATCH] improve the pg_upgrade error message

2021-07-12 Thread Suraj Kharage
t; > *"All non-template0 databases must allow connections, i.e. their > pg_database.datallowconn must be true; database "mydb" has datallowconn set > to false."* > > > > Regards, > > Jeevan Ladhe > -- -- Thanks & Regards, Suraj kharage, edbpostgres.com

Re: [PATCH] improve the pg_upgrade error message

2021-07-13 Thread Suraj Kharage
all the databases other than > template0 > that are having their pg_database.datallowconn to false in a file. Similar > approach is seen in other functions like check_for_data_types_usage(), > check_for_data_types_usage() etc. Thanks Suraj Kharage for the offline > suggestion. >

Re: postgres_fdw: wrong results with self join + enable_nestloop off

2023-06-01 Thread Suraj Kharage
ne place in > add_foreign_final_paths that uses NULL for List *. I changed it to NIL. > > Thanks > Richard > -- -- Thanks & Regards, Suraj kharage, edbpostgres.com

Re: refactoring basebackup.c

2020-06-29 Thread Suraj Kharage
8kb which took more time than others might be because of noise at that time. -- -- Thanks & Regards, Suraj kharage, edbpostgres.com

Re: refactoring basebackup.c

2020-05-12 Thread Suraj Kharage
know your thoughts/suggestions on the same. -- -- Thanks & Regards, Suraj kharage, EnterpriseDB Corporation, The Postgres Database Company.

Re: refactoring basebackup.c

2020-05-13 Thread Suraj Kharage
9.774s sys 7m41.032s real 7m26.678s user 0m54.833s sys 6m20.057s real 19m5.218s user 1m44.122s sys 10m17.623s 2nd run real 11m30.500s user 1m45.221s sys 9m37.815s real 9m4.103s user 1m6.893s sys 7m49.393s real 7m26.713s user 0m54.868s sys 6m19.652s real 18m17.230s user 1m42.749s sys 9m53.704s -- --

Catalog views failed to show partitioned table information.

2018-12-14 Thread Suraj Kharage
Hi, There are some catalog views which do not show the partitioned table and its index entry. One of them is "pg_indexes" which failed to show the partitioned index. Attached the patch which fixes the same. Other views such as pg_stat*,pg_statio_* has the same problem for partitioned tables and i

Re: Catalog views failed to show partitioned table information.

2018-12-18 Thread Suraj Kharage
ng close to what Suraj is proposing if there are > > no objections from others. At least we agree on that part ;) > > And this part is done. > -- > Michael > -- -- Thanks & Regards, Suraj kharage, EnterpriseDB Corporation, The Postgres Database Company. *Are you updated: La

Re: allow frontend use of the backend's core hashing functions

2020-02-13 Thread Suraj Kharage
re. > > Thanks, > > -- > Robert Haas > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > [1] > http://postgr.es/m/CA+Tgmob8oyh02NrZW=xCScB+5GyJ-jVowE3+TWTUmPF=fsg...@mail.gmail.com > -- -- Thanks & Regards, Suraj kharage, EnterpriseDB Corporation, The Postgres Database Company.

Re: backup manifests

2020-03-04 Thread Suraj Kharage
rds, > Yeah, This handling is missing in the provided WIP patch. I believe Robert will consider this fixing in upcoming version of validator patch. -- -- Thanks & Regards, Suraj kharage, EnterpriseDB Corporation, The Postgres Database Company.

Re: backup manifests

2020-03-04 Thread Suraj Kharage
ave tablespace with no objects (empty tablespace) then backup_manifest does not have any entry for this hence when you remove this tablespace directory, validator could not detect it. We can either document it or add the entry for directories in the manifest. Robert may have a better idea on this.

Re: backup manifests

2020-03-06 Thread Suraj Kharage
with a slash-terminated pathname doesn't work > as expected. > Response: Fixed, thanks for the report. > > Off-List Report: You forgot a PG_BINARY flag. > Response: Fixed. I thought I'd done this before but there were two > places and I'd only fixed one of them. > > -- > Robert Haas > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > -- -- Thanks & Regards, Suraj kharage, EnterpriseDB Corporation, The Postgres Database Company.

Re: Support for NO INHERIT to INHERIT state change with named NOT NULL constraints

2024-11-24 Thread Suraj Kharage
Thanks for the review comments. On Wed, Nov 20, 2024 at 9:13 AM jian he wrote: > On Thu, Nov 14, 2024 at 1:02 PM Suraj Kharage > wrote: > > > > Hi, > > > > Upstream commit 14e87ffa5c543b5f30ead7413084c25f7735039f added the > support for named NOT NULL constra

Re: Support for NO INHERIT to INHERIT state change with named NOT NULL constraints

2024-11-24 Thread Suraj Kharage
-- Thanks & Regards, Suraj kharage, enterprisedb.com <https://www.enterprisedb.com/> On Tue, Nov 19, 2024 at 6:52 PM jian he wrote: > On Fri, Nov 15, 2024 at 11:15 AM Robert Haas > wrote: > > > > On Thu, Nov 14, 2024 at 12:02 AM Suraj Kharage < > sur

Support for NO INHERIT to INHERIT state change with named NOT NULL constraints

2024-11-13 Thread Suraj Kharage
oing to be quite problematic because of the number of weird cases to avoid, so for now that support is not added. Please share your thoughts on the same. -- Thanks & Regards, Suraj kharage, enterprisedb.com <https://www.enterprisedb.com/> diff --git a/doc/src/sgml/ref/alter_tab

Re: Support for NO INHERIT to INHERIT state change with named NOT NULL constraints

2025-02-09 Thread Suraj Kharage
Thanks, Alvaro, for the review. I have addressed your comments per the above suggestions in the attached v4 patch. -- Thanks & Regards, Suraj kharage, enterprisedb.com <https://www.enterprisedb.com/> On Wed, Feb 5, 2025 at 12:11 AM Alvaro Herrera wrote: > On 2025-Jan-13,

Re: Support for NO INHERIT to INHERIT state change with named NOT NULL constraints

2025-01-12 Thread Suraj Kharage
On Wed, Jan 8, 2025 at 2:43 PM Alvaro Herrera wrote: > On 2024-Nov-25, Suraj Kharage wrote: > > > Another case which needs conclusion is - > > When changing from INHERIT to NO INHERIT, we need to walk all children > and > > decrement coninhcount for the corresponding

Re: simplifying grammar for ALTER CONSTRAINT .. SET [NO] INHERIT

2025-03-26 Thread Suraj Kharage
On Tue, Mar 25, 2025 at 9:32 PM Álvaro Herrera wrote: > With commit f4e53e10b6ce we introduced a way to flip the NO INHERIT bit > on not-null constraints. However, because of the way the grammar > dealt with ALTER CONSTRAINT, we were too blind to see a way to implement > it using the existing pr

Re: Support for NO INHERIT to INHERIT state change with named NOT NULL constraints

2025-03-03 Thread Suraj Kharage
Thanks Alvaro for the review and fixup patch. I agree with your changes and merged that into the main patch along with a couple of other changes. Please find attached v6 for further review. -- Thanks & Regards, Suraj kharage, enterprisedb.com <https://www.enterprisedb.com/> On

Re: Support for NO INHERIT to INHERIT state change with named NOT NULL constraints

2025-02-21 Thread Suraj Kharage
Thanks, Alvaro. I have revised the patch as per your last update. Please find attached v5 for further review. -- Thanks & Regards, Suraj kharage, enterprisedb.com <https://www.enterprisedb.com/> On Wed, Feb 19, 2025 at 9:16 PM Alvaro Herrera wrote: > On 2025-Feb-10, Suraj

Replication slot is not able to sync up

2025-05-22 Thread Suraj Kharage
| t| f (2 rows) = Is there any way to sync up the replication slot after the catalog changes have been made after creation? -- Thanks & Regards, Suraj kharage, enterprisedb.com <https://www.enterprisedb.com/>

Potential issue in listExtensions()

2025-06-04 Thread Suraj Kharage
pg_description, so that we retrieve only pg_extension entries from pg_description. Attaching the patch for the same. Thoughts? [1] https://www.postgresql.org/message-id/cabueveytmyxc6ovcwkj+rpnhrfi8_rw_+dd_jzgffnpqgf+...@mail.gmail.com -- Thanks & Regards, Suraj kharage, enterprisedb.com &

Re: Potential issue in listExtensions()

2025-06-05 Thread Suraj Kharage
Thanks, Magnus and Dilip. -- Thanks & Regards, Suraj kharage, enterprisedb.com <https://www.enterprisedb.com/> On Thu, Jun 5, 2025 at 1:34 PM Magnus Hagander wrote: > > > On Thu, Jun 5, 2025 at 9:50 AM Dilip Kumar wrote: > >> On Thu, Jun 5, 2025 at 5:5