Github user asfgit closed the pull request at:
https://github.com/apache/trafodion/pull/1522
---
Github user selvaganesang commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1522#discussion_r182612555
--- Diff: core/sql/executor/ExHdfsScan.cpp ---
@@ -626,14 +625,23 @@ ExWorkProcRetcode ExHdfsScanTcb::work()
else
Github user DaveBirdsall commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1522#discussion_r182604952
--- Diff: core/sql/executor/ExHdfsScan.cpp ---
@@ -626,14 +625,23 @@ ExWorkProcRetcode ExHdfsScanTcb::work()
else
Github user selvaganesang commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1522#discussion_r182166447
--- Diff: core/sql/executor/ExExeUtilCli.cpp ---
@@ -1119,7 +1119,7 @@ Lng32 ExeCliInterface::executeImmediate(const char *
stmtStr,
{
GitHub user selvaganesang opened a pull request:
https://github.com/apache/trafodion/pull/1522
[TRAFODION-3009] Streamline error handling in Executor utility commands
Changes to avoid allocation of ComDiagsArea in some more places unless it is
needed to pass errors or warnings.