Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support parsing CREATE SYNONYM in openGauss #17861

Merged
merged 2 commits into from May 23, 2022

Conversation

everly-gif
Copy link
Contributor

Ref #17736

Changes proposed in this pull request:

  • Adds CREATE SYNONYM grammar
  • Supports Parsing CREATE SYNONYM in openGauss
  • Adds tests

✅ Builds locally

@codecov-commenter
Copy link

Codecov Report

Merging #17861 (052f687) into master (d2d4631) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master   #17861      +/-   ##
============================================
- Coverage     59.21%   59.20%   -0.01%     
  Complexity     2134     2134              
============================================
  Files          3606     3607       +1     
  Lines         53416    53418       +2     
  Branches       9110     9110              
============================================
- Hits          31630    31628       -2     
- Misses        19105    19109       +4     
  Partials       2681     2681              
Impacted Files Coverage Δ
...opengauss/ddl/OpenGaussCreateSynonymStatement.java 0.00% <0.00%> (ø)
...tatement/impl/OpenGaussDDLStatementSQLVisitor.java 72.82% <100.00%> (+0.09%) ⬆️
...d/text/distsql/ral/common/hint/HintSourceType.java 0.00% <0.00%> (-42.86%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d2d4631...052f687. Read the comment docs.

@strongduanmu strongduanmu merged commit 823aeb0 into apache:master May 23, 2022
zhaoguhong added a commit to zhaoguhong/shardingsphere that referenced this pull request May 23, 2022
* master: (43 commits)
  Move CachedDatabaseMetaData to jdbc module (apache#17868)
  Update scaling doc (apache#17867)
  Remove ShardingSphereResource.cachedDatabaseMetaData (apache#17866)
  Add JDBCContext for refresh (apache#17865)
  Remove unused Exception in ShardingSphereSQLParserEngine (apache#17864)
  Add binding table rule check. (apache#17810)
  Support parsing `CREATE SYNONYM` in openGauss (apache#17861)
  support parsing drop synonym in openGauss (apache#17862)
  Move CachedDatabaseMetaData to ShardingSphereDataSource (apache#17863)
  Remove getter for ShardingSphereDataSource (apache#17860)
  support pasring alter synonym in openGauss (apache#17856)
  Refactor optional named test cases (apache#17859)
  Refactor ShardingSphereRuleMetaData (apache#17857)
  Rename backendDatabaseType to storageType (apache#17854)
  feat: disable indication of mvn install in CI (apache#17851)
  Spotless format XML file format (apache#17853)
  Rename frontendDatabaseType to protocolType (apache#17852)
  update pom.xml and LICENSE. (apache#17785)
  Support base-seata transaction for jdbc (apache#17846)
  Add PostgreSQL CALL Statement (apache#17847)
  ...

# Conflicts:
#	shardingsphere-features/shardingsphere-encrypt/shardingsphere-encrypt-core/src/test/java/org/apache/shardingsphere/encrypt/merge/dql/EncryptAlgorithmMetaDataTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants