On 22-08-2016 13:52, Aline Manera wrote:
Hi Lucio,
Hi,
Task is an internal mechanism to Wok and its plugins execute long time
processing. So that is not part of user knowledge.
User log functionality is for
admins, not regular users, right?
Adding a log record as Task will make the user confused IMO.
I don't think so, have you seen my patches running? I've changed the UI
too. It's pretty clear IMHO.
So my proposal is: when the Task is started in the backend, add a log
record and set its status as 'pending'
Once the Task is completed (with success or not), change the log record
to success or failure according to what happened there.
To do that you will need to find a way to query a log entry in the JSON
file and edit/replace it to the updated entry.
What do you think about it?
To do that we will also need to differentiate in wok/control/base.py the
type of request (if it generates task or not) and I tried to avoid that
with my design.
Also, I believe the way I proposed to do it makes it more clear for the
guy debugging through logs what's happening behind the scenes.
So, IMHO, it will increase code complexity to differentiate tasks
through some variable, handle it in base.py, code for changing log
entries, for no real gain.
Regards,
Aline Manera
Regards,
--
Lucio Correia
Software Engineer
IBM LTC Brazil