[GitHub] [pinot] Jackie-Jiang commented on pull request #8557: Adding DML definition and parse SQL InsertFile

[GitBox <gi...@apache.org>]

Jackie-Jiang commented on PR #8557:
URL: https://github.com/apache/pinot/pull/8557#issuecomment-1110216143

   > > High level question: should we reuse the current SQL endpoint (HTTP GET), or introduce a new one. How do other systems handle it?
   > 
   > For most of the DB, there would be just only one endpoint for executing all statements. So I feel we should just extend the existing SQL endpoint.
   
   Agree that we should use one single endpoint to handle all statements. My concern is that the single SQL endpoint should probably be a `POST` request instead of the current `GET` request. I'd suggest adding a new `POST` endpoint for all statements, and keep the current `GET` endpoint which can only be used to query data


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org