sandhyasun commented on a change in pull request #1840: [TRAFODION-3308] 
Improve error reporting when a process cannot be started
URL: https://github.com/apache/trafodion/pull/1840#discussion_r284512969
 
 

 ##########
 File path: core/sql/cli/ExSqlComp.cpp
 ##########
 @@ -172,7 +172,8 @@ ExSqlComp::ReturnStatus ExSqlComp::createServer()
   //  
   if (ret == ERROR)
   {
-       error(arkcmpErrorServer);
+       if ((!diagArea_->contains(-2013)) && (!diagArea_->contains(-2012))) // 
avoid generating redundant error
+         error(arkcmpErrorServer);
 
 Review comment:
   2012 is a retryable error. Will avoiding rgenerating it here cause a 
difference in behavior in createServer() ? 

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to