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

    https://github.com/apache/trafodion/pull/1612#discussion_r198638476
  
    --- Diff: core/sql/cli/SessionDefaults.cpp ---
    @@ -108,6 +108,7 @@ static const SessionDefaults::SessionDefaultMap 
sessionDefaultMap[] =
       SDEntry(SessionDefaults::SCHEMA,                   SCHEMA,               
      SessionDefaults::SDT_ASCII,          TRUE,    TRUE,  FALSE, FALSE),
       SDEntry(SessionDefaults::STATISTICS_VIEW_TYPE,     STATISTICS_VIEW_TYPE, 
      SessionDefaults::SDT_ASCII,          FALSE,   FALSE, TRUE,  TRUE),
       SDEntry(SessionDefaults::SUSPEND_LOGGING,          SUSPEND_LOGGING,      
      SessionDefaults::SDT_BOOLEAN,        FALSE,   FALSE, TRUE,  FALSE),
    +  SDEntry(SessionDefaults::USE_LIBHDFS,              USE_LIBHDFS,          
      SessionDefaults::SDT_BOOLEAN,        TRUE,    TRUE,  FALSE, FALSE),
    --- End diff --
    
    The update and extract LOB commands were not adhering to USE_LIBHDFS 
settings. This has been fixed in commit ee44ad9. However, all the DDL lob 
functions executed via the CLI call SQL_EXEC_LOBddlInterface will not adhere to 
USE_LIBHDFS settings if the CQD is set in defaults table.  Instead the CQD 
should be set via DCS profile till we fix the issue of transforming CQD into 
SSD for the CQDs in defaults table.  Because it is is DDLinterface, it should 
be ok even if it couldn't switch to older implementation.


---

Reply via email to