From 6310f31d435af8506d7bf28c85eef9c999d6dbd5 Mon Sep 17 00:00:00 2001
From: Danny Auble <da@llnl.gov>
Date: Fri, 3 Jun 2011 15:25:14 -0700
Subject: [PATCH] BLUEGENE - removed debug and fixed memory leak

---
 src/plugins/select/bluegene/runjob_plugin.cc | 39 ++++++++++----------
 1 file changed, 20 insertions(+), 19 deletions(-)

diff --git a/src/plugins/select/bluegene/runjob_plugin.cc b/src/plugins/select/bluegene/runjob_plugin.cc
index bb5c282f250..f5d276caf76 100644
--- a/src/plugins/select/bluegene/runjob_plugin.cc
+++ b/src/plugins/select/bluegene/runjob_plugin.cc
@@ -240,28 +240,29 @@ void Plugin::execute(bgsched::runjob::Verify& verify)
 		goto deny_job;
 	}
 
-	std::cout << "executable: " << verify.exe() << std::endl;
-	std::cout << "args      : ";
-	std::copy(verify.args().begin(), verify.args().end(),
-		  std::ostream_iterator<std::string>(std::cout, " "));
-	std::cout << std::endl;
+	// std::cout << "executable: " << verify.exe() << std::endl;
+	// std::cout << "args      : ";
+	// std::copy(verify.args().begin(), verify.args().end(),
+	// 	  std::ostream_iterator<std::string>(std::cout, " "));
+	// std::cout << std::endl;
 	// std::cout << "envs      : ";
 	// std::copy(verify.envs().begin(), verify.envs().end(),
 	// 	  std::ostream_iterator<std::string>(std::cout, " "));
 	// std::cout << std::endl;
-	std::cout << "block     : " << verify.block() << std::endl;
-	if (!verify.corner().location().empty()) {
-		std::cout << "corner:     " <<
-			verify.corner().location() << std::endl;
-	}
-	if (!verify.shape().value().empty()) {
-		std::cout << "shape:      " << verify.shape().value()
-			  << std::endl;
-	}
+	// std::cout << "block     : " << verify.block() << std::endl;
+	// if (!verify.corner().location().empty()) {
+	// 	std::cout << "corner:     " <<
+	// 		verify.corner().location() << std::endl;
+	// }
+	// if (!verify.shape().value().empty()) {
+	// 	std::cout << "shape:      " << verify.shape().value()
+	// 		  << std::endl;
+	// }
 
 	// const ProcessTree tree( verify.pid() );
 	// std::cout << tree << std::endl;
 
+	slurm_free_job_step_info_response_msg(step_resp);
 	return;
 
 deny_job:
@@ -273,16 +274,16 @@ deny_job:
 void Plugin::execute(const bgsched::runjob::Started& data)
 {
 	boost::lock_guard<boost::mutex> lock( _mutex );
-	std::cout << "runjob " << data.pid()
-		  << " started with ID " << data.job() << std::endl;
+	// std::cout << "runjob " << data.pid()
+	// 	  << " started with ID " << data.job() << std::endl;
 }
 
 void Plugin::execute(const bgsched::runjob::Terminated& data)
 {
 	boost::lock_guard<boost::mutex> lock( _mutex );
-	std::cout << "runjob " << data.pid() << " shadowing job "
-		  << data.job() << " finished with status "
-		  << data.status() << std::endl;
+	// std::cout << "runjob " << data.pid() << " shadowing job "
+	// 	  << data.job() << " finished with status "
+	// 	  << data.status() << std::endl;
 
 	// output failed nodes
 	const bgsched::runjob::Terminated::Nodes& nodes =
-- 
GitLab