zhang1002 opened a new pull request #1002:
URL: https://github.com/apache/incubator-kyuubi/pull/1002


   Add classification of SQL_Type: this PR is just for DDL
   
   Now we use the simple name to judge this operation belong to which 
calssification of SQL_Type.
   
   The SQL_Type will store in the spark's conf.
   The configuration item's name is ${sql.type}.
   
   Notice:
   1. The classification of SQL_Type is based on Spark_v_3.1.2.
   2. If the sql grammars have added in the future, the matching rule should 
changed. We can add a configuration item that allow user to add the sql grammar 
that the code is not defined.
   
   ### _Why are the changes needed?_
   By these SQL_Type, we can do something below:
   1. Reduce some log print:
        For Auxiliary Statements(the sql type is others), we don't need to 
print operation log.
   2. Optimizing some configuration item.
        For example, in final stage, the conf is different between DML and DQL.
        Through this configuration item, we can use different conf for them.
   
   
   ### _How was this patch tested?_
   - [ ] Add some test cases that check the changes thoroughly including 
negative and positive cases if possible
   
   - [ ] Add screenshots for manual tests if appropriate
   
   - [ ] [Run 
test](https://kyuubi.readthedocs.io/en/latest/develop_tools/testing.html#running-tests)
 locally before make a pull request
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to