You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by el...@apache.org on 2016/06/21 22:02:31 UTC

[1/4] phoenix git commit: PHOENIX-3017 Catch TableNotFoundException and avoid logging at warn

Repository: phoenix
Updated Branches:
  refs/heads/4.x-HBase-0.98 f26a3219e -> 37f80712b
  refs/heads/4.x-HBase-1.0 3b06216ae -> 499f5c416
  refs/heads/4.x-HBase-1.1 155144093 -> fa8df989f
  refs/heads/master 9e03a48fb -> 332e6cb63


PHOENIX-3017 Catch TableNotFoundException and avoid logging at warn


Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/332e6cb6
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/332e6cb6
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/332e6cb6

Branch: refs/heads/master
Commit: 332e6cb6328d42f840054473c11e24b88a96cdd0
Parents: 9e03a48
Author: Josh Elser <el...@apache.org>
Authored: Tue Jun 21 16:43:36 2016 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Tue Jun 21 16:43:36 2016 -0400

----------------------------------------------------------------------
 .../src/main/java/org/apache/phoenix/query/TableStatsCache.java | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/332e6cb6/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java b/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
index 287886e..2c7b2db 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
@@ -22,6 +22,7 @@ import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 
 import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.hbase.TableNotFoundException;
 import org.apache.hadoop.hbase.client.HTableInterface;
 import org.apache.hadoop.hbase.util.Bytes;
 import org.apache.phoenix.hbase.index.util.ImmutableBytesPtr;
@@ -93,6 +94,10 @@ public class TableStatsCache {
                         Long.MAX_VALUE);
                 traceStatsUpdate(tableNameBytes, stats);
                 return stats;
+            } catch (TableNotFoundException e) {
+                // On a fresh install, stats might not yet be created, don't warn about this.
+                logger.debug("Unable to locate Phoenix stats table", e);
+                return PTableStats.EMPTY_STATS;
             } catch (IOException e) {
                 logger.warn("Unable to read from stats table", e);
                 // Just cache empty stats. We'll try again after some time anyway.


[4/4] phoenix git commit: PHOENIX-3017 Catch TableNotFoundException and avoid logging at warn

Posted by el...@apache.org.
PHOENIX-3017 Catch TableNotFoundException and avoid logging at warn


Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/37f80712
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/37f80712
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/37f80712

Branch: refs/heads/4.x-HBase-0.98
Commit: 37f80712bdd9632a5b00c845ffa231b5c8c8f590
Parents: f26a321
Author: Josh Elser <el...@apache.org>
Authored: Tue Jun 21 16:43:36 2016 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Tue Jun 21 17:54:10 2016 -0400

----------------------------------------------------------------------
 .../src/main/java/org/apache/phoenix/query/TableStatsCache.java | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/37f80712/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java b/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
index 287886e..2c7b2db 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
@@ -22,6 +22,7 @@ import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 
 import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.hbase.TableNotFoundException;
 import org.apache.hadoop.hbase.client.HTableInterface;
 import org.apache.hadoop.hbase.util.Bytes;
 import org.apache.phoenix.hbase.index.util.ImmutableBytesPtr;
@@ -93,6 +94,10 @@ public class TableStatsCache {
                         Long.MAX_VALUE);
                 traceStatsUpdate(tableNameBytes, stats);
                 return stats;
+            } catch (TableNotFoundException e) {
+                // On a fresh install, stats might not yet be created, don't warn about this.
+                logger.debug("Unable to locate Phoenix stats table", e);
+                return PTableStats.EMPTY_STATS;
             } catch (IOException e) {
                 logger.warn("Unable to read from stats table", e);
                 // Just cache empty stats. We'll try again after some time anyway.


[3/4] phoenix git commit: PHOENIX-3017 Catch TableNotFoundException and avoid logging at warn

Posted by el...@apache.org.
PHOENIX-3017 Catch TableNotFoundException and avoid logging at warn


Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/499f5c41
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/499f5c41
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/499f5c41

Branch: refs/heads/4.x-HBase-1.0
Commit: 499f5c4160becae75e91ebf904a7e2e95831a196
Parents: 3b06216
Author: Josh Elser <el...@apache.org>
Authored: Tue Jun 21 16:43:36 2016 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Tue Jun 21 17:50:09 2016 -0400

----------------------------------------------------------------------
 .../src/main/java/org/apache/phoenix/query/TableStatsCache.java | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/499f5c41/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java b/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
index 287886e..2c7b2db 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
@@ -22,6 +22,7 @@ import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 
 import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.hbase.TableNotFoundException;
 import org.apache.hadoop.hbase.client.HTableInterface;
 import org.apache.hadoop.hbase.util.Bytes;
 import org.apache.phoenix.hbase.index.util.ImmutableBytesPtr;
@@ -93,6 +94,10 @@ public class TableStatsCache {
                         Long.MAX_VALUE);
                 traceStatsUpdate(tableNameBytes, stats);
                 return stats;
+            } catch (TableNotFoundException e) {
+                // On a fresh install, stats might not yet be created, don't warn about this.
+                logger.debug("Unable to locate Phoenix stats table", e);
+                return PTableStats.EMPTY_STATS;
             } catch (IOException e) {
                 logger.warn("Unable to read from stats table", e);
                 // Just cache empty stats. We'll try again after some time anyway.


[2/4] phoenix git commit: PHOENIX-3017 Catch TableNotFoundException and avoid logging at warn

Posted by el...@apache.org.
PHOENIX-3017 Catch TableNotFoundException and avoid logging at warn


Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/fa8df989
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/fa8df989
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/fa8df989

Branch: refs/heads/4.x-HBase-1.1
Commit: fa8df989f084a27c2c7e96bfa662a007212bfcf9
Parents: 1551440
Author: Josh Elser <el...@apache.org>
Authored: Tue Jun 21 16:43:36 2016 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Tue Jun 21 17:46:21 2016 -0400

----------------------------------------------------------------------
 .../src/main/java/org/apache/phoenix/query/TableStatsCache.java | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/fa8df989/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java b/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
index 287886e..2c7b2db 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/query/TableStatsCache.java
@@ -22,6 +22,7 @@ import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 
 import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.hbase.TableNotFoundException;
 import org.apache.hadoop.hbase.client.HTableInterface;
 import org.apache.hadoop.hbase.util.Bytes;
 import org.apache.phoenix.hbase.index.util.ImmutableBytesPtr;
@@ -93,6 +94,10 @@ public class TableStatsCache {
                         Long.MAX_VALUE);
                 traceStatsUpdate(tableNameBytes, stats);
                 return stats;
+            } catch (TableNotFoundException e) {
+                // On a fresh install, stats might not yet be created, don't warn about this.
+                logger.debug("Unable to locate Phoenix stats table", e);
+                return PTableStats.EMPTY_STATS;
             } catch (IOException e) {
                 logger.warn("Unable to read from stats table", e);
                 // Just cache empty stats. We'll try again after some time anyway.