Re: Require suggestion for issue #8007

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Pravesh Sharma <pravesh(dot)sharma(at)enterprisedb(dot)com>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Require suggestion for issue #8007
Date: 2024-10-09 08:59:43
Message-ID: CA+OCxoyJnhprqdvVQ0nPOemLNpXghahTgJCaPd3+QYMioM9+nw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, 8 Oct 2024 at 16:30, Pravesh Sharma <pravesh(dot)sharma(at)enterprisedb(dot)com>
wrote:

> Hi Dave,
>
> On Tue, Oct 8, 2024 at 6:34 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>>
>>
>> On Tue, 8 Oct 2024 at 13:56, Pravesh Sharma <
>> pravesh(dot)sharma(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> We have been working on issue #8007
>>> <https://github.com/pgadmin-org/pgadmin4/issues/8007>. After our
>>> analysis, we found that the length and scale of the argument's data type
>>> are not stored in the database server, making it difficult to fetch. In the
>>> debugger, pgAdmin typecasts the provided value to its data type when
>>> calling the function, which is causing the issue.
>>>
>>> If we typecast the value to a character without specifying the length,
>>> it is treated as a character (1) (character of length 1). For example, the
>>> output of SELECT 'pgadmin', 'pgadmin'::character; is different.
>>>
>>> There are two solutions we can consider:
>>>
>>> 1. Do not typecast if the data type is character.
>>> 2. Typecast to varchar when the data type is character.
>>>
>>> Please provide your suggestions which solution looks good to you.
>>>
>>
>> Why not include the length in the cast? We know what the input string is,
>> so can't we just do the equivalent of:
>>
>> 'pgadmin'::character(7);
>>
> Just to confirm, this will be only for the character data type?
>

Yes, if that's the only type with which the issue is seen.

--
Dave Page
pgAdmin: https://www.pgadmin.org
PostgreSQL: https://www.postgresql.org
EDB: https://www.enterprisedb.com

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2024-10-10 12:40:36 Translators: Release next week (v8.13)
Previous Message Pravesh Sharma 2024-10-08 15:30:03 Re: Require suggestion for issue #8007