Avoid round trips from job-entities -> uuids -> job-entities in generating /list API endpoint.

Open scrosby opened this issue 8 years ago • 3 comments

In cook.mesos.api/list-jobs, we lookup jobs by entity, then map them to UUID. Then in https://github.com/twosigma/Cook/blob/master/scheduler/src/cook/mesos/api.clj#L2164, we call cook.mesos.api/fetch-job-map that maps the UUID's back into job-entities. Lets not do this round trip.

Mar 28 '18 16:03 scrosby

Is this a duplicate of issue #617 ?

Mar 28 '18 17:03 dposada

I'm closing this as a duplicate of #617—but I reported #617, and I wholeheartedly agree that we should fix this!

Mar 31 '18 14:03 DaoWen

Its related to #617 in that I'm avoiding round-tripping to and from UUID. But its not a duplicate. Fixing this does not involve any changes to what goes into the liberator context --- the dataflow is via function arguments.

May 31 '18 15:05 scrosby