[GitHub] [kylin] nichunen commented on issue #1026: delete /execute_output metadata at the same time when deleting a job

[GitBox <gi...@apache.org>]

nichunen commented on issue #1026: delete /execute_output metadata at the same time when deleting a job
URL: https://github.com/apache/kylin/pull/1026#issuecomment-569616496
 
 
   @ChangjiGuo 
   Hi, I think your point of this PR is right, job's tasks' should also be deleted once the job is deleted (they will be deleted by MetadataCleanupJob). 
   But I doubt the code change will also bring instability of the job deleting operation. Because before this change, dropping a job only needs one time of metadata deleting, but after this change, it needs n+1 times(n is the number of tasks). If one task failed to be deleted, an exception will be thrown which indicates the deletion is failed, but actually, the job's metadata has been deleted.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services