You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by "FlyingZC (via GitHub)" <gi...@apache.org> on 2023/08/02 10:01:02 UTC

[GitHub] [shardingsphere] FlyingZC opened a new issue, #27803: Support parsing openGauss create FOREIGN sql

FlyingZC opened a new issue, #27803:
URL: https://github.com/apache/shardingsphere/issues/27803

   # Background
   Hi community,
   
   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 openGauss SQL parsing in ShardingSphere.
   
   More details:
   https://shardingsphere.apache.org/document/current/en/reference/sharding/parse/
   
   # Task
   This issue is to support more openGauss sql parse, as follows:
   ```sql
   create FOREIGN table bmsql_oorder ( 
     o_w_id       integer      not null, 
     o_d_id       integer      not null, 
     o_id         integer      not null, 
     o_c_id       integer not null, 
     o_carrier_id integer,          
     o_ol_cnt     integer, 
     o_all_local  integer, 
     o_entry_d    timestamp, 
     primary key (o_w_id, o_d_id, o_id) 
   ); 
   create index  bmsql_oorder_index1 on bmsql_oorder(o_w_id, o_d_id, o_c_id, o_id) ;
   ```
   
   ```sql
   benchmarksql=# select * from customer;
   ```
   
   ```sql
   benchmarksql=# alter table customer rename to customer_bk;
   ```
   
   ```sql
   benchmarksql=# create foreign table customer (x int, y int); 
   CREATE FOREIGN TABLE 
   benchmarksql=# select * from customer;
   ```
   
   ```sql
   $ gs_restore -C -d benchmarksql customer.dump -p 16000
   restore operation successful 
   total time: 24  ms 
   Check that the data was imported successfully. 
   benchmarksql=# select * from customer;
   ```
   
   # Process
   1. First confirm that this is a correct openGauss 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 openGauss SQLs


-- 
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: notifications-unsubscribe@shardingsphere.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Support parsing openGauss create FOREIGN sql [shardingsphere]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #27803:
URL: https://github.com/apache/shardingsphere/issues/27803#issuecomment-1783911491

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


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [shardingsphere] github-actions[bot] commented on issue #27803: Support parsing openGauss create FOREIGN sql

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #27803:
URL: https://github.com/apache/shardingsphere/issues/27803#issuecomment-1703266936

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


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Support parsing openGauss create FOREIGN sql [shardingsphere]

Posted by "FlyingZC (via GitHub)" <gi...@apache.org>.
FlyingZC closed issue #27803: Support parsing openGauss create FOREIGN sql
URL: https://github.com/apache/shardingsphere/issues/27803


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org