[GitHub] [phoenix] gjacoby126 commented on a change in pull request #1055: PHOENIX-6296 Synchronize @Parameters, @BeforeClass and @AfterClass me…

[GitBox <gi...@apache.org>]

gjacoby126 commented on a change in pull request #1055:
URL: https://github.com/apache/phoenix/pull/1055#discussion_r551367729



##########
File path: phoenix-core/src/it/java/org/apache/phoenix/end2end/RowValueConstructorOffsetIT.java
##########
@@ -76,7 +76,7 @@
     private static Connection conn = null;
 
     @BeforeClass
-    public static void init() throws SQLException {
+    public static synchornized void init() throws SQLException {

Review comment:
       typo here (syncronized misspelled) : curious how this is compiling?

##########
File path: phoenix-core/src/it/java/org/apache/phoenix/end2end/ViewIndexIdRetrieveIT.java
##########
@@ -62,7 +62,7 @@
             " FROM SYSTEM.CATALOG WHERE TABLE_NAME='%s' AND COLUMN_COUNT IS NOT NULL";
 
     @BeforeClass
-    public static void setUp() throws Exception {
+    public static snychronized void setUp() throws Exception {

Review comment:
       typo here: shouldn't compile




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org