[jira] [Commented] (TRAFODION-2603) Remove obsolete utility commands, turn off obsolete privileges

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TRAFODION-2603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15993869#comment-15993869 ] 

ASF GitHub Bot commented on TRAFODION-2603:
-------------------------------------------

GitHub user DaveBirdsall opened a pull request:

    https://github.com/apache/incubator-trafodion/pull/1080

    [TRAFODION-2603] Remove obsolete utilities and component privileges

    This set of changes does the following:
    
    1. Removes the PUBLISH and UNPUBLISH utility statements. They are from a predecessor product, and do not work in Trafodion. And there is no specific requirement for them in Trafodion.
    
    2. Removes the INITIALIZE TRAFODION, CREATE SEQUENCE utility statement. Now that the SEQ_GEN table is automatically created as part of the metadata schema, this utility is no longer needed.
    
    3. Changes GRANT COMPONENT PRIVILEGE to give an unsupported statement error (SQL code -1010) if one of the DML_* privileges is given. They don't work, and there is consideration being given to removing them.
    
    Note: If you upgrade your development instance to include this change, you may experience a make failure complaining about the lack of file parser/StmtDDLPublish.h.
    
    The problem is that parser make dependency files still include a reference to this file. "make clean" does not delete the dependency files. (Maybe it should!)
    
    To recover from this make failure, do the following:
    
    cd $TRAF_HOME/../sql/nskgmake/parser/linux/64bit/debug/depend
    
    rm *
    
    cd $TRAF_HOME/../sql/parser
    
    touch sqlparser.y
    
    These commands remove the make dependency files for the parser, and tell make to build the parser file. When you next do a "make all", the dependency files will be regenerated.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DaveBirdsall/incubator-trafodion RemoveOldUtils

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-trafodion/pull/1080.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1080
    
----

----


> Remove obsolete utility commands, turn off obsolete privileges
> --------------------------------------------------------------
>
>                 Key: TRAFODION-2603
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-2603
>             Project: Apache Trafodion
>          Issue Type: Improvement
>          Components: sql-cmp
>    Affects Versions: 2.2-incubating
>         Environment: All
>            Reporter: David Wayne Birdsall
>            Assignee: David Wayne Birdsall
>
> The PUBLISH and UNPUBLISH commands were a feature in the predecessor product and do not work in Trafodion. The specific need for them does not exist in Trafodion, so these can be removed.
> The INITIALIZE TRAFODION, CREATE SEQUENCE utility was used to create the SEQ_GEN table in the metadata to support sequences. That table is now created as part of INITIALIZE TRAFODION, so there is no longer a need for the CREATE SEQUENCE option. It can be removed.
> The DML_* component privileges allow the Root ID to grant various DML privileges to any user, giving them the ability to do a given DML operation on any Trafodion object. They don't work at the present. And the wisdom of having such component privileges is debatable. For now, we would like to add code that simply raises an error and says these privileges are not supported if an attempt is made to grant them.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)