Icesource commented on a change in pull request #12634:
URL: https://github.com/apache/shardingsphere/pull/12634#discussion_r716058092



##########
File path: 
shardingsphere-test/shardingsphere-parser-test/src/main/resources/sql/supported/ddl/drop-view.xml
##########
@@ -17,5 +17,5 @@
   -->
 
 <sql-cases>
-    <sql-case id="drop_view" value="DROP VIEW kinds" db-types="PostgreSQL" />
+    <sql-case id="drop_view" value="DROP VIEW kinds" db-types="PostgreSQL, 
SQLServer" />
 </sql-cases>

Review comment:
       ```
   <sql-cases>
   -    <sql-case id="drop_view" value="DROP VIEW kinds" db-types="PostgreSQL" 
/>
   +    <sql-case id="drop_view" value="DROP VIEW kinds" db-types="PostgreSQL, 
SQLServer" />
   </sql-cases>
   ```
   @jingshanglu This shows that I added a test on my page. It turned out that 
this test was only performed for PostgreSQL. Since the delete statement is the 
same, I also added SQLServer here. I did not delete any tests and the tests did 
not fail. 
   Maybe you read it wrong ? 




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to