Github user karanmehta93 commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/359#discussion_r222444065
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java
 ---
    @@ -3206,6 +3219,9 @@ public void upgradeSystemTables(final String url, 
final Properties props) throws
                 try {
                     
metaConnection.createStatement().executeUpdate(getMutexDDL());
                 } catch (NewerTableAlreadyExistsException e) {} catch 
(TableAlreadyExistsException e) {}
    +            try {
    --- End diff --
    
    nit: Can we refactor and extract out these dupe pieces of code into a 
method?


---

Reply via email to