Re: [pgAdmin][GI-5308]: Cannot set a data field "default", makes it impossible to copy tuples with PK or SERIAL

Hi,

The patch looks good to me.

On Wed, Sep 21, 2022 at 12:35 PM Nikhil Mohite <
nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:

> Hi Hackers,
>
> PFA updated the patch.
>
> On Wed, Sep 21, 2022 at 12:14 PM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Nikhil,
>>
>> I think this fix is incomplete. This is not particularly for PK. Non-PK
>> columns can also have default values.
>> To cover all, the solution should be if a column which has default value
>> is made empty then use default and not null.
>>
>> On Wed, Sep 21, 2022 at 11:10 AM Nikhil Mohite <
>> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Please find attached the patch for GI-5308
>>> <https://github.com/pgadmin-org/pgadmin4/issues/5308>: Cannot set a
>>> data field "default", makes it impossible to copy tuples with PK or SERIAL
>>>
>>>
>>> --
>>> *Thanks & Regards,*
>>> *Nikhil Mohite*
>>> *Senior Software Engineer.*
>>> *EDB Postgres* <https://www.enterprisedb.com/>
>>> *Mob.No: +91-7798364578.*
>>>
>>
>>
>> --
>> Thanks,
>> Aditya Toshniwal
>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>> <http://edbpostgres.com>
>> "Don't Complain about Heat, Plant a TREE"
>>
>
> Regards,
> Nikhil Mohite
>

--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | *edbpostgres.com*
<http://edbpostgres.com>
"Don't Complain about Heat, Plant a TREE"