scheduler: rename JobID to JobName to avoid overloading JobID meaning.

scheduler/appengine/engine/engine.go

Index: scheduler/appengine/engine/engine.go
diff --git a/scheduler/appengine/engine/engine.go b/scheduler/appengine/engine/engine.go
index 6aa52893e2ba00001d9ac372da1b84d74a6814f3..5a2e828ad78f4d07c05dd2e243e92015344f583e 100644
--- a/scheduler/appengine/engine/engine.go
+++ b/scheduler/appengine/engine/engine.go
@@ -257,6 +257,12 @@ type Job struct {
 	State JobState
 }
 
+func (e *Job) GetJobName() string {

[tandrii(chromium), 2017/06/21 16:16:07]

it seems you've meant to avoid public functions here, but it seems the logic of
splitting JobID really belongs to Job object itself.

+	// JobID has form <project>/<id>. Split it into components.
+	chunks := strings.Split(e.JobID, "/")
+	return chunks[1]
+}
+
 // effectiveSchedule returns schedule string to use for the job, considering its
 // Paused field.
 //