Makes sure we don't create a full table watch for tombstones.

consul/state/state_store.go

@@ -88,7 +88,7 @@ func NewStateStore(logOutput io.Writer) (*StateStore, error) {
 	// Build up the all-table watches.
 	tableWatches := make(map[string]*FullTableWatch)
 	for table, _ := range schema.Tables {
-		if table == "kvs" {
+		if table == "kvs" || table == "tombstones" {
 			continue
 		}
 

consul/state/state_store_test.go

@@ -240,12 +240,24 @@ func TestStateStore_GetTableWatch(t *testing.T) {
 	// This test does two things - it makes sure there's no full table
 	// watch for KVS, and it makes sure that asking for a watch that
 	// doesn't exist causes a panic.
+	func() {
 		defer func() {
 			if r := recover(); r == nil {
 				t.Fatalf("didn't get expected panic")
 			}
 		}()
 		s.GetTableWatch("kvs")
+	}()
+
+	// Similar for tombstones; those don't support watches at all.
+	func() {
+		defer func() {
+			if r := recover(); r == nil {
+				t.Fatalf("didn't get expected panic")
+			}
+		}()
+		s.GetTableWatch("tombstones")
+	}()
 }
 
 func TestStateStore_EnsureRegistration(t *testing.T) {