[GitHub] phoenix pull request #418: [PHOENIX-3623] Integrate Omid with Phoenix.

[JamesRTaylor]

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

    https://github.com/apache/phoenix/pull/418#discussion_r242226148
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/transaction/OmidTransactionContext.java
 ---
    @@ -18,89 +18,279 @@
     package org.apache.phoenix.transaction;
     
     import java.sql.SQLException;
    +import java.util.Arrays;
    +import java.util.HashSet;
    +import java.util.Set;
     
     import org.apache.hadoop.hbase.client.Table;
    +import org.apache.omid.proto.TSOProto;
    +import org.apache.omid.transaction.AbstractTransaction.VisibilityLevel;
    +import org.apache.omid.transaction.HBaseCellId;
    +import org.apache.omid.transaction.HBaseTransaction;
    +import org.apache.omid.transaction.HBaseTransactionManager;
    +import org.apache.omid.transaction.RollbackException;
    +import org.apache.omid.transaction.Transaction;
    +//import org.apache.omid.tso.TSOMockModule;
    +import org.apache.omid.transaction.Transaction.Status;
    +import org.apache.omid.transaction.TransactionException;
    +import org.apache.phoenix.exception.SQLExceptionCode;
    +import org.apache.phoenix.exception.SQLExceptionInfo;
     import org.apache.phoenix.jdbc.PhoenixConnection;
     import org.apache.phoenix.schema.PTable;
     import org.apache.phoenix.transaction.TransactionFactory.Provider;
    +import org.slf4j.Logger;
    +import org.slf4j.LoggerFactory;
    +
    +import com.google.protobuf.InvalidProtocolBufferException;
    +//import org.apache.omid.tso.TSOMockModule;
    --- End diff --
    
    Remove commented out import


---