You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by ashutosh-bapat <gi...@git.apache.org> on 2018/10/11 05:42:38 UTC

[GitHub] hive pull request #445: HIVE-20542

GitHub user ashutosh-bapat opened a pull request:

    https://github.com/apache/hive/pull/445

    HIVE-20542

    Changes for HIVE-20542. There are three separate commits, with each commit message explaining purpose of that commit. They all should be pulled together as a single commit into Apache hive repository. 

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

    $ git pull https://github.com/ashutosh-bapat/hive hive20542

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

    https://github.com/apache/hive/pull/445.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 #445
    
----
commit e76dd0d51662d5e1bc1890010b95d305f2505ea8
Author: Ashutosh Bapat <as...@...>
Date:   2018-10-01T05:17:06Z

    HIVE-20542: Insert NULL value for columns of NOTIFICATION_LOG for which values are not available
    
    When no database is associated with an event we insert 'null' as database name in the metastore.
    With this commit, we insert NULL as database name.
    
    When no tablename is associated with an event we insert an empty string as table name in the
    metastore. With this commit, we insert NULL as table name.
    
    Even if a catalog name is associated with an event, addNotificationLog() doesn't insert catalog in
    the metastore. With this commit we take care of that as well.
    
    Ashutosh Bapat.

commit 31aee469baffb95641fb68f70a6bcaa0ca725d28
Author: Ashutosh Bapat <as...@...>
Date:   2018-10-01T06:11:15Z

    HIVE-20542: Modify query used to count the number of events to be replicated incrementally
    
    The query used to count the events for a given incremental replication does not
    1. Count event with NULL database, table or catalog names.
    2. Does not consider toEventId and Limit for the given incremental replication.
    
    Ashutosh Bapat.

commit d3a319c5fd347018572c13b40e1e8f7cdbe72050
Author: Ashutosh Bapat <as...@...>
Date:   2018-10-04T04:08:44Z

    HIVE-20644: Add tests for testing getNotificationEventsCount().
    
    Ashutosh Bapat.

----


---

[GitHub] hive pull request #445: HIVE-20542

Posted by ashutosh-bapat <gi...@git.apache.org>.

Github user ashutosh-bapat closed the pull request at:

    https://github.com/apache/hive/pull/445


---