We talked. The conclusion was that it's OK to change the interface, as long as it works properly. Chen Luo mentioned that the new interface may not work correctly.
On Mon, Apr 8, 2019 at 1:32 PM Till Westmann <[email protected]> wrote: > Just one request: please bring the results of your f2f meeting back to > the list. > > Thanks, > Till > > On 8 Apr 2019, at 9:20, Chen Li wrote: > > > Qiushi and Chen Luo: please do a F2F meeting to discuss this issue and > > make > > a plan. Please include me in the meeting. > > > > On Sun, Apr 7, 2019 at 11:05 PM Till Westmann <[email protected]> > > wrote: > > > >> Hi Chen, > >> > >> We could revert the change, but I'd prefer to fix the issues that we > >> have in > >> the (incompletely) documented API. > >> I think that the solution for multiple statements should be the > >> "multi-statement" parameter. However I thought that the default value > >> for this > >> parameter was "true", so I'm not sure what causes the additional > >> statements to > >> be ignored. > >> And I have no idea, why double quotes would be ignored by any parser. > >> Is > >> this > >> the SQL++ parser that causes problems? > >> > >> I'd like to get this resolved (and the docs updated to the point > >> where > >> they are > >> useful) as soon as possible - we clearly would like to keep > >> Cloudberry > >> up and > >> running. > >> > >> Could you send an example (or file an issue) that shows the problem? > >> > >> Cheers, > >> Till > >> > >> On 6 Apr 2019, at 16:18, Chen Luo wrote: > >> > >>> Hi devs, > >>> > >>> I noticed there is a recent change on master that removes the > >>> undocumented > >>> ability to use the request body as the statement [1]. This patch > >>> breaks > >>> many of my experiment scripts and *many data preparation scripts > >>> used > >>> by > >>> Cloudberry.* Also, I had a hard time to modify my scripts to use the > >>> "statement" parameter for two difficulties: > >>> > >>> 1. It seems that only the first statement is executed but the > >>> rest > >>> are > >>> simply ignored; > >>> 2. Double quotes are always ignored by the parser. > >>> > >>> Can this patch be reverted? If not, can we at least update our wiki > >>> [2][3] > >>> to give examples about multiple queries and handling double quotes? > >>> > >>> Best regards, > >>> Chen Luo > >>> > >>> [1] https://asterix-gerrit.ics.uci.edu/#/c/3267/ > >>> [2] > >>> > >> > https://cwiki.apache.org/confluence/display/ASTERIXDB/New+HTTP+API+Design > >>> [3] https://ci.apache.org/projects/asterixdb/api.html > >> >
