<div dir="ltr">Hi Trevor<div>I am huichun </div><div><br></div><div>I agree with you, currently sahara edp engine and DB mixed with lots of oozieness info, abstract incompletely.</div><div><br></div><div>i find this issue because i want add recurrence edp job in sahara, and with oozie implementation, i find i need these oozie information not only the "status" value. </div><div><br></div><div>so i will make a little change with current code with little impact, but in the long term plan,  I think we should make a disscussion about refactoring current edp engine and oozie implementation</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-06-23 21:17 GMT+08:00 Trevor McKay <span dir="ltr"><<a href="mailto:tmckay@redhat.com" target="_blank">tmckay@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Lu,<br>
<br>
  yes, you're right.  Return is a dictionary and for the other EDP<br>
engines only "status" is returned (and we primarily care about<br>
"status").  For Oozie, there is more information.<br>
<br>
  I'm fine with changing the name to "get_job_info()" throughout the<br>
job_manager and EDP.<br>
<br>
  It actually raises the question for me about whether or not in the<br>
Oozie case we really even need the extra Oozie information in the Sahara<br>
database.  I don't think we use it anywhere, not even sure the UI<br>
displays it (but it might) or how much comes through the REST responses.<br>
<br>
  Maybe we should have get_job_status() which returns only "status", and<br>
an optional get_job_info() that returns more? But that may be a bigger<br>
discussion.<br>
<br>
Best,<br>
<br>
Trevor<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, 2015-06-23 at 15:18 +0800, lu jander wrote:<br>
> Hi Trevor<br>
><br>
><br>
> in sahara oozie engine (sahara/service/edp/oozie/engine.py<br>
> sahara/service/edp/oozie/oozie.py)<br>
><br>
><br>
> function get_job_status actually returns not only the status of the<br>
> job, but it returns all the info about the job, so i think that we<br>
> should  rename this function as get_job_info maybe more convenient for<br>
> us? cause I want add a function named get_job_info but i find that it<br>
> already exists here with a confused name.<br>
<br>
<br>
</div></div></blockquote></div><br></div>