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

Enis Soztutar commented on HBASE-14123:
---------------------------------------

bq. This JIRA: https://issues.apache.org/jira/browse/HBASE-16825 Contains 2 
follow up sub tasks and 3 related JIRAs.
This came up before, but still the way that this effort is tracked is not 
correct. 
There are a couple of things that we are not following in the "standard way". 
This is important because otherwise reviewers like me cannot figure out what is 
committed and where, and what is to follow once the merge is complete. 

First, when an issue is resolved with a fixVersion set, it means that the patch 
itself is available in the branch that will be released as that version. 
Looking at all 9 subtasks of HBASE-16825, they are marked as fixed with 
fixVersion 2.0.0. This is wrong, since they are not committed in the master 
code (or anywhere for that matter). 
If an issue is committed to a branch, it should be resolved with a fixVersion 
equal to the branch name. Once the branch is merged, then we have the option to 
add fixVersions. 

Second, when doing patch iterations, we do not track "addressing comments" with 
individual jiras. In that sense, the first 9 subtasks of HBASE-HBASE-16825 
seems completely unnecessary. You should open a jira only in cases where the 
comments are not addressed, and we need a jira for tracking those changes 
afterwards. 

Third, looking at the Phase 3 jiras, bunch of them are in fixed state, again 
with fixVersion of 2.0.0. Are any of these jiras committed anywhere? Some of 
them I can find it in the branch. [~ted_yu] why are these jiras marked with 
2.0.0? 

Can you guys please mark all the jiras in the branch with the 
fixVersion=HBASE-7912 (and not 2.0.0). Once the merge, we can correct the 
fixVersions. 

Where is the jira for moving to the backup module? 

Can you please check or comment on the findbugs warnings and test failures. 
Also please do the write up (that I asked above) for the other reviewers.  

> HBase Backup/Restore Phase 2
> ----------------------------
>
>                 Key: HBASE-14123
>                 URL: https://issues.apache.org/jira/browse/HBASE-14123
>             Project: HBase
>          Issue Type: Umbrella
>            Reporter: Vladimir Rodionov
>            Assignee: Vladimir Rodionov
>            Priority: Blocker
>             Fix For: 2.0.0
>
>         Attachments: 14123-master.v14.txt, 14123-master.v15.txt, 
> 14123-master.v16.txt, 14123-master.v17.txt, 14123-master.v18.txt, 
> 14123-master.v19.txt, 14123-master.v20.txt, 14123-master.v21.txt, 
> 14123-master.v24.txt, 14123-master.v25.txt, 14123-master.v27.txt, 
> 14123-master.v28.txt, 14123-master.v29.full.txt, 14123-master.v2.txt, 
> 14123-master.v30.txt, 14123-master.v31.txt, 14123-master.v32.txt, 
> 14123-master.v33.txt, 14123-master.v34.txt, 14123-master.v35.txt, 
> 14123-master.v36.txt, 14123-master.v37.txt, 14123-master.v38.txt, 
> 14123.master.v39.patch, 14123-master.v3.txt, 14123.master.v40.patch, 
> 14123.master.v41.patch, 14123.master.v42.patch, 14123.master.v44.patch, 
> 14123.master.v45.patch, 14123.master.v46.patch, 14123.master.v48.patch, 
> 14123.master.v49.patch, 14123.master.v50.patch, 14123.master.v51.patch, 
> 14123.master.v52.patch, 14123.master.v54.patch, 14123.master.v56.patch, 
> 14123.master.v57.patch, 14123-master.v5.txt, 14123-master.v6.txt, 
> 14123-master.v7.txt, 14123-master.v8.txt, 14123-master.v9.txt, 14123-v14.txt, 
> HBASE-14123-for-7912-v1.patch, HBASE-14123-for-7912-v6.patch, 
> HBASE-14123-v10.patch, HBASE-14123-v11.patch, HBASE-14123-v12.patch, 
> HBASE-14123-v13.patch, HBASE-14123-v15.patch, HBASE-14123-v16.patch, 
> HBASE-14123-v1.patch, HBASE-14123-v2.patch, HBASE-14123-v3.patch, 
> HBASE-14123-v4.patch, HBASE-14123-v5.patch, HBASE-14123-v6.patch, 
> HBASE-14123-v7.patch, HBASE-14123-v9.patch
>
>
> Phase 2 umbrella JIRA. See HBASE-7912 for design document and description. 



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

Reply via email to