Github user sandhyasun commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1612#discussion_r198346030
--- 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 --
I am confused by this now. Are we saying if we set the CQD USE_LIBHDFS to
ON, the code today will not revert to the old behavior ? Given several POCs
are using this feature and usually under a time crunch, it will not be
practical provide a patch. I think we need a mechanism to revert to the old
interface for 2 reasons. We also need to run performance tests with very large
LOBs. China team is waiting for this and need a way to turn this feature off
and on for sure. Maybe envvar is the way to go if this CQD/SSD is only partial.
---
