trafodion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ansha...@apache.org
Subject [14/15] incubator-trafodion git commit: Update HTableClient.java
Date Thu, 15 Sep 2016 22:48:09 GMT
Update HTableClient.java

Project: http://git-wip-us.apache.org/repos/asf/incubator-trafodion/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-trafodion/commit/7cb5d9e9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-trafodion/tree/7cb5d9e9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-trafodion/diff/7cb5d9e9

Branch: refs/heads/master
Commit: 7cb5d9e9af4aa3e519d7019e742245e058abc295
Parents: e3b9e96
Author: Sean Broeder <sean.broeder@esgyn.com>
Authored: Tue Sep 13 14:38:14 2016 -0700
Committer: GitHub <noreply@github.com>
Committed: Tue Sep 13 14:38:14 2016 -0700

----------------------------------------------------------------------
 .../java/org/trafodion/sql/HTableClient.java    | 20 +++++---------------
 1 file changed, 5 insertions(+), 15 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-trafodion/blob/7cb5d9e9/core/sql/src/main/java/org/trafodion/sql/HTableClient.java
----------------------------------------------------------------------
diff --git a/core/sql/src/main/java/org/trafodion/sql/HTableClient.java b/core/sql/src/main/java/org/trafodion/sql/HTableClient.java
index 8ee33d8..a00d01d 100644
--- a/core/sql/src/main/java/org/trafodion/sql/HTableClient.java
+++ b/core/sql/src/main/java/org/trafodion/sql/HTableClient.java
@@ -1393,7 +1393,7 @@ public class HTableClient {
 //                        table.deleteRegionTx(del, /* auto-commit */ true);
 //                    }
                     else {
-                       logger.info("deleteRow without transID ");
+//                       logger.info("deleteRow without transID ");
                        table.delete(del);
                     }
 					return true;
@@ -1409,7 +1409,6 @@ public class HTableClient {
 //               table.deleteRegionTx(del, /* auto-commit */ true);
 //            }
             else {
-               logger.info("deleteRow without transID ");
                table.delete(del);
             }
 		}
@@ -1509,7 +1508,6 @@ public class HTableClient {
 //               		         del, /* autoCommit */ true);
 //            }
             else {
-               logger.info("checkAndDeleteRow without transID ");
 			   res = table.checkAndDelete(rowID, family, qualifier, colValToCheck, del);
             }
 
@@ -1564,32 +1562,28 @@ public class HTableClient {
 
 					if (checkAndPut) {
                        if (useTRex && (transID != 0)){
-                          logger.info("checkAndPut using put with transID " + transID);
                           res = table.checkAndPut(transID, rowID,
                                      family1, qualifier1, colValToCheck, put);
                        }
                        else if (useRegionTransaction){
-                           logger.info("checkAndPutRegionTx with regionTX ");
+                           if (logger.isTraceEnabled()) logger.trace("checkAndPutRegionTx
with regionTX ");
                            res = table.checkAndPutRegionTx(rowID, 
                                            family1, qualifier1, colValToCheck, put, /* auto-commit
*/ true);
                     	   
                        }
                        else {
-                          logger.info("checkAndPut without transID ");
                           res = table.checkAndPut(rowID, 
                                           family1, qualifier1, colValToCheck, put);
                        }
 					}
 					else {
                        if (useTRex && (transID != 0)){
-                          logger.info("putRow using put with transID " + transID);
                           table.put(transID, put);
                        }
                        else if (useRegionTransaction){
-                          logger.info("putRow using putRegionTx");
+                          if (logger.isTraceEnabled()) logger.trace("putRow using putRegionTx");
                           table.putRegionTx(put, /* auto-commit */ true);
                        }else{ 
-                          logger.info("putRow not using putRegionTx");
                           table.put(put);
 					   }
 					}
@@ -1601,31 +1595,27 @@ public class HTableClient {
 		 	boolean result = true;
 			if (checkAndPut) {
                if (useTRex && (transID != 0)){
-                  logger.info("checkAndPut using put with transID " + transID);
                   result = table.checkAndPut(transID, rowID, 
 						family1, qualifier1, colValToCheck, put);
                }
                else if (useRegionTransaction){
-                   logger.info("checkAndPutRegionTx using regionTX ");
+                   if (logger.isTraceEnabled()) logger.trace("checkAndPutRegionTx using regionTX
");
                    result = table.checkAndPutRegionTx(rowID, family1, qualifier1,
                 		   colValToCheck, put, /* auto-commit */ true);
                }
                else {
-                  logger.info("checkAndPut without transID ");
                   result = table.checkAndPut(rowID, 
 						family1, qualifier1, colValToCheck, put);
                }
 			}
 			else {
                if (useTRex && (transID != 0)){
-                  logger.info("putRow using put with transID " + transID);
                   table.put(transID, put);
                }
                else if (useRegionTransaction){
-                  logger.info("putRow using putRegionTx");
+                  if (logger.isTraceEnabled()) logger.trace("putRow using putRegionTx");
                   table.putRegionTx(put, true /* also commit */);
                }else{
-                  logger.info("putRow not using putRegionTx");
                   table.put(put);
                }
 			}


Mime
View raw message