+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 >