Skip to content

Commit dd9ac0f

Browse files
committed
[core] Correct scope of local offer
1 parent 12a972b commit dd9ac0f

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

core/task/scheduler.go

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -536,10 +536,12 @@ func (state *schedulerState) resourceOffers(fidStore store.Singleton) events.Han
536536
var offerWaitGroup sync.WaitGroup
537537
offerWaitGroup.Add(len(offers))
538538

539-
for offerIndex, offer := range offers {
540-
go func(offerIndex int, offer *mesos.Offer) {
539+
for offerIndex, _ := range offers {
540+
go func(offerIndex int) {
541541
defer offerWaitGroup.Done()
542542

543+
offer := offers[offerIndex]
544+
543545
timeSingleOffer := time.Now()
544546
var (
545547
remainingResourcesInOffer = mesos.Resources(offer.Resources)
@@ -697,7 +699,7 @@ func (state *schedulerState) resourceOffers(fidStore store.Singleton) events.Han
697699
state.taskman.AgentCache.Update(agentForCache) //thread safe
698700
machinesUsed[offer.Hostname] = struct{}{}
699701

700-
taskPtr := state.taskman.newTaskForMesosOffer(offer, descriptor, bindMap, targetExecutorId)
702+
taskPtr := state.taskman.newTaskForMesosOffer(&offer, descriptor, bindMap, targetExecutorId)
701703
if taskPtr == nil {
702704
log.WithPrefix("scheduler").
703705
WithField("partition", envId.String()).
@@ -965,7 +967,7 @@ func (state *schedulerState) resourceOffers(fidStore store.Singleton) events.Han
965967
WithField("offers", len(offers)).
966968
WithField("offerHost", offer.Hostname))
967969

968-
}(offerIndex, &offer) // end for offer closure
970+
}(offerIndex) // end for offer closure
969971
} // end for _, offer := range offers
970972
offerWaitGroup.Wait()
971973

0 commit comments

Comments
 (0)