You are viewing a plain text version of this content. The canonical link for it is here.

Posted to codereview@trafodion.apache.org by selvaganesang <gi...@git.apache.org> on 2017/09/14 01:15:59 UTC

[GitHub] incubator-trafodion pull request #1231: [Trafodion 2739] RMS semaphore handl...

Github user selvaganesang commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/1231#discussion_r138777511
  
    --- Diff: core/sql/cli/Cli.cpp ---
    @@ -8325,19 +8325,15 @@ Lng32 SQLCLI_RegisterQuery(CliGlobals *cliGlobals,
       StatsGlobals *statsGlobals = cliGlobals->getStatsGlobals();
       if (statsGlobals == NULL)
         return retcode;
    -  short error;
    -  short savedPriority, savedStopMode;
    +  int error;
       error = statsGlobals->getStatsSemaphore(cliGlobals->getSemId(),
    -                 cliGlobals->myPin(), 
    -                savedPriority, savedStopMode, FALSE /*shouldTimeout*/);
    -  ex_assert(error == 0, "getStatsSemaphore() returned an error");
    -
    +                 cliGlobals->myPin());
    --- End diff --
    
    Any error would have asserted in StatsGlobals::getStatsSemaphore() method itself


---