Github user DaveBirdsall commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1723#discussion_r225695077
--- Diff: core/sql/optimizer/NATable.cpp ---
@@ -6805,46 +6809,94 @@ void NATable::getPrivileges(TrafDesc * priv_desc)
ComSecurityKeySet secKeyVec(heap_);
if (priv_desc == NULL)
{
- if (isHiveTable() || isHbaseCellTable() ||
- isHbaseRowTable() || isHbaseMapTable())
+ if (!isSeabaseTable())
--- End diff --
The old code would return if this "if" were true, but the new code does not
(because the return statement after the "else" was placed inside braces. Was
this intentional?
---
