pvary commented on code in PR #10112:
URL: https://github.com/apache/iceberg/pull/10112#discussion_r1560439436
##########
flink/v1.19/flink/src/test/java/org/apache/iceberg/flink/TestChangeLogTable.java:
##########
@@ -98,7 +98,7 @@ public void before() {
@Override
public void clean() {
sql("DROP TABLE IF EXISTS %s", TABLE_NAME);
- sql("DROP DATABASE IF EXISTS %s", DATABASE_NAME);
+ dropDatabase(DATABASE_NAME, true);
Review Comment:
Do we really need this? Something behind the scenes issue a `USE DATABASE`
command?
If so, this could be a breaking change which we might want to highlight, as
creating and dropping a database immediately after that won't work anymore
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
