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 ALTER SYNONYM in openGauss #17856

Merged
merged 1 commit into from May 22, 2022

Conversation

everly-gif
Copy link
Contributor

Ref #17736

Changes proposed in this pull request:

  • Adds ALTER SYNONYM GRAMMAR
  • Supports Parsing ALTER SYNONYM in openGauss
  • Adds tests

✅ Builds locally

@codecov-commenter
Copy link

Codecov Report

Merging #17856 (d733e11) into master (55c26fb) will decrease coverage by 0.00%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master   #17856      +/-   ##
============================================
- Coverage     59.20%   59.19%   -0.01%     
  Complexity     2134     2134              
============================================
  Files          3604     3606       +2     
  Lines         53432    53435       +3     
  Branches       9111     9111              
============================================
+ Hits          31632    31633       +1     
- Misses        19119    19121       +2     
  Partials       2681     2681              
Impacted Files Coverage Δ
...ql/common/statement/ddl/AlterSynonymStatement.java 0.00% <0.00%> (ø)
.../opengauss/ddl/OpenGaussAlterSynonymStatement.java 0.00% <0.00%> (ø)
...tatement/impl/OpenGaussDDLStatementSQLVisitor.java 72.72% <100.00%> (+0.09%) ⬆️

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 55c26fb...d733e11. Read the comment docs.

@strongduanmu strongduanmu merged commit fe82048 into apache:master May 22, 2022
@everly-gif everly-gif deleted the alter-synonym branch May 22, 2022 13:09
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
kaori-seasons added a commit to kaori-seasons/shardingsphere that referenced this pull request May 26, 2022
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