You are viewing a plain text version of this content. The canonical link for it is here.
Posted to codereview@trafodion.apache.org by anoopsharma00 <gi...@git.apache.org> on 2017/04/14 19:45:30 UTC

[GitHub] incubator-trafodion pull request #1066: [TRAFODION-2584] Add support to regi...

GitHub user anoopsharma00 opened a pull request:

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

    [TRAFODION-2584] Add support to register hive objects in traf metadata

    Syntax:
      register hive table/view [if not exists] <object-name> [cascade]
      unregister hive table/view  [if exists] <object-name> [cascade]
      cascade option: register/unregister all underlying objects that are
                      part of the specified view
    
    update statistics, grant/revoke, traf views or external table creation
    on hive objects will automatically and internally register those objects
    in trafodion metadata.
    
    invoke/showddl will show if this object is registered and whether
    that registration was internal or explicit.
    
    Get command extensions:
      get hive registered tables/view/objects in catalog trafodion;
      get hive external tables in catalog trafodion;
    
    Cleanup command extensions:
      cleanup metadata command will clean up  inconsistent hive objects
       (underlying hive object is missing but object is registered or
        an external table exists)
    
       cleanup [hive table | hive view] on <object-name>;
    
    Existing hive objects that had implicit or explicit external tables
    created prior to this checkin will have no change in behavior.
    ObjectUID of those external tables will continue to be used for
    relevant operations.
    One can drop those external tables and explicitly register them,
    or a subsequent operation(upd stats, grant, etc) that needs objectUID will
    automatically register them.
    
    minor changes based on review comments of previous checkin
    
    get all objects   command on hive metadata no longer fails.
    get views on objects   return 3-part name that could be used to
      differentiate between a traf and hive view.
    
    regress/hive/TEST007 has been extended.
    
    TBD: Add register/unregister privileges

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

    $ git pull https://github.com/anoopsharma00/incubator-trafodion ansharma_hivereg_br_deliver

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

    https://github.com/apache/incubator-trafodion/pull/1066.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 #1066
    
----
commit 9c58649f930a799f03a2703d012371e4e66fa1a2
Author: Anoop Sharma <an...@esgyn.com>
Date:   2017-04-14T19:38:18Z

    [TRAFODION-2584] Add support to register hive objects in traf metadata
    
    Syntax:
      register hive table/view [if not exists] <object-name> [cascade]
      unregister hive table/view  [if exists] <object-name> [cascade]
      cascade option: register/unregister all underlying objects that are
                      part of the specified view
    
    update statistics, grant/revoke, traf views or external table creation
    on hive objects will automatically and internally register those objects
    in trafodion metadata.
    
    invoke/showddl will show if this object is registered and whether
    that registration was internal or explicit.
    
    Get command extensions:
      get hive registered tables/view/objects in catalog trafodion;
      get hive external tables in catalog trafodion;
    
    Cleanup command extensions:
      cleanup metadata command will clean up  inconsistent hive objects
       (underlying hive object is missing but object is registered or
        an external table exists)
    
       cleanup [hive table | hive view] on <object-name>;
    
    Existing hive objects that had implicit or explicit external tables
    created prior to this checkin will have no change in behavior.
    ObjectUID of those external tables will continue to be used for
    relevant operations.
    One can drop those external tables and explicitly register them,
    or a subsequent operation(upd stats, grant, etc) that needs objectUID will
    automatically register them.
    
    minor changes based on review comments of previous checkin
    
    get all objects   command on hive metadata no longer fails.
    get views on objects   return 3-part name that could be used to
      differentiate between a traf and hive view.
    
    regress/hive/TEST007 has been extended.
    
    TBD: Add register/unregister privileges

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1066: [TRAFODION-2584] Add support to regi...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---