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

    https://github.com/apache/trafodion/pull/1723#discussion_r227473870
  
    --- 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 new code is cleaner.  If it isn't a seabase table, then we need to get 
privileges, otherwise they are passed in with the TrafDesc.


---

Reply via email to