apache/shardingsphere

Support parsing Oracle CREATE /*+ sql

Open

#27.019 aperta il 13 lug 2023

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Java (6902 fork)batch import
db: Oraclegood first issuein: SQL parsetype: enhancement

Metriche repository

Star
 (20.714 star)
Metriche merge PR
 (Merge medio 3g 14h) (194 PR mergiate in 30 g)

Descrizione

Background

Hi community, This issue is for #26878.

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

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

Task

This issue is to support more oracle sql parse, as follows:

CREATE /*+ PQ_DISTRIBUTE(lineitem, PARTITION) */ TABLE lineitem
  NOLOGGING PARALLEL 16
  PARTITION BY HASH (l_orderkey) PARTITIONS 512
  AS SELECT * FROM lineitemxt;
CREATE /*+ PQ_DISTRIBUTE(target_table, PARTITION) */ TABLE target_table
  NOLOGGING PARALLEL 16
  PARTITION BY HASH (l_orderkey) PARTITIONS 512
  AS SELECT * FROM source_table;
CONNNECT SCOTT


DROP TABLE orders_tab;
DROP TABLE customers;
COUNT({ * | [ DISTINCT | ALL ] expr }) [ OVER (analytic_clause) ]
CONTROLFILE REUSE 
LOGFILE

Process

  1. First confirm that this is a correct oracle sql syntax, if not please ignore;
  2. Compare SQL definitions in Oficial 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 Statment 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 Oracle SQLs

Guida contributor