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

ASF GitHub Bot commented on TEPHRA-263:
---------------------------------------

GitHub user anwar6953 opened a pull request:

    https://github.com/apache/incubator-tephra/pull/64

    TEPHRA-263 Fix test cases and TTL enforcement

    TEPHRA-263 Fix test cases and TTL enforcement for tables configured to read 
non transactional data.

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

    $ git pull https://github.com/anwar6953/incubator-tephra 
bugfix_release/TEPHRA-263

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

    https://github.com/apache/incubator-tephra/pull/64.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 #64
    
----
commit 4a6cb30de7573ab33a47e6199286d8e9db3adf43
Author: Ali Anwar <[email protected]>
Date:   2017-09-22T00:08:02Z

    TEPHRA-263 Fix test cases and TTL enforcement for tables configured to read 
non transactional data.

----


> TTL is not strictly enforced, if there are long transactions running
> --------------------------------------------------------------------
>
>                 Key: TEPHRA-263
>                 URL: https://issues.apache.org/jira/browse/TEPHRA-263
>             Project: Tephra
>          Issue Type: Bug
>            Reporter: Ali Anwar
>            Assignee: Ali Anwar
>             Fix For: 0.14.0-incubating
>
>
> The logic for filtering for TTL:
> https://github.com/apache/incubator-tephra/blob/release/0.12.0-incubating/tephra-core/src/main/java/org/apache/tephra/util/TxUtils.java#L66
> It is subtracting the TTL duration from the visibility upper bound, but it 
> should be subtracting from the current time or the current write pointer 
> instead. Otherwise, if the TTL is 1 hour, but the visibility upper bound is 
> 22 hours ago (due to some MR that has been in progress for 22 hours), then 
> the TTL that is actually enforced will be 23 hours and older data will be 
> filtered.
> After the long transactions are invalidated, the TTL is then strictly 
> enforced.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to