+1 this is much nicer
On 2023/10/26 05:05:01 Mike Carey wrote:
> PS - I assume the semantics will be UPSERT-based? (Vs. one-time or
> INSERT-based?)
>
> On 10/24/23 10:16 AM, Wail Alkowaileet 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"
> >> };
> >>
