From b097022bba81dba7e82af4481537a7f877ad2a42 Mon Sep 17 00:00:00 2001
From: Moe Jette <jette1@llnl.gov>
Date: Thu, 15 Dec 2005 22:53:18 +0000
Subject: [PATCH] Clarify comment. No change in logic.

---
 src/slurmctld/job_mgr.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/slurmctld/job_mgr.c b/src/slurmctld/job_mgr.c
index 9ad5542ebc6..8e2da4b6166 100644
--- a/src/slurmctld/job_mgr.c
+++ b/src/slurmctld/job_mgr.c
@@ -2704,10 +2704,14 @@ void pack_job(struct job_record *dump_job_ptr, Buf buffer)
 	select_g_pack_jobinfo(dump_job_ptr->select_jobinfo, buffer);
 
 	detail_ptr = dump_job_ptr->details;
-	if (detail_ptr)		/* pack "features" for prolog/epilog use */
+	/* A few details are always dumped here, others are only dumped 
+	 * until the job starts running (at which time they become 
+	 * meaningless) */
+	if (detail_ptr) {
 		packstr(detail_ptr->features, buffer);
-	else
+	} else {
 		packstr(NULL, buffer);
+	}
 
 	if (detail_ptr && dump_job_ptr->job_state == JOB_PENDING)
 		_pack_job_details(detail_ptr, buffer);
-- 
GitLab