walterddr commented on PR #8557: URL: https://github.com/apache/pinot/pull/8557#issuecomment-1104432567
I think `INSERT INTO` makes more sense for several reasons 1. there's no ANSI standard for loading data from static objects; it is a DML specific to a database system. 2. `INSERT INTO FROM xx` supports default SqlInsert syntactic extension from `INSERT` statement which is a type of extensible statement similar to `CREATE xxx` is extensible statement as DDL; I don't think there's a statement extension from `LOAD xxx` keyword. 3. `INSERT INTO (SELECT * FROM MYTABLE)` is actually a valid syntax in many databases -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
