[GitHub] trafodion pull request #1405: [TRAFODION-2873] Removed usage of system new f...

[DaveBirdsall]

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

    https://github.com/apache/trafodion/pull/1405#discussion_r162492113
  
    --- Diff: core/sql/cli/Cli.cpp ---
    @@ -9611,12 +9611,28 @@ Lng32 SQLCLI_LOBddlInterface
            
          } // for
     
    +        //Initialize LOB interface 
    +        
    +        Int32 rc= 
ExpLOBoper::initLOBglobal(exLobGlob,currContext.exHeap(),&currContext,hdfsServer,hdfsPort);
    +        if (rc)
    +          {
    +            {
    --- End diff --
    
    It doesn't hurt anything, but I'm curious: Why two sets of braces? Looks 
like one would do?


---