This is an automated email from the ASF dual-hosted git repository.

ccondit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/yunikorn-k8shim.git


The following commit(s) were added to refs/heads/master by this push:
     new 2eebc3a8 [YUNIKORN-2077] Cleanup scheduler event interface in shim 
(#704)
2eebc3a8 is described below

commit 2eebc3a8e97af7bd9af18b7f6b9979d7566af452
Author: Yu-Lin Chen <[email protected]>
AuthorDate: Thu Oct 26 09:58:39 2023 -0500

    [YUNIKORN-2077] Cleanup scheduler event interface in shim (#704)
    
    Closes: #704
    
    Signed-off-by: Craig Condit <[email protected]>
---
 pkg/cmd/shim/main.go         |  2 +-
 pkg/common/events/events.go  | 12 ------------
 pkg/dispatcher/dispatcher.go |  3 ---
 3 files changed, 1 insertion(+), 16 deletions(-)

diff --git a/pkg/cmd/shim/main.go b/pkg/cmd/shim/main.go
index cc6785df..5d31e6a8 100644
--- a/pkg/cmd/shim/main.go
+++ b/pkg/cmd/shim/main.go
@@ -52,7 +52,7 @@ func main() {
        if serviceContext.RMProxy != nil {
                ss := shim.NewShimScheduler(serviceContext.RMProxy, 
conf.GetSchedulerConf(), configMaps)
                if err := ss.Run(); err != nil {
-                       log.Log(log.Shim).Fatal("Unable tto start scheduler", 
zap.Error(err))
+                       log.Log(log.Shim).Fatal("Unable to start scheduler", 
zap.Error(err))
                }
 
                signalChan := make(chan os.Signal, 1)
diff --git a/pkg/common/events/events.go b/pkg/common/events/events.go
index b3f70819..395f6626 100644
--- a/pkg/common/events/events.go
+++ b/pkg/common/events/events.go
@@ -64,18 +64,6 @@ type TaskEvent interface {
        GetArgs() []interface{}
 }
 
-// --------------------------------------
-// scheduler events
-// --------------------------------------
-type SchedulerEvent interface {
-       // the type of this event
-       GetEvent() string
-
-       // an event can have multiple arguments, these arguments will be passed 
to
-       // state machines' callbacks when doing state transition
-       GetArgs() []interface{}
-}
-
 // --------------------------------------
 // scheduler node events
 // --------------------------------------
diff --git a/pkg/dispatcher/dispatcher.go b/pkg/dispatcher/dispatcher.go
index 65d88efb..719c5ccb 100644
--- a/pkg/dispatcher/dispatcher.go
+++ b/pkg/dispatcher/dispatcher.go
@@ -40,7 +40,6 @@ const (
        EventTypeApp EventType = iota
        EventTypeTask
        EventTypeNode
-       EventTypeScheduler
 )
 
 var (
@@ -194,8 +193,6 @@ func Start() {
                                        getEventHandler(EventTypeApp)(v)
                                case events.SchedulerNodeEvent:
                                        getEventHandler(EventTypeNode)(v)
-                               case events.SchedulerEvent:
-                                       getEventHandler(EventTypeScheduler)(v)
                                default:
                                        
log.Log(log.ShimDispatcher).Fatal("unsupported event",
                                                zap.Any("event", v))


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to