shardingsphere icon indicating copy to clipboard operation
shardingsphere copied to clipboard

Support parsing SQL Server SELECT TRIM( sql

Open FlyingZC opened this issue 2 years ago • 7 comments

Background

Hi community. This issue is for #29149.

The ShardingSphere SQL parser engine helps users to parse SQL to create the AST (Abstract Syntax Tree) and visit the AST to get SQLStatement (Java Object). Currently, we are planning to enhance the support for SQL Server SQL parsing in ShardingSphere.

More details: https://shardingsphere.apache.org/document/current/en/reference/sharding/parse/

Task

This issue is to support more SQL Server sql parsing, as follows:

SELECT TRIM( '.,! ' FROM '     #     test    .') AS Result

link

SELECT TRIM(LEADING '.,! ' FROM  '     .#     test    .') AS Result

link

SELECT TRIM(TRAILING '.,! ' FROM '     .#     test    .') AS Result

link

SELECT TRIM(BOTH '123' FROM '123abc123') AS Result

link

INSERT INTO [HR].[Employees]
        ([SSN]
        ,[FirstName]
        ,[LastName]
        ,[Salary])
    VALUES
        ('795-73-9838'
        , N'Catherine'
        , N'Abel'
        , $31692)

link

Process

  1. First confirm that this is a correct SQL Server sql syntax, if not please leave a message under the issue and ignore it;
  2. Compare SQL definitions in Official SQL Doc and ShardingSphere SQL Doc;
  3. If there is any difference in ShardingSphere SQL Doc, please correct them by referring to the Official SQL Doc;
  4. Run mvn install the current_file_module;
  5. Check whether there are any exceptions. If indeed, please fix them. (Especially xxxVisitor.class);
  6. Add new corresponding SQL case in SQL Cases and expected parsed result in Expected Statement XML;
  7. Run SQLParserParameterizedTest to make sure no exceptions.

Relevant Skills

  1. Master JAVA language
  2. Have a basic understanding of Antlr g4 file
  3. Be familiar with SQL Server SQLs

FlyingZC avatar Nov 24 '23 07:11 FlyingZC

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale.

github-actions[bot] avatar Dec 24 '23 20:12 github-actions[bot]

Hi @FlyingZC I am new to the community and trying to add the support for TRIM function.

Please review: https://github.com/apache/shardingsphere/pull/29542

venkatdatta0 avatar Dec 25 '23 16:12 venkatdatta0

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale.

github-actions[bot] avatar Jan 25 '24 20:01 github-actions[bot]

Hi @venkatdatta0, I'm so sorry for late repley. I will review this pr.

strongduanmu avatar Jan 26 '24 06:01 strongduanmu

Hi, @strongduanmu can you assigned this to me ? i want to try

zhengkezhou1 avatar Feb 21 '24 12:02 zhengkezhou1

@strongduanmu Hello, Can you assign me this task? I'll try to solve it.

zhuruixuan avatar Feb 21 '24 14:02 zhuruixuan

I assigned the task to @zzzk1 according to first come, first served basis. @zhuruixuan Are you interested in other SQL parsing tasks? There are many more tasks on our issue list.

strongduanmu avatar Feb 22 '24 00:02 strongduanmu