mchades commented on code in PR #4996:
URL: https://github.com/apache/gravitino/pull/4996#discussion_r1798996634


##########
trino-connector/integration-test/src/test/java/org/apache/gravitino/trino/connector/integration/test/TrinoQueryITBase.java:
##########
@@ -68,14 +68,17 @@ public class TrinoQueryITBase {
   protected static final String metalakeName = "test";
   protected static GravitinoMetalake metalake;
 
-  private static void setEnv() throws Exception {
+  private static BaseIT baseIT;
+
+  private void setEnv() throws Exception {
+    baseIT = new BaseIT();

Review Comment:
   How to use `BaseIT`? What is the difference between creating an instance and 
inheritance? Can we add an abstract modifier to `baseIT` for uniform usage?



-- 
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