Re: [VOTE] Changing COPY FROM syntax

[Mike Carey]

+1 for this change!  It will provide a superior user experience for sure...


On 10/24/23 11:12 AM, Ian Maxon wrote:

  +1, looks more consistent with other statements to me.

On Oct 24, 2023 at 10:16:48, Wail Alkowaileet<wael....@gmail.com>  wrote:

Hi all,

I'm proposing to change the current syntax for COPY FROM. The current
syntax looks as follows:

COPY Customers

USING localfs (

   ("path"="asterix_nc1://data/nontagged/customerData.json"),

   ("format"="json")

);



This syntax uses the old way of configuring the adapter localfs. In our
feeds, we use the WITH clause. Another issue is that the current syntax is
missing the keyword FROM, which makes it ambiguous if we add support for
COPY TO.

I propose to change the syntax to be as follows:

COPY Customers

FROM localfs

PATH ("asterix_nc1://data/nontagged/customerData.json")

WITH {

     "format": "json"

};



First, the proposed syntax introduces the use of FROM <adapterName>.
Second, it mandates the use of PATH (instead of having it in the WITH
clause). Additionally, the proposed syntax will make both COPY FROM and
COPY TO less different.

Example of COPY TO:

COPY Customers

TO localfs

PATH("localhost:///myData/Customers")

WITH {

     "format" : "json"

};


--

*Regards,*
Wail Alkowaileet