Re: Add new error_action COPY ON_ERROR "log"

On 2024-03-28 21:36, torikoshia wrote:
> On 2024-03-28 17:27, Bharath Rupireddy wrote:
>> On Thu, Mar 28, 2024 at 1:43 PM torikoshia
>> <torikoshia(at)oss(dot)nttdata(dot)com> wrote:
>>>
>>> Attached patch fixes the doc,
>>
>> May I know which patch you are referring to? And, what do you mean by
>> "fixes the doc"?
>
> Ugh, I replied to the wrong thread.
> Sorry for making you confused and please ignore it.
>
>>> but I'm wondering perhaps it might be
>>> better to modify the codes to prohibit abbreviation of the value.
>>
>> Please help me understand the meaning here.
>>
>>> When seeing the query which abbreviates ON_ERROR value, I feel it's
>>> not
>>> obvious what happens compared to other options which tolerates
>>> abbreviation of the value such as FREEZE or HEADER.
>>>
>>> COPY t1 FROM stdin WITH (ON_ERROR);
>>>
>>> What do you think?
>>
>> So, do you mean to prohibit ON_ERROR being specified without any value
>> like in COPY t1 FROM stdin WITH (ON_ERROR);? If yes, I think all the
>> other options do allow that [1].
>>
>> Even if we were to do something like this, shall we discuss this
>> separately?
>>
>> Having said that, what do you think of the v13 patch posted upthread?

It looks good to me other than what Sawada-san lastly pointed out.

--
Regards,

--
Atsushi Torikoshi
NTT DATA Group Corporation