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

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


The following commit(s) were added to refs/heads/master by this push:
     new 7f45d2c3 [YUNIKORN-1918] Remove UpdateSchedulerConfig from core (#622)
7f45d2c3 is described below

commit 7f45d2c3560fe312e864ae92ed69e30fb79dc7e1
Author: Lyu Bo Cian <[email protected]>
AuthorDate: Tue Aug 22 09:13:16 2023 +0200

    [YUNIKORN-1918] Remove UpdateSchedulerConfig from core (#622)
    
    Closes: #622
    
    Signed-off-by: Peter Bacsko <[email protected]>
---
 pkg/scheduler/context.go | 20 --------------------
 1 file changed, 20 deletions(-)

diff --git a/pkg/scheduler/context.go b/pkg/scheduler/context.go
index c20a3e8c..c944d886 100644
--- a/pkg/scheduler/context.go
+++ b/pkg/scheduler/context.go
@@ -324,26 +324,6 @@ func (cc *ClusterContext) removePartitionsByRMID(event 
*rmevent.RMPartitionsRemo
        }
 }
 
-// Locked version of the configuration update called from the webservice
-// NOTE: this call assumes one RM which is registered and uses that RM for the 
updates
-func (cc *ClusterContext) UpdateSchedulerConfig(conf *configs.SchedulerConfig) 
error {
-       cc.Lock()
-       defer cc.Unlock()
-       // hack around the missing rmID
-       for _, pi := range cc.partitions {
-               rmID := pi.RmID
-               log.Log(log.SchedContext).Debug("Assuming one RM on config 
update call from webservice",
-                       zap.String("rmID", rmID))
-               if err := cc.updateSchedulerConfig(conf, rmID); err != nil {
-                       return err
-               }
-               // update global scheduler configs
-               configs.ConfigContext.Set(cc.policyGroup, conf)
-               return nil
-       }
-       return fmt.Errorf("RM has no active partitions, make sure it is 
registered")
-}
-
 // Locked version of the configuration update called outside of event system.
 // Updates the current config via the config loader.
 // Used in test only, normal updates use the internal call


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

Reply via email to