Browse Source

etcdserver: remove panic and return default err

Blake Mizerany 11 years ago
parent
commit
c98f1cb501
1 changed files with 6 additions and 1 deletions
  1. 6 1
      etcdserver2/server.go

+ 6 - 1
etcdserver2/server.go

@@ -1,12 +1,16 @@
 package etcdserver
 
 import (
+	"errors"
+
 	"code.google.com/p/go.net/context"
 	"github.com/coreos/etcd/raft"
 	"github.com/coreos/etcd/store"
 	"github.com/coreos/etcd/wait"
 )
 
+var ErrUnknownMethod = errors.New("etcdserver: unknown method")
+
 type Response struct {
 	// The last seen term raft was at when this request was built.
 	Term int
@@ -95,8 +99,9 @@ func (s *Server) Do(ctx context.Context, r Request) (Response, error) {
 			}
 			return Response{Event: ev}, nil
 		}
+	default:
+		return Response{}, ErrUnknownMethod
 	}
-	panic("not reached") // for some reason the compiler wants this... :/
 }
 
 // apply interprets r as a call to store.X and returns an Response interpreted from store.Event