zhztheplayer commented on issue #1150: [CALCITE-2985] Add the JSON_STORAGE_SIZE function URL: https://github.com/apache/calcite/pull/1150#issuecomment-483931057 > Are we agreed to add all these functions to standard sql operator table? If the add of `MysqlSqlOperatorTable` should be done in some other independent patches (maybe in the fix of [CALCITE-2846](https://issues.apache.org/jira/browse/CALCITE-2846)?), I think it's not unacceptable to put them into std table temporarily. But per a [discussion](https://github.com/apache/calcite/pull/1146#discussion_r274074700) from another ongoing PR (cc @julianhyde), I think maybe we haven't yet come into consensus of changing the file `Parser.jj`. Should we wait for a clear result?
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
