[ 
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)

Reply via email to