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

    https://github.com/apache/phoenix/pull/329#discussion_r211447479
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/schema/MetaDataClient.java ---
    @@ -3681,7 +3691,12 @@ public MutationState addColumn(PTable table, 
List<ColumnDef> origColumnDefs,
                 }
             } finally {
                 connection.setAutoCommit(wasAutoCommit);
    -            if (!acquiredColumnMutexSet.isEmpty()) {
    +            if (EncodedColumnsUtil.usesEncodedColumnNames(table)) {
    +                // release the mutex on the physical table (used to 
prevent concurrent conflicting
    +                // add column changes)
    +                deleteCell(null, physicalSchemaName, physicalTableName, 
null);
    --- End diff --
    
    done, I missed that.


---

Reply via email to