You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tephra.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/02/12 07:55:42 UTC

[jira] [Commented] (TEPHRA-219) Setup proper security context in co-processor for compaction and flushes

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

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

GitHub user gokulavasan opened a pull request:

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

    (TEPHRA-219) Execute cross region calls in Coprocessor as the login user

    i) Fixes a problem where the pruneThread would exit if the state of the service is not yet set to RUNNING state. This also fixes the flakiness in the PruneUpperBoundSupplierTest which was caused due to the above problem.
    
    ii) Load the txMaxLifeTimeMillis and pruneEnable properties dynamically.
    
    iii) Add a hook around cross region calls in the PruneUpperBoundWriter.

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

    $ git pull https://github.com/gokulavasan/incubator-tephra feature/dynamic-loading-props

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

    https://github.com/apache/incubator-tephra/pull/35.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 #35
    
----
commit 5fd4c6f54f9c17500788f1ae7bea38be073729fc
Author: Gokul Gunasekaran <go...@cask.co>
Date:   2017-02-12T03:42:02Z

    Check all valid states

commit 4b7a837ae8584c18437984bf13579d31628f9487
Author: Gokul Gunasekaran <go...@cask.co>
Date:   2017-02-12T07:13:53Z

    Load properties dynamically

commit 2c8b8952025adaa7cef96c90a9d60ffcce233da4
Author: Gokul Gunasekaran <go...@cask.co>
Date:   2017-02-12T07:41:58Z

    TEPHRA-219 execute cross region calls in coprocessor as the login user

commit 9a120fe2db1a6e99f2d81dda00f99346a9cc4457
Author: Gokul Gunasekaran <go...@cask.co>
Date:   2017-02-12T07:54:45Z

    We can;t guarantee to reload pruneTable and pruneFlushInterval since we use single thread across all regions and thus the thread might be already running

----


> Setup proper security context in co-processor for compaction and flushes
> ------------------------------------------------------------------------
>
>                 Key: TEPHRA-219
>                 URL: https://issues.apache.org/jira/browse/TEPHRA-219
>             Project: Tephra
>          Issue Type: Sub-task
>            Reporter: Poorna Chandra
>            Assignee: Gokul Gunasekaran
>             Fix For: 0.11.0-incubating
>
>
> From PHOENIX-3037, HBASE-16115 and HBASE-16141 - HBase runs compaction/flush co-processor hooks as the user (current user) who started the compaction/flush. This becomes an issue when a co-processor makes cross region server calls in hooks. If the calls are made as the current user, then the call may fail since the current user may not have sufficient privileges to perform the call.
> Tephra TransactionProcessor will need to run all calls in compaction/flush hooks as the login user instead of the current user.



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