walterddr opened a new pull request #7485:
URL: https://github.com/apache/pinot/pull/7485


   Currently if there's a double quoted PQL syntax ingestion transform such as
   ```
     "ingestionConfig": {
       "transformConfigs": [
         {
           "columnName": "created_at_timestamp",
           "transformFunction": "fromDateTime(created_at, 
\"yyyy-MM-dd''T''HH:mm:ss''Z''\")"
         }
       ]
     },
   ```
   It will fail because the double quoted literal is considered as quoted 
identifier in Calcite SQL parser. 
   This is not a problem during SQL execution because the column will not be 
found. but during ingestion we dont have a proper schema validation thus 
exception will not be meaningful.
   
   Propose to wrap exception in InbuiltFunctionEvaluator that executes any 
ScalarFuntion in FunctionRegistry and provide extra context information about 
the argument that passed in. 


-- 
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

Reply via email to