[GitHub] [incubator-dolphinscheduler] Technoboy- edited a comment on issue #1519: return result state for each interface method

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

Technoboy- edited a comment on issue #1519: return result state for each interface method
URL: https://github.com/apache/incubator-dolphinscheduler/pull/1519#issuecomment-567380815
 
 
   > > > > Hi, bro, thanks for contribution.
   > > > > this pr looks not useful, it we add return value for these methods, we should use the return value to decide something. such as ： if add method return false, we may log or retry, but doing nothing . These methods you modify have printed the log, it's enough .
   > > > > So do you agree me ?
   > > > 
   > > > 
   > > > Yes, I agree. I don't know **the context**. Those who know their own context should handle this.
   > > > But the interface should has the return value to indicate whether the operation is successed. All exceptions now has been passed, without any alert to interface API user.
   > > 
   > > 
   > > Yes. But add return value according to executing without error or not is not accurate.
   > 
   > FYI
   
   I mean :  return true is not accurate when executing without exception .  srem or sadd may return false due to zk not has the relative value .  it's like -- update in MySQL, the return value represents for the influence count.  

----------------------------------------------------------------
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