[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-20 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang updated HBASE-16261:
-
Description: 
aurelius
(1) HFileOutputFormat2 can read one table's region split keys. and then output 
multiple hfiles for one family, and each hfile map to one region. We can add 
partitioner in MultiHFileOutputFormat to make it support this feature.

(2) HFileOutputFormat2 support Customized Compression algorithm for column 
family and BloomFilter, also support customized DataBlockEncoding for the 
output hfiles. We can also make MultiHFileOutputFormat to support these 
features.

  was:

(1) HFileOutputFormat2 can read one table's region split keys. and then output 
multiple hfiles for one family, and each hfile map to one region. We can add 
partitioner in MultiHFileOutputFormat to make it support this feature.

(2) HFileOutputFormat2 support Customized Compression algorithm for column 
family and BloomFilter, also support customized DataBlockEncoding for the 
output hfiles. We can also make MultiHFileOutputFormat to support these 
features.


>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Fix For: 2.0.0
>
>
> aurelius
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-20 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang updated HBASE-16261:
-
Description: 

(1) HFileOutputFormat2 can read one table's region split keys. and then output 
multiple hfiles for one family, and each hfile map to one region. We can add 
partitioner in MultiHFileOutputFormat to make it support this feature.

(2) HFileOutputFormat2 support Customized Compression algorithm for column 
family and BloomFilter, also support customized DataBlockEncoding for the 
output hfiles. We can also make MultiHFileOutputFormat to support these 
features.

  was:
(1) HFileOutputFormat2 can read one table's region split keys. and then output 
multiple hfiles for one family, and each hfile map to one region. We can add 
partitioner in MultiHFileOutputFormat to make it support this feature.

(2) HFileOutputFormat2 support Customized Compression algorithm for column 
family and BloomFilter, also support customized DataBlockEncoding for the 
output hfiles. We can also make MultiHFileOutputFormat to support these 
features.


>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Fix For: 2.0.0
>
>
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-20 Thread yi liang (JIRA)
yi liang created HBASE-16261:


 Summary:  MultiHFileOutputFormat Enhancement 
 Key: HBASE-16261
 URL: https://issues.apache.org/jira/browse/HBASE-16261
 Project: HBase
  Issue Type: Sub-task
  Components: hbase
Affects Versions: 2.0.0
Reporter: yi liang
Assignee: yi liang
Priority: Minor


(1) HFileOutputFormat2 can read one table's region split keys. and then output 
multiple hfiles for one family, and each hfile map to one region. We can add 
partitioner in MultiHFileOutputFormat to make it support this feature.

(2) HFileOutputFormat2 support Customized Compression algorithm for column 
family and BloomFilter, also support customized DataBlockEncoding for the 
output hfiles. We can also make MultiHFileOutputFormat to support these 
features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-20 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang updated HBASE-16261:
-
Description: 
MultiHFileOutputFormat follow HFileOutputFormat2
(1) HFileOutputFormat2 can read one table's region split keys. and then output 
multiple hfiles for one family, and each hfile map to one region. We can add 
partitioner in MultiHFileOutputFormat to make it support this feature.

(2) HFileOutputFormat2 support Customized Compression algorithm for column 
family and BloomFilter, also support customized DataBlockEncoding for the 
output hfiles. We can also make MultiHFileOutputFormat to support these 
features.

  was:
aurelius
(1) HFileOutputFormat2 can read one table's region split keys. and then output 
multiple hfiles for one family, and each hfile map to one region. We can add 
partitioner in MultiHFileOutputFormat to make it support this feature.

(2) HFileOutputFormat2 support Customized Compression algorithm for column 
family and BloomFilter, also support customized DataBlockEncoding for the 
output hfiles. We can also make MultiHFileOutputFormat to support these 
features.


>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Fix For: 2.0.0
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-20 Thread yi liang (JIRA)

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

yi liang commented on HBASE-16261:
--

will provide patch shortly 

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Fix For: 2.0.0
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-20 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang updated HBASE-16261:
-
Attachment: HBASE-16261-V1.patch

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-20 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang updated HBASE-16261:
-
Status: Patch Available  (was: Open)

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-21 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang updated HBASE-16261:
-
Attachment: HBASE-16261-V2.patch

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch, HBASE-16261-V2.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-22 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16261:
-
Attachment: (was: HBASE-16261-V2.patch)

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch, HBASE-16261-V2.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-22 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16261:
-
Attachment: HBASE-16261-V2.patch

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch, HBASE-16261-V2.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-22 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16261:
-
Attachment: (was: HBASE-16261-V2.patch)

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-22 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16261:
-
Attachment: HBASE-16261-V2.patch

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch, HBASE-16261-V2.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-07-25 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16261:
-
Attachment: HBASE-16261-V3.patch

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch, HBASE-16261-V2.patch, 
> HBASE-16261-V3.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-25 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

Is it fine if I pick up this jira. I have done some researches in the code, and 
patch will be provided soon.

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 1.2.0
>Reporter: Harsh J
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-25 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16234:
-
Attachment: HBASE-16234-V1.patch

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 1.2.0
>Reporter: Harsh J
> Attachments: HBASE-16234-V1.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-25 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

replicaRegionsNotRecordedInMeta is used in 3 class: 

1.AssignmentManager(used when master start and assign all user regions)
2.EnableTableProcedure(used when master try to enable tables)
3.EnableTableHandler(used when master try to recover tables that are not fully 
moved to ENABLE state)


my idea is that if one table descriptor is null, we just skip this table and 
give some warning about which table is corrupted, so that user can use hbck to 
restore it, then we can enable or recover this table after restoring it.

And the patch is for master(2.0.0) branch


> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
> Attachments: HBASE-16234-V1.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-25 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16234:
-
 Assignee: Yi Liang
Affects Version/s: (was: 1.2.0)
   2.0.0
   Status: Patch Available  (was: Open)

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-26 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16234:
-
Attachment: HBASE-16234-V2.patch

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-26 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

Hi [~chenheng], 
thanks for reviewing, i just read through the AssignmentManager.java file. 
I found there are same problem at other places in this file, some place handle 
the exception, some place not (nullpointer not belong to IOException). So in 
V2, I fix all the similar problems and make all the fix consistency. Could you 
take a look, thanks.   

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-27 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16234:
-
Attachment: HBASE-16234-V3.patch

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch, HBASE-16234-V3.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-27 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

Agreed, We'd better to use IOException. Please take a look at v3, thanks 

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch, HBASE-16234-V3.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-28 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

As the code shows, if htd is null, then the loop will continue, I just give a 
warning.  You mean I should stop the loop?  

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch, HBASE-16234-V3.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-28 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

public static List replicaRegionsNotRecordedInMeta(
  Set regionsRecordedInMeta, MasterServices master)throws 
IOException {
List regionsNotRecordedInMeta = new ArrayList();
for (HRegionInfo hri : regionsRecordedInMeta) {
  TableName table = hri.getTable();
  // look at the HTD for the replica count. That's the source of truth
  int desiredRegionReplication = 1;
  try {
HTableDescriptor htd = master.getTableDescriptors().get(table);
if (htd == null) {
  LOG.warn("master can not get TableDescriptor from table '" + table);
} else {
  desiredRegionReplication = htd.getRegionReplication();
}
  } catch (IOException e){
LOG.warn("Couldn't get the replication attribute of the table " + table 
+ " due to "
+ e.getMessage());
  }
  for (int i = 0; i < desiredRegionReplication; i++) {
HRegionInfo replica = RegionReplicaUtil.getRegionInfoForReplica(hri, i);
if (regionsRecordedInMeta.contains(replica)) continue;
regionsNotRecordedInMeta.add(replica);
  }
}
return regionsNotRecordedInMeta;
  }


ok, i got your idea, in my code, i set desiredRegionReplication==1, so if 
htd==null, in the inner for() loop 
the method 
HRegionInfo replica = RegionReplicaUtil.getRegionInfoForReplica(hri, i);
will only get the primary replica, since every region should have primary 
replica, I just want to try at least whether primary replica go to the 
regionsNotRecordedInMeta list or not. 

But I think you solution is also good, if the descriptor is broken, we can just 
ignore this region and continue.
 
Which one you think is better?
and maybe we  can discuss this with [~jerryhe].

Thanks :)




> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch, HBASE-16234-V3.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-28 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

Hi [~chenheng], i just check the following code in the AssignmentManager.java, 
it seems that all the other place do not deal with the primary replica, all of 
them skip the primary replica. So you are right, we can continue directly,  I 
will modify the code. Thanks for your help.

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch, HBASE-16234-V3.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Issue Comment Deleted] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-28 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16234:
-
Comment: was deleted

(was: Hi [~chenheng], i just check the following code in the 
AssignmentManager.java, it seems that all the other place do not deal with the 
primary replica, all of them skip the primary replica. So you are right, we can 
continue directly,  I will modify the code. Thanks for your help.)

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch, HBASE-16234-V3.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-28 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16234:
-
Attachment: HBASE-16234-V4.patch

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch, HBASE-16234-V3.patch, HBASE-16234-V4.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-28 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

[~chenheng], I have made changes for 4 place in the java code. i think only the 
first place need to use continue, it seems that only in the first method, inner 
loop for (int i = 0; i < desiredRegionReplication; i++) , start from i=0, all 
the other 3 places, the inner for loop start from i=1. So I think only the 
first place need to use continue, the other places only need to keep it same as 
original code. What do you think?

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V1.patch, 
> HBASE-16234-V2.patch, HBASE-16234-V3.patch, HBASE-16234-V4.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-28 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16234:
-
Attachment: (was: HBASE-16234-V1.patch)

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V2.patch, 
> HBASE-16234-V3.patch, HBASE-16234-V4.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Assigned] (HBASE-15656) Fix unused protobuf warning in Admin.proto

2016-07-29 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15656?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang reassigned HBASE-15656:


Assignee: Yi Liang

> Fix unused protobuf warning in Admin.proto
> --
>
> Key: HBASE-15656
> URL: https://issues.apache.org/jira/browse/HBASE-15656
> Project: HBase
>  Issue Type: Task
>Reporter: Elliott Clark
>Assignee: Yi Liang
>Priority: Minor
>
> {code}
> Warning: Unused import: "Admin.proto" imports "Client.proto" which is not 
> used.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-15656) Fix unused protobuf warning in Admin.proto

2016-07-29 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-15656:
--

{code}
yis-macbook-pro:hbase-protocol yliang$ protoc -Isrc/main/protobuf 
--java_out=src/main/java src/main/protobuf/Admin.proto
[libprotobuf WARNING google/protobuf/descriptor.cc:5411] Warning: Unused 
import: "Admin.proto" imports "Client.proto" which is not used.
yis-macbook-pro:hbase-protocol yliang$ protoc -Isrc/main/protobuf 
--java_out=src/main/java src/main/protobuf/HBase.proto
[libprotobuf WARNING google/protobuf/descriptor.cc:5411] Warning: Unused 
import: "HBase.proto" imports "Cell.proto" which is not used.
yis-macbook-pro:hbase-protocol yliang$ protoc -Isrc/main/protobuf 
--java_out=src/main/java src/main/protobuf/SecureBulkLoad.proto
[libprotobuf WARNING google/protobuf/descriptor.cc:5411] Warning: Unused 
import: "SecureBulkLoad.proto" imports "HBase.proto" which is not used.
yis-macbook-pro:hbase-protocol yliang$ protoc -Isrc/main/protobuf 
--java_out=src/main/java src/main/protobuf/WAL.proto
[libprotobuf WARNING google/protobuf/descriptor.cc:5411] Warning: Unused 
import: "WAL.proto" imports "Client.proto" which is not used.
{code}


I have made changes for this 4 proto file

> Fix unused protobuf warning in Admin.proto
> --
>
> Key: HBASE-15656
> URL: https://issues.apache.org/jira/browse/HBASE-15656
> Project: HBase
>  Issue Type: Task
>Reporter: Elliott Clark
>Priority: Minor
>
> {code}
> Warning: Unused import: "Admin.proto" imports "Client.proto" which is not 
> used.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-15656) Fix unused protobuf warning in Admin.proto

2016-07-29 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15656?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-15656:
-
Attachment: HBASE-15656-V1.patch

> Fix unused protobuf warning in Admin.proto
> --
>
> Key: HBASE-15656
> URL: https://issues.apache.org/jira/browse/HBASE-15656
> Project: HBase
>  Issue Type: Task
>Reporter: Elliott Clark
>Assignee: Yi Liang
>Priority: Minor
> Attachments: HBASE-15656-V1.patch
>
>
> {code}
> Warning: Unused import: "Admin.proto" imports "Client.proto" which is not 
> used.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-15656) Fix unused protobuf warning in Admin.proto

2016-07-29 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15656?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-15656:
-
Fix Version/s: 2.0.0
   Status: Patch Available  (was: Open)

> Fix unused protobuf warning in Admin.proto
> --
>
> Key: HBASE-15656
> URL: https://issues.apache.org/jira/browse/HBASE-15656
> Project: HBase
>  Issue Type: Task
>Reporter: Elliott Clark
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-15656-V1.patch
>
>
> {code}
> Warning: Unused import: "Admin.proto" imports "Client.proto" which is not 
> used.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-15656) Fix unused protobuf warning in Admin.proto

2016-07-29 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15656?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-15656:
-
Affects Version/s: 2.0.0

> Fix unused protobuf warning in Admin.proto
> --
>
> Key: HBASE-15656
> URL: https://issues.apache.org/jira/browse/HBASE-15656
> Project: HBase
>  Issue Type: Task
>Affects Versions: 2.0.0
>Reporter: Elliott Clark
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-15656-V1.patch
>
>
> {code}
> Warning: Unused import: "Admin.proto" imports "Client.proto" which is not 
> used.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-30 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

ok i will break out a method for those newly added code, i will name it as  
getNumReplicas(TableName), which is suggested by @matteo . 

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V2.patch, 
> HBASE-16234-V3.patch, HBASE-16234-V4.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-30 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16234:
-
Attachment: HBASE-16234-V5.patch

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V2.patch, 
> HBASE-16234-V3.patch, HBASE-16234-V4.patch, HBASE-16234-V5.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16234) Expect and handle nulls when assigning replicas

2016-07-30 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16234:
--

patch V5 is attached, please take a look. Thanks :)

> Expect and handle nulls when assigning replicas
> ---
>
> Key: HBASE-16234
> URL: https://issues.apache.org/jira/browse/HBASE-16234
> Project: HBase
>  Issue Type: Bug
>  Components: Region Assignment
>Affects Versions: 2.0.0
>Reporter: Harsh J
>Assignee: Yi Liang
> Attachments: HBASE-16234-V1.patch, HBASE-16234-V2.patch, 
> HBASE-16234-V3.patch, HBASE-16234-V4.patch, HBASE-16234-V5.patch
>
>
> Observed this on a cluster:
> {code}
> FATAL org.apache.hadoop.hbase.master.HMaster: Unhandled exception. Starting 
> shutdown. 
> java.lang.NullPointerException 
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.replicaRegionsNotRecordedInMeta(AssignmentManager.java:2799)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.assignAllUserRegions(AssignmentManager.java:2778)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.processDeadServersAndRegionsInTransition(AssignmentManager.java:638)
>  
> at 
> org.apache.hadoop.hbase.master.AssignmentManager.joinCluster(AssignmentManager.java:485)
>  
> at 
> org.apache.hadoop.hbase.master.HMaster.finishActiveMasterInitialization(HMaster.java:723)
>  
> at org.apache.hadoop.hbase.master.HMaster.access$500(HMaster.java:169) 
> at org.apache.hadoop.hbase.master.HMaster$1.run(HMaster.java:1481) 
> at java.lang.Thread.run(Thread.java:745) 
> {code}
> It looks like {{FSTableDescriptors#get(…)}} can be expected to return null in 
> some cases, but {{AssignmentManager.replicaRegionsNotRecordedInMeta(…)}} does 
> not currently have any handling for such a possibility.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16353) Deprecate / Remove Get.isClosestRowBefore()

2016-08-04 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16353:
--

protobuf

> Deprecate / Remove Get.isClosestRowBefore() 
> 
>
> Key: HBASE-16353
> URL: https://issues.apache.org/jira/browse/HBASE-16353
> Project: HBase
>  Issue Type: Bug
>Reporter: Enis Soztutar
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0
>
>
> We no longer have "get closest row" functionality in 1.0+. We forgot to 
> remove Get.setClosestRowBefore() /  isClosestRowBefore and deprecate the 
> field in PB {{closest_row_before}}. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16353) Deprecate / Remove Get.isClosestRowBefore()

2016-08-04 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16353:
--

I have done some research in the code, found that 
1. in 2.0 branch
(1) All the getClosestRow related methods have been removed, except the one in 
HBaseTestingUtility. And that one in HBaseTestUtility is implemented using 
reversed scanner, so no need to remove this one. 
(2) I did not found any related filed in PB, it seems the below filed has been 
removed in 2.0   
 optional bool closest_row_before = 11 [default = false]; (which found in 1.4 
client.proto)

So, for 2.0 we just need to remove Get.setClosestRowBefore() / 
isClosestRowBefore, and in 1.4 we need more modify. 

I will first attach the patch for 2.0 now, and if patch for 1.4 is needed, I 
will attach it later. 




> Deprecate / Remove Get.isClosestRowBefore() 
> 
>
> Key: HBASE-16353
> URL: https://issues.apache.org/jira/browse/HBASE-16353
> Project: HBase
>  Issue Type: Bug
>Reporter: Enis Soztutar
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0
>
>
> We no longer have "get closest row" functionality in 1.0+. We forgot to 
> remove Get.setClosestRowBefore() /  isClosestRowBefore and deprecate the 
> field in PB {{closest_row_before}}. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16353) Deprecate / Remove Get.isClosestRowBefore()

2016-08-04 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16353?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16353:
-
Attachment: HBASE-16353-V1.patch

> Deprecate / Remove Get.isClosestRowBefore() 
> 
>
> Key: HBASE-16353
> URL: https://issues.apache.org/jira/browse/HBASE-16353
> Project: HBase
>  Issue Type: Bug
>Reporter: Enis Soztutar
>  Labels: beginner
> Fix For: 2.0.0
>
> Attachments: HBASE-16353-V1.patch
>
>
> We no longer have "get closest row" functionality in 1.0+. We forgot to 
> remove Get.setClosestRowBefore() /  isClosestRowBefore and deprecate the 
> field in PB {{closest_row_before}}. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16353) Deprecate / Remove Get.isClosestRowBefore()

2016-08-04 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16353?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16353:
-
 Assignee: Yi Liang
Fix Version/s: (was: 1.4.0)
   Status: Patch Available  (was: Open)

> Deprecate / Remove Get.isClosestRowBefore() 
> 
>
> Key: HBASE-16353
> URL: https://issues.apache.org/jira/browse/HBASE-16353
> Project: HBase
>  Issue Type: Bug
>Reporter: Enis Soztutar
>Assignee: Yi Liang
>  Labels: beginner
> Fix For: 2.0.0
>
> Attachments: HBASE-16353-V1.patch
>
>
> We no longer have "get closest row" functionality in 1.0+. We forgot to 
> remove Get.setClosestRowBefore() /  isClosestRowBefore and deprecate the 
> field in PB {{closest_row_before}}. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16353) Deprecate / Remove Get.isClosestRowBefore()

2016-08-04 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16353?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16353:
-
Fix Version/s: 1.4.0

> Deprecate / Remove Get.isClosestRowBefore() 
> 
>
> Key: HBASE-16353
> URL: https://issues.apache.org/jira/browse/HBASE-16353
> Project: HBase
>  Issue Type: Bug
>Reporter: Enis Soztutar
>Assignee: Yi Liang
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0
>
> Attachments: HBASE-16353-V1.patch
>
>
> We no longer have "get closest row" functionality in 1.0+. We forgot to 
> remove Get.setClosestRowBefore() /  isClosestRowBefore and deprecate the 
> field in PB {{closest_row_before}}. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16261) MultiHFileOutputFormat Enhancement

2016-08-08 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16261:
--

could any one help me to review the code, Thanks. :)

>  MultiHFileOutputFormat Enhancement 
> 
>
> Key: HBASE-16261
> URL: https://issues.apache.org/jira/browse/HBASE-16261
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16261-V1.patch, HBASE-16261-V2.patch, 
> HBASE-16261-V3.patch
>
>
> MultiHFileOutputFormat follow HFileOutputFormat2
> (1) HFileOutputFormat2 can read one table's region split keys. and then 
> output multiple hfiles for one family, and each hfile map to one region. We 
> can add partitioner in MultiHFileOutputFormat to make it support this feature.
> (2) HFileOutputFormat2 support Customized Compression algorithm for column 
> family and BloomFilter, also support customized DataBlockEncoding for the 
> output hfiles. We can also make MultiHFileOutputFormat to support these 
> features.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16385) Have hbase-rest pull hbase.rest.port from Constants.java

2016-08-10 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16385:
--

Hi [~dimaspivak],
   From [HBASE-3272|https://issues.apache.org/jira/browse/HBASE-3272], you can 
take a look at  the patch, the code used to get {{hbase.rest.port}} from 
Constant.java, but it have changed to hardcoded,  do you think we need to 
remove the {{DEFAULT_LISTEN_PORT}} in constant.java which no longer use, or 
change it back ?

> Have hbase-rest pull hbase.rest.port from Constants.java
> 
>
> Key: HBASE-16385
> URL: https://issues.apache.org/jira/browse/HBASE-16385
> Project: HBase
>  Issue Type: Improvement
>  Components: REST
>Reporter: Dima Spivak
>Priority: Minor
>  Labels: beginner
>
> [A while back|https://issues.apache.org/jira/browse/HBASE-3272], we 
> [hardcoded|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java#L266]
>  the {{hbase.rest.port}} instead of getting it from 
> [Constants.java|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/Constants.java#L36].
>  We should fix that.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Assigned] (HBASE-16385) Have hbase-rest pull hbase.rest.port from Constants.java

2016-08-10 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang reassigned HBASE-16385:


Assignee: Yi Liang

> Have hbase-rest pull hbase.rest.port from Constants.java
> 
>
> Key: HBASE-16385
> URL: https://issues.apache.org/jira/browse/HBASE-16385
> Project: HBase
>  Issue Type: Improvement
>  Components: REST
>Reporter: Dima Spivak
>Assignee: Yi Liang
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-16385-V1.patch
>
>
> [A while back|https://issues.apache.org/jira/browse/HBASE-3272], we 
> [hardcoded|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java#L266]
>  the {{hbase.rest.port}} instead of getting it from 
> [Constants.java|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/Constants.java#L36].
>  We should fix that.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16385) Have hbase-rest pull hbase.rest.port from Constants.java

2016-08-10 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16385:
-
Attachment: HBASE-16385-V1.patch

> Have hbase-rest pull hbase.rest.port from Constants.java
> 
>
> Key: HBASE-16385
> URL: https://issues.apache.org/jira/browse/HBASE-16385
> Project: HBase
>  Issue Type: Improvement
>  Components: REST
>Reporter: Dima Spivak
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-16385-V1.patch
>
>
> [A while back|https://issues.apache.org/jira/browse/HBASE-3272], we 
> [hardcoded|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java#L266]
>  the {{hbase.rest.port}} instead of getting it from 
> [Constants.java|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/Constants.java#L36].
>  We should fix that.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16385) Have hbase-rest pull hbase.rest.port from Constants.java

2016-08-10 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16385:
--

I provided a patch, please take a look, thanks :)

> Have hbase-rest pull hbase.rest.port from Constants.java
> 
>
> Key: HBASE-16385
> URL: https://issues.apache.org/jira/browse/HBASE-16385
> Project: HBase
>  Issue Type: Improvement
>  Components: REST
>Reporter: Dima Spivak
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-16385-V1.patch
>
>
> [A while back|https://issues.apache.org/jira/browse/HBASE-3272], we 
> [hardcoded|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java#L266]
>  the {{hbase.rest.port}} instead of getting it from 
> [Constants.java|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/Constants.java#L36].
>  We should fix that.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16385) Have hbase-rest pull hbase.rest.port from Constants.java

2016-08-10 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16385:
-
Status: Patch Available  (was: Open)

> Have hbase-rest pull hbase.rest.port from Constants.java
> 
>
> Key: HBASE-16385
> URL: https://issues.apache.org/jira/browse/HBASE-16385
> Project: HBase
>  Issue Type: Improvement
>  Components: REST
>Reporter: Dima Spivak
>Assignee: Yi Liang
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-16385-V1.patch
>
>
> [A while back|https://issues.apache.org/jira/browse/HBASE-3272], we 
> [hardcoded|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java#L266]
>  the {{hbase.rest.port}} instead of getting it from 
> [Constants.java|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/Constants.java#L36].
>  We should fix that.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16385) Have hbase-rest pull hbase.rest.port from Constants.java

2016-08-10 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16385:
-
Fix Version/s: 2.0.0

> Have hbase-rest pull hbase.rest.port from Constants.java
> 
>
> Key: HBASE-16385
> URL: https://issues.apache.org/jira/browse/HBASE-16385
> Project: HBase
>  Issue Type: Improvement
>  Components: REST
>Reporter: Dima Spivak
>Assignee: Yi Liang
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0
>
> Attachments: HBASE-16385-V1.patch
>
>
> [A while back|https://issues.apache.org/jira/browse/HBASE-3272], we 
> [hardcoded|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java#L266]
>  the {{hbase.rest.port}} instead of getting it from 
> [Constants.java|https://github.com/apache/hbase/blob/master/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/Constants.java#L36].
>  We should fix that.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-17 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

Hi [~apurtell],  
   I am now working on this jira, I think we had better add a new command like 
'deleterows' for this jira. Since deleteall is used for deleting all cells in a 
given row. It is only used for only one row, but if expend this command to 
delete multiple rows will make deleteall command not so consistency(one command 
has different functionalities), and the usage of this command will become a 
little complicated.  so I suggest adding a new command.

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Priority: Minor
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Assigned] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-17 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang reassigned HBASE-16381:


Assignee: Yi Liang

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16353) Deprecate / Remove Get.isClosestRowBefore()

2016-08-18 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16353:
--

@[~enis], are you ok with this patch?

> Deprecate / Remove Get.isClosestRowBefore() 
> 
>
> Key: HBASE-16353
> URL: https://issues.apache.org/jira/browse/HBASE-16353
> Project: HBase
>  Issue Type: Bug
>Reporter: Enis Soztutar
>Assignee: Yi Liang
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0
>
> Attachments: HBASE-16353-V1.patch
>
>
> We no longer have "get closest row" functionality in 1.0+. We forgot to 
> remove Get.setClosestRowBefore() /  isClosestRowBefore and deprecate the 
> field in PB {{closest_row_before}}. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-23 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16381:
-
Attachment: HBASE-16381-V1.patch

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Attachments: HBASE-16381-V1.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-23 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16381:
-
Fix Version/s: 2.0.0
   Status: Patch Available  (was: Open)

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-23 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

Hi [~apurtell],
   I have attached a patch, which I add a new command called 'deleterows', the 
usage is as below:

  hbase> deleterows 'ns1:t1', 'prefix' 
  hbase> deleterows 't1', 'prefix' // delete all cells 
for specific rows
  hbase> deleterows 't1', 'prefix', 'c1' // delete specific 
columns for specific rows
  hbase> deleterows 't1', 'prefix', 'c1', ts1  // delete specific 
columns for specific rows before ts1
  hbase> deleterows 't1', 'prefix', 'c1', {VISIBILITY=>'PRIVATE|SECRET'}  
  hbase> deleterows 't1', 'prefix', 'c1', ts1, {VISIBILITY=>'PRIVATE|SECRET'}  
//above two considering cell visibility



> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-23 Thread Yi Liang (JIRA)

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

Yi Liang edited comment on HBASE-16381 at 8/23/16 6:41 PM:
---

Hi [~apurtell],
   I have attached a patch, which I add a new command called 'deleterows', the 
usage is as below:
{code}
  hbase> deleterows 'ns1:t1', 'prefix' 
  hbase> deleterows 't1', 'prefix' // delete all cells 
for specific rows
  hbase> deleterows 't1', 'prefix', 'c1' // delete specific 
columns for specific rows
  hbase> deleterows 't1', 'prefix', 'c1', ts1  // delete specific 
columns for specific rows before ts1
  hbase> deleterows 't1', 'prefix', 'c1', {VISIBILITY=>'PRIVATE|SECRET'}  
  hbase> deleterows 't1', 'prefix', 'c1', ts1, {VISIBILITY=>'PRIVATE|SECRET'}  
//above two considering cell visibility
{code}



was (Author: easyliangjob):
Hi [~apurtell],
   I have attached a patch, which I add a new command called 'deleterows', the 
usage is as below:

  hbase> deleterows 'ns1:t1', 'prefix' 
  hbase> deleterows 't1', 'prefix' // delete all cells 
for specific rows
  hbase> deleterows 't1', 'prefix', 'c1' // delete specific 
columns for specific rows
  hbase> deleterows 't1', 'prefix', 'c1', ts1  // delete specific 
columns for specific rows before ts1
  hbase> deleterows 't1', 'prefix', 'c1', {VISIBILITY=>'PRIVATE|SECRET'}  
  hbase> deleterows 't1', 'prefix', 'c1', ts1, {VISIBILITY=>'PRIVATE|SECRET'}  
//above two considering cell visibility



> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-08-23 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16311:
-
Attachment: HBASE-16311-V1.patch

> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Reporter: Abhishek Kumar
>Priority: Minor
> Attachments: HBASE-16311-V1.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Assigned] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-08-23 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang reassigned HBASE-16311:


Assignee: Yi Liang

> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Attachments: HBASE-16311-V1.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-08-23 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16311:
--

I found that no activity for this jira for 3 weeks, so I take this jira and 
provide a patch.

in my patch, i finished two parts:
  (1)TODO part:
 log the delete_snapshot and list_snapshot info into audti log
   (2) log snapshot name into audit logs, 
Since 'AuthResult'(which is used to store audit log related information) 
class does not have field/variable for snapshot, so I just put the snapshot 
name into the 'request' filed in AuthResult. The output is showed below:
{code}
2016-08-23 16:29:47,423 TRACE 
SecurityLogger.org.apache.hadoop.hbase.security.access.AccessController: Access 
allowed for user hbase; reason: Global check allowed; remote address: 
/172.16.156.175; request: deleteSnapshot my-snapshot; context: (user=hbase, 
scope=GLOBAL, action=ADMIN)
{code}




> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Attachments: HBASE-16311-V1.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-08-23 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16311:
-
Fix Version/s: 2.0.0
Affects Version/s: 2.0.0
   Status: Patch Available  (was: Open)

> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Affects Versions: 2.0.0
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16311-V1.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)



[jira] [Updated] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-24 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16381:
-
Attachment: HBASE-16381-V2.patch

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-08-24 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16311:
-
Attachment: HBASE-16311-V2.patch

> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Affects Versions: 2.0.0
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16311-V1.patch, HBASE-16311-V2.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-08-24 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16311:
--

patch V2 carry Matteo's comments

> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Affects Versions: 2.0.0
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16311-V1.patch, HBASE-16311-V2.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-24 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

V2 fix some warnings and format issues. 

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-24 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

ping [~tedyu] [~chenheng] [~apurtell] [~stack],  could you guys take a look at 
this jira :), I think this is a useful feature.  Right now, I have some concerns
(1) do we need to add new command, usage seen last comment, 
(2) or just as Andrew suggest that we can expend this deleteall command

if expand this command I think we can replace the 'row' argument in deleteall 
command, shows as below.
{code}
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1'
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1', ts1
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1', ts1, 
{VISIBILITY=>'PRIVATE|SECRET'}
{code}

Or you guys have any other good solutions.


> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-24 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

Thanks for your feedback, the current deleteall is used to delete all cells in 
one row.  Just like what you said, to delete rows having 'prefix' in one cell, 
we can 

(1)deleterows 't1', 'prefix', 'c1'  > add new command
(2)deleteall 't1', {ROWPREFIXFILTER=>'prefix'}, c1 ==> extend old one

if we want to delete all rows in a cell, we may use '' empty string
(1) deleterows 't1', '', 'c1'
(2) deleteall 't1',{ROWPREFIXFILTER=>''}, 'c1'


But I think using new command is better, since deleteall is only for deleting 
one row, extend it to delete multiple rows can make this command not so 
consistent. 

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-24 Thread Yi Liang (JIRA)

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

Yi Liang edited comment on HBASE-16381 at 8/25/16 3:27 AM:
---

Thanks for your feedback, the current deleteall is used to delete all cells in 
one row.  Just like what you said, to delete rows having 'prefix' in one cell, 
we can either

(1)deleterows 't1', 'prefix', 'c1' 
(2)deleteall 't1', {ROWPREFIXFILTER=>'prefix'}, c1

if we want to delete all rows in a cell, we may use '' empty string
(1) deleterows 't1', '', 'c1'
(2) deleteall 't1',{ROWPREFIXFILTER=>''}, 'c1'


But I think using new command is better, since deleteall is only for deleting 
one row, extend it to delete multiple rows can make this command not so 
consistent. 


was (Author: easyliangjob):
Thanks for your feedback, the current deleteall is used to delete all cells in 
one row.  Just like what you said, to delete rows having 'prefix' in one cell, 
we can 

(1)deleterows 't1', 'prefix', 'c1'  > add new command
(2)deleteall 't1', {ROWPREFIXFILTER=>'prefix'}, c1 ==> extend old one

if we want to delete all rows in a cell, we may use '' empty string
(1) deleterows 't1', '', 'c1'
(2) deleteall 't1',{ROWPREFIXFILTER=>''}, 'c1'


But I think using new command is better, since deleteall is only for deleting 
one row, extend it to delete multiple rows can make this command not so 
consistent. 

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-08-24 Thread Yi Liang (JIRA)

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

Yi Liang edited comment on HBASE-16381 at 8/25/16 3:27 AM:
---

Thanks for your feedback, the current deleteall is used to delete all cells in 
one row.  Just like what you said, to delete rows having 'prefix' in one cell, 
we can either

(1)deleterows 't1', 'prefix', 'c1' 
(2)deleteall 't1', {ROWPREFIXFILTER=>'prefix'}, 'c1'

if we want to delete all rows in a cell, we may use '' empty string
(1) deleterows 't1', '', 'c1'
(2) deleteall 't1',{ROWPREFIXFILTER=>''}, 'c1'


But I think using new command is better, since deleteall is only for deleting 
one row, extend it to delete multiple rows can make this command not so 
consistent. 


was (Author: easyliangjob):
Thanks for your feedback, the current deleteall is used to delete all cells in 
one row.  Just like what you said, to delete rows having 'prefix' in one cell, 
we can either

(1)deleterows 't1', 'prefix', 'c1' 
(2)deleteall 't1', {ROWPREFIXFILTER=>'prefix'}, c1

if we want to delete all rows in a cell, we may use '' empty string
(1) deleterows 't1', '', 'c1'
(2) deleteall 't1',{ROWPREFIXFILTER=>''}, 'c1'


But I think using new command is better, since deleteall is only for deleting 
one row, extend it to delete multiple rows can make this command not so 
consistent. 

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16353) Deprecate / Remove Get.isClosestRowBefore()

2016-08-25 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16353:
--

Just review the related code, from the comments found that they will be removed 
in 3.0.0, so we can mark it as resolved later.
{code}
  /**
   * This will always return the default value which is false as client cannot 
set the value to this
   * property any more.
   * @deprecated since 2.0.0 and will be removed in 3.0.0
   */
  @Deprecated
  public boolean isClosestRowBefore() {
return closestRowBefore;
  }

  /**
   * This is not used any more and does nothing. Use reverse scan instead.
   * @deprecated since 2.0.0 and will be removed in 3.0.0
   */
  @Deprecated
  public Get setClosestRowBefore(boolean closestRowBefore) {
// do Nothing
return this;
  }
{code}

> Deprecate / Remove Get.isClosestRowBefore() 
> 
>
> Key: HBASE-16353
> URL: https://issues.apache.org/jira/browse/HBASE-16353
> Project: HBase
>  Issue Type: Bug
>Reporter: Enis Soztutar
>Assignee: Yi Liang
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0
>
> Attachments: HBASE-16353-V1.patch
>
>
> We no longer have "get closest row" functionality in 1.0+. We forgot to 
> remove Get.setClosestRowBefore() /  isClosestRowBefore and deprecate the 
> field in PB {{closest_row_before}}. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)



[jira] [Updated] (HBASE-16353) Deprecate / Remove Get.isClosestRowBefore()

2016-08-25 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16353?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16353:
-
Resolution: Later
Status: Resolved  (was: Patch Available)

> Deprecate / Remove Get.isClosestRowBefore() 
> 
>
> Key: HBASE-16353
> URL: https://issues.apache.org/jira/browse/HBASE-16353
> Project: HBase
>  Issue Type: Bug
>Reporter: Enis Soztutar
>Assignee: Yi Liang
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0
>
> Attachments: HBASE-16353-V1.patch
>
>
> We no longer have "get closest row" functionality in 1.0+. We forgot to 
> remove Get.setClosestRowBefore() /  isClosestRowBefore and deprecate the 
> field in PB {{closest_row_before}}. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-08-25 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16311:
--

[~mbertozzi], have a question about errors in the 
hadoop.hbase.security.access.TestAccessController 
in the test code, the 'snapshot' parameter in clone action has been set as 
'NULL' on purpose, that's why not pass test, but all other action such as 
listsnapshot, and deletesnapshot, their snapshot parameter are not set null.  
However, in its original code,all the Accesscontroller#preXXXSnapshot method 
have not deal with situation when 'snapshot' parameter is 'NULL'

code below is in the  hadoop.hbase.security.access.TestAccessController, where 
the programmer set snapshot name as null in cloneAction on purpose. 
{code}  
2052 AccessTestAction cloneAction = new AccessTestAction() {
2053  @Override
2054  public Object run() throws Exception {
2055
ACCESS_CONTROLLER.preCloneSnapshot(ObserverContext.createAndPrepare(CP_ENV, 
null),
2056  null, null);
2057return null;
2058  }
2059  };
{code}

code below is  in Accesscontroller#preXXXSnapshot,where the programmer did not 
deal with snapshot parameter as null, and all other preXXX are not deal with 
null snapshot as well
{code}
1335  @Override
1336  public void preCloneSnapshot(final 
ObserverContext ctx,
1337  final SnapshotDescription snapshot, final HTableDescriptor 
hTableDescriptor)
1338  throws IOException {
1339requirePermission(getActiveUser(ctx), "clone " + snapshot.getName(), 
Action.ADMIN);
1340  }
{code}

do you think I need to change the code to deal with null snapshot parameter, 
which has lot of works; or I just easily change the cloneAction parameter to a 
non-null snapshot in test code. 

Or do you know anybody who is familiar with this part of code, we can ask his 
advice. Thanks 



> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Affects Versions: 2.0.0
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16311-V1.patch, HBASE-16311-V2.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-08-25 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-15523:
-
Resolution: Invalid
Status: Resolved  (was: Patch Available)

> enhance hbase-daemon.sh to enable autorestart.
> --
>
> Key: HBASE-15523
> URL: https://issues.apache.org/jira/browse/HBASE-15523
> Project: HBase
>  Issue Type: Improvement
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Attachments: HBASE-15523.patch
>
>
> enhance hbase-daemon.sh to enable autorestart.
> component(like master, region server) will auto-start when terminated/killed 
> abnormally if 
>(a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e. 
>  export HBASE_AUTORESTART=true
>   (b) Then add the following 3 simple lines(59-61) to  
> /bin/hbase-daemon.sh
>  
>  51 # get arguments
>  52 startStop=$1
>  53 shift
>  54
>  55 command=$1
>  56 shift
>  57
>  58 #make sure the auto-restart are default settings
>  59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
> then 
>  60   startStop="autorestart" 
>  61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-03-23 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang resolved HBASE-15523.
--
Resolution: Fixed

> enhance hbase-daemon.sh to enable autorestart.
> --
>
> Key: HBASE-15523
> URL: https://issues.apache.org/jira/browse/HBASE-15523
> Project: HBase
>  Issue Type: Improvement
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
>
> enhance hbase-daemon.sh to enable autorestart.
> component(like master, region server) will auto-start when terminated/killed 
> abnormally if 
>(a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e. 
>  export HBASE_AUTORESTART=true
>   (b) Then add the following 3 simple lines(59-61) to  
> /bin/hbase-daemon.sh
>  
>  51 # get arguments
>  52 startStop=$1
>  53 shift
>  54
>  55 command=$1
>  56 shift
>  57
>  58 #make sure the auto-restart are default settings
>  59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
> then 
>  60   startStop="autorestart" 
>  61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-03-23 Thread yi liang (JIRA)
yi liang created HBASE-15523:


 Summary: enhance hbase-daemon.sh to enable autorestart.
 Key: HBASE-15523
 URL: https://issues.apache.org/jira/browse/HBASE-15523
 Project: HBase
  Issue Type: Improvement
Reporter: yi liang
Assignee: yi liang
Priority: Minor


enhance hbase-daemon.sh to enable autorestart.

component(like master, region server) will auto-start when terminated/killed 
abnormally if 
   (a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e.   
   export HBASE_AUTORESTART=true
  (b) Then add the following 3 simple lines(59-61) to  
/bin/hbase-daemon.sh
 
 51 # get arguments
 52 startStop=$1
 53 shift
 54
 55 command=$1
 56 shift
 57
 58 #make sure the auto-restart are default settings
 59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
then 
 60   startStop="autorestart" 
 61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Work started] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-03-23 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Work on HBASE-15523 started by yi liang.

> enhance hbase-daemon.sh to enable autorestart.
> --
>
> Key: HBASE-15523
> URL: https://issues.apache.org/jira/browse/HBASE-15523
> Project: HBase
>  Issue Type: Improvement
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
>
> enhance hbase-daemon.sh to enable autorestart.
> component(like master, region server) will auto-start when terminated/killed 
> abnormally if 
>(a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e. 
>  export HBASE_AUTORESTART=true
>   (b) Then add the following 3 simple lines(59-61) to  
> /bin/hbase-daemon.sh
>  
>  51 # get arguments
>  52 startStop=$1
>  53 shift
>  54
>  55 command=$1
>  56 shift
>  57
>  58 #make sure the auto-restart are default settings
>  59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
> then 
>  60   startStop="autorestart" 
>  61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-03-24 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang updated HBASE-15523:
-
Status: Patch Available  (was: In Progress)

> enhance hbase-daemon.sh to enable autorestart.
> --
>
> Key: HBASE-15523
> URL: https://issues.apache.org/jira/browse/HBASE-15523
> Project: HBase
>  Issue Type: Improvement
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Attachments: HBASE-15523.patch
>
>
> enhance hbase-daemon.sh to enable autorestart.
> component(like master, region server) will auto-start when terminated/killed 
> abnormally if 
>(a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e. 
>  export HBASE_AUTORESTART=true
>   (b) Then add the following 3 simple lines(59-61) to  
> /bin/hbase-daemon.sh
>  
>  51 # get arguments
>  52 startStop=$1
>  53 shift
>  54
>  55 command=$1
>  56 shift
>  57
>  58 #make sure the auto-restart are default settings
>  59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
> then 
>  60   startStop="autorestart" 
>  61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-03-24 Thread yi liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yi liang updated HBASE-15523:
-
Attachment: HBASE-15523.patch

> enhance hbase-daemon.sh to enable autorestart.
> --
>
> Key: HBASE-15523
> URL: https://issues.apache.org/jira/browse/HBASE-15523
> Project: HBase
>  Issue Type: Improvement
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Attachments: HBASE-15523.patch
>
>
> enhance hbase-daemon.sh to enable autorestart.
> component(like master, region server) will auto-start when terminated/killed 
> abnormally if 
>(a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e. 
>  export HBASE_AUTORESTART=true
>   (b) Then add the following 3 simple lines(59-61) to  
> /bin/hbase-daemon.sh
>  
>  51 # get arguments
>  52 startStop=$1
>  53 shift
>  54
>  55 command=$1
>  56 shift
>  57
>  58 #make sure the auto-restart are default settings
>  59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
> then 
>  60   startStop="autorestart" 
>  61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-03-24 Thread yi liang (JIRA)

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

yi liang commented on HBASE-15523:
--

hi Dima Spivak,
   This issue come from one of my customers, they use management console like 
Ambari-UI to manage the hbase, and start function of Hbase in Ambari-UI is 
hard-coded. So every time, when hbase component like region server terminated 
abnormally, they have to manually restart the HBase Component. they can not 
tolerate with it, so they want the flexibitliy to use an env variable they can 
input in the console to control the behavior.  

> enhance hbase-daemon.sh to enable autorestart.
> --
>
> Key: HBASE-15523
> URL: https://issues.apache.org/jira/browse/HBASE-15523
> Project: HBase
>  Issue Type: Improvement
>Reporter: yi liang
>Assignee: yi liang
>Priority: Minor
> Attachments: HBASE-15523.patch
>
>
> enhance hbase-daemon.sh to enable autorestart.
> component(like master, region server) will auto-start when terminated/killed 
> abnormally if 
>(a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e. 
>  export HBASE_AUTORESTART=true
>   (b) Then add the following 3 simple lines(59-61) to  
> /bin/hbase-daemon.sh
>  
>  51 # get arguments
>  52 startStop=$1
>  53 shift
>  54
>  55 command=$1
>  56 shift
>  57
>  58 #make sure the auto-restart are default settings
>  59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
> then 
>  60   startStop="autorestart" 
>  61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-08-30 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16311:
-
Attachment: HBASE-16311-V3.patch

> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Affects Versions: 2.0.0
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16311-V1.patch, HBASE-16311-V2.patch, 
> HBASE-16311-V3.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-01 Thread Yi Liang (JIRA)
Yi Liang created HBASE-16544:


 Summary: Remove or Clarify  'Using Amazon S3 Storage' section in 
the reference guide
 Key: HBASE-16544
 URL: https://issues.apache.org/jira/browse/HBASE-16544
 Project: HBase
  Issue Type: Bug
  Components: documentation, snapshots
Affects Versions: 2.0.0
Reporter: Yi Liang


reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration

(1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
think this title means that we can use S3 storage to replace HDFS, I really 
tried this :(,   always give me errors and hbase even can not start, see error 
mentioned in jira HBASE-11045.

(2) And the details in this section are more about deploying HBase on Amazon 
EC2 cluster, which has nothing to do 'using Amazon S3 storage'

(3) In all, I think we need to remove this section, or at least clarify this 
section if someone fully test HBase on S3.  see HBASE-15646 for more details 
about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-09-01 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16311:
--

thank for reviewing [~jerryhe], yes,  the snapshot owner may not have global 
Action.ADMIN permission, so we can just set it as null

> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Affects Versions: 2.0.0
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16311-V1.patch, HBASE-16311-V2.patch, 
> HBASE-16311-V3.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16311) Audit log for delete snapshot operation is missing in case of snapshot owner deleting the same

2016-09-01 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16311:
-
Attachment: HBASE-16311-V4.patch

> Audit log for delete snapshot operation is missing in case of snapshot owner 
> deleting the same
> --
>
> Key: HBASE-16311
> URL: https://issues.apache.org/jira/browse/HBASE-16311
> Project: HBase
>  Issue Type: Sub-task
>  Components: snapshots
>Affects Versions: 2.0.0
>Reporter: Abhishek Kumar
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16311-V1.patch, HBASE-16311-V2.patch, 
> HBASE-16311-V3.patch, HBASE-16311-V4.patch
>
>
> 1. Audit log seems to be left as a TODO task in AccessController.java:
> {code}
>   @Override
>   public void preDeleteSnapshot(final 
> ObserverContext ctx,
>   final SnapshotDescription snapshot) throws IOException {
> if (SnapshotDescriptionUtils.isSnapshotOwner(snapshot, getActiveUser())) {
>   // Snapshot owner is allowed to delete the snapshot
>   // TODO: We are not logging this for audit
> } else {
>   requirePermission("deleteSnapshot", Action.ADMIN);
> }
>   }
> {code}
> 2. Also, snapshot name is not getting logged in the audit logs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Assigned] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-01 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang reassigned HBASE-16544:


Assignee: Yi Liang

> Remove or Clarify  'Using Amazon S3 Storage' section in the reference guide
> ---
>
> Key: HBASE-16544
> URL: https://issues.apache.org/jira/browse/HBASE-16544
> Project: HBase
>  Issue Type: Bug
>  Components: documentation, snapshots
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
>
> reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration
> (1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
> think this title means that we can use S3 storage to replace HDFS, I really 
> tried this :(,   always give me errors and hbase even can not start, see 
> error mentioned in jira HBASE-11045.
> (2) And the details in this section are more about deploying HBase on Amazon 
> EC2 cluster, which has nothing to do 'using Amazon S3 storage'
> (3) In all, I think we need to remove this section, or at least clarify this 
> section if someone fully test HBase on S3.  see HBASE-15646 for more details 
> about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-01 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16544:
-
Component/s: (was: snapshots)
 Filesystem Integration

> Remove or Clarify  'Using Amazon S3 Storage' section in the reference guide
> ---
>
> Key: HBASE-16544
> URL: https://issues.apache.org/jira/browse/HBASE-16544
> Project: HBase
>  Issue Type: Bug
>  Components: documentation, Filesystem Integration
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
>
> reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration
> (1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
> think this title means that we can use S3 storage to replace HDFS, I really 
> tried this :(,   always give me errors and hbase even can not start, see 
> error mentioned in jira HBASE-11045.
> (2) And the details in this section are more about deploying HBase on Amazon 
> EC2 cluster, which has nothing to do 'using Amazon S3 storage'
> (3) In all, I think we need to remove this section, or at least clarify this 
> section if someone fully test HBase on S3.  see HBASE-15646 for more details 
> about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-02 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16544:
-
Attachment: HBASE-16544-V1.patch

> Remove or Clarify  'Using Amazon S3 Storage' section in the reference guide
> ---
>
> Key: HBASE-16544
> URL: https://issues.apache.org/jira/browse/HBASE-16544
> Project: HBase
>  Issue Type: Bug
>  Components: documentation, Filesystem Integration
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
> Attachments: HBASE-16544-V1.patch
>
>
> reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration
> (1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
> think this title means that we can use S3 storage to replace HDFS, I really 
> tried this :(,   always give me errors and hbase even can not start, see 
> error mentioned in jira HBASE-11045.
> (2) And the details in this section are more about deploying HBase on Amazon 
> EC2 cluster, which has nothing to do 'using Amazon S3 storage'
> (3) In all, I think we need to remove this section, or at least clarify this 
> section if someone fully test HBase on S3.  see HBASE-15646 for more details 
> about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-02 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16544:
-
Fix Version/s: 2.0.0
   Status: Patch Available  (was: Open)

> Remove or Clarify  'Using Amazon S3 Storage' section in the reference guide
> ---
>
> Key: HBASE-16544
> URL: https://issues.apache.org/jira/browse/HBASE-16544
> Project: HBase
>  Issue Type: Bug
>  Components: documentation, Filesystem Integration
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
> Fix For: 2.0.0
>
> Attachments: HBASE-16544-V1.patch
>
>
> reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration
> (1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
> think this title means that we can use S3 storage to replace HDFS, I really 
> tried this :(,   always give me errors and hbase even can not start, see 
> error mentioned in jira HBASE-11045.
> (2) And the details in this section are more about deploying HBase on Amazon 
> EC2 cluster, which has nothing to do 'using Amazon S3 storage'
> (3) In all, I think we need to remove this section, or at least clarify this 
> section if someone fully test HBase on S3.  see HBASE-15646 for more details 
> about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-02 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16544:
--

Provide a patch to remove the 'Using Amazon S3 Storage',  because, this title 
is confusing and hbase does not fully support Amazon S3, see HBASE-11045. And 
also 'Storing Snapshots in an Amazon S3' has been covered in section 138.9. no 
need to have 'using amazon s3 storage' this section in ref guide. [~misty],  
are you ok with this patch?

> Remove or Clarify  'Using Amazon S3 Storage' section in the reference guide
> ---
>
> Key: HBASE-16544
> URL: https://issues.apache.org/jira/browse/HBASE-16544
> Project: HBase
>  Issue Type: Bug
>  Components: documentation, Filesystem Integration
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
> Fix For: 2.0.0
>
> Attachments: HBASE-16544-V1.patch
>
>
> reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration
> (1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
> think this title means that we can use S3 storage to replace HDFS, I really 
> tried this :(,   always give me errors and hbase even can not start, see 
> error mentioned in jira HBASE-11045.
> (2) And the details in this section are more about deploying HBase on Amazon 
> EC2 cluster, which has nothing to do 'using Amazon S3 storage'
> (3) In all, I think we need to remove this section, or at least clarify this 
> section if someone fully test HBase on S3.  see HBASE-15646 for more details 
> about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-07 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16544:
-
Status: Open  (was: Patch Available)

> Remove or Clarify  'Using Amazon S3 Storage' section in the reference guide
> ---
>
> Key: HBASE-16544
> URL: https://issues.apache.org/jira/browse/HBASE-16544
> Project: HBase
>  Issue Type: Bug
>  Components: documentation, Filesystem Integration
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
> Fix For: 2.0.0
>
> Attachments: HBASE-16544-V1.patch
>
>
> reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration
> (1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
> think this title means that we can use S3 storage to replace HDFS, I really 
> tried this :(,   always give me errors and hbase even can not start, see 
> error mentioned in jira HBASE-11045.
> (2) And the details in this section are more about deploying HBase on Amazon 
> EC2 cluster, which has nothing to do 'using Amazon S3 storage'
> (3) In all, I think we need to remove this section, or at least clarify this 
> section if someone fully test HBase on S3.  see HBASE-15646 for more details 
> about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-07 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16544:
-
Status: Patch Available  (was: Open)

> Remove or Clarify  'Using Amazon S3 Storage' section in the reference guide
> ---
>
> Key: HBASE-16544
> URL: https://issues.apache.org/jira/browse/HBASE-16544
> Project: HBase
>  Issue Type: Bug
>  Components: documentation, Filesystem Integration
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
> Fix For: 2.0.0
>
> Attachments: HBASE-16544-V1.patch
>
>
> reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration
> (1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
> think this title means that we can use S3 storage to replace HDFS, I really 
> tried this :(,   always give me errors and hbase even can not start, see 
> error mentioned in jira HBASE-11045.
> (2) And the details in this section are more about deploying HBase on Amazon 
> EC2 cluster, which has nothing to do 'using Amazon S3 storage'
> (3) In all, I think we need to remove this section, or at least clarify this 
> section if someone fully test HBase on S3.  see HBASE-15646 for more details 
> about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-09 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16381:
-
Attachment: HBASE-16381-V3.patch

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-09 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

yes, agree with Jerry. You can see from below code, the delete also use 
deleteall's implementation, there maybe some reason why two similar delete 
commands could exist.  
{code}
# Delete a cell
def _delete_internal(row, column,
timestamp = 
org.apache.hadoop.hbase.HConstants::LATEST_TIMESTAMP, args = {})
  _deleteall_internal(row, column, timestamp, args)
end
{code}

I provide a patch that extend deleteall to support row prefix.
the usage is also below, 
{code}
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1'//delete 
certain cf in the row ranges
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1', ts1
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1', ts1, 
{VISIBILITY=>'PRIVATE|SECRET'}
{code}

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-09 Thread Yi Liang (JIRA)

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

Yi Liang edited comment on HBASE-16381 at 9/9/16 5:35 PM:
--

yes, agree with Jerry. You can see from below code, the delete also use 
deleteall's implementation, there maybe some reason why two similar delete 
commands could co-exist.  
{code}
# Delete a cell
def _delete_internal(row, column,
timestamp = 
org.apache.hadoop.hbase.HConstants::LATEST_TIMESTAMP, args = {})
  _deleteall_internal(row, column, timestamp, args)
end
{code}

I provide a patch that extend deleteall to support row prefix.
the usage is also below, 
{code}
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1'//delete 
certain cf in the row ranges
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1', ts1
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1', ts1, 
{VISIBILITY=>'PRIVATE|SECRET'}
{code}


was (Author: easyliangjob):
yes, agree with Jerry. You can see from below code, the delete also use 
deleteall's implementation, there maybe some reason why two similar delete 
commands could exist.  
{code}
# Delete a cell
def _delete_internal(row, column,
timestamp = 
org.apache.hadoop.hbase.HConstants::LATEST_TIMESTAMP, args = {})
  _deleteall_internal(row, column, timestamp, args)
end
{code}

I provide a patch that extend deleteall to support row prefix.
the usage is also below, 
{code}
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1'//delete 
certain cf in the row ranges
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1', ts1
  hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix'}, 'c1', ts1, 
{VISIBILITY=>'PRIVATE|SECRET'}
{code}

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-09 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

yes, when we want to delete multiple rows,  we just need to replace 'row' 
parameter with {ROWPREFIXFILTER => 'prefix'}, this is the usage. 

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-12 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16381:
-
Attachment: HBASE-16381-V4.patch

Add test in hbase shell tests

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch, HBASE-16381-V4.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-13 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

yes, i will change it to reuse this list

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch, HBASE-16381-V4.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-13 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16381:
-
Attachment: HBASE-16381-V5.patch

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch, HBASE-16381-V4.patch, HBASE-16381-V5.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-14 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16381?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16381:
-
Attachment: HBASE-16381-V6.patch

Carry Chen Heng's comments

{code}
 hbase> deleteall 't1', {ROWPREFIXFILTER => 'prefix', CACHE => 100}
{code}

CACHE is optional, and is used to specify how many deletes batched to be sent 
to server at one time, default is 100

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch, HBASE-16381-V4.patch, HBASE-16381-V5.patch, 
> HBASE-16381-V6.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-16381) Shell deleteall command should support row key prefixes

2016-09-15 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-16381:
--

Thanks for committing [~chenheng] and thanks for reviewing [~jerryhe]

> Shell deleteall command should support row key prefixes
> ---
>
> Key: HBASE-16381
> URL: https://issues.apache.org/jira/browse/HBASE-16381
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Andrew Purtell
>Assignee: Yi Liang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16381-V1.patch, HBASE-16381-V2.patch, 
> HBASE-16381-V3.patch, HBASE-16381-V4.patch, HBASE-16381-V5.patch, 
> HBASE-16381-V6.patch
>
>
> The shell's deleteall command should support deleting a row range using a row 
> key prefix. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Reopened] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-09-15 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-15523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang reopened HBASE-15523:
--

> enhance hbase-daemon.sh to enable autorestart.
> --
>
> Key: HBASE-15523
> URL: https://issues.apache.org/jira/browse/HBASE-15523
> Project: HBase
>  Issue Type: Improvement
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Attachments: HBASE-15523.patch
>
>
> enhance hbase-daemon.sh to enable autorestart.
> component(like master, region server) will auto-start when terminated/killed 
> abnormally if 
>(a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e. 
>  export HBASE_AUTORESTART=true
>   (b) Then add the following 3 simple lines(59-61) to  
> /bin/hbase-daemon.sh
>  
>  51 # get arguments
>  52 startStop=$1
>  53 shift
>  54
>  55 command=$1
>  56 shift
>  57
>  58 #make sure the auto-restart are default settings
>  59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
> then 
>  60   startStop="autorestart" 
>  61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-15523) enhance hbase-daemon.sh to enable autorestart.

2016-09-16 Thread Yi Liang (JIRA)

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

Yi Liang commented on HBASE-15523:
--

I closed this one on Aug by accident, I reopen it to see if anyone have any 
other suggestion or idea of this proposal. From my point of view,  the 
autostart command is also a kind of extension for start command, we need 
somehow extend start command to support this autostart, not have a new command 
for autostart. 

> enhance hbase-daemon.sh to enable autorestart.
> --
>
> Key: HBASE-15523
> URL: https://issues.apache.org/jira/browse/HBASE-15523
> Project: HBase
>  Issue Type: Improvement
>Reporter: Yi Liang
>Assignee: Yi Liang
>Priority: Minor
> Attachments: HBASE-15523.patch
>
>
> enhance hbase-daemon.sh to enable autorestart.
> component(like master, region server) will auto-start when terminated/killed 
> abnormally if 
>(a) Add a new env variable $HBASE_AUTORESTART to hbase-env.sh i.e. 
>  export HBASE_AUTORESTART=true
>   (b) Then add the following 3 simple lines(59-61) to  
> /bin/hbase-daemon.sh
>  
>  51 # get arguments
>  52 startStop=$1
>  53 shift
>  54
>  55 command=$1
>  56 shift
>  57
>  58 #make sure the auto-restart are default settings
>  59 if [ "$HBASE_AUTORESTART" == "true" ] && [ "$startStop" == "start" ]; 
> then 
>  60   startStop="autorestart" 
>  61 fi



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-16544) Remove or Clarify 'Using Amazon S3 Storage' section in the reference guide

2016-09-16 Thread Yi Liang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-16544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yi Liang updated HBASE-16544:
-
Attachment: HBASE-16544-addendum.patch

forget to remove links to the removed section 'using amazon s3'. Provide 
addendum to remove it.

> Remove or Clarify  'Using Amazon S3 Storage' section in the reference guide
> ---
>
> Key: HBASE-16544
> URL: https://issues.apache.org/jira/browse/HBASE-16544
> Project: HBase
>  Issue Type: Bug
>  Components: documentation, Filesystem Integration
>Affects Versions: 2.0.0
>Reporter: Yi Liang
>Assignee: Yi Liang
> Fix For: 2.0.0
>
> Attachments: HBASE-16544-V1.patch, HBASE-16544-addendum.patch
>
>
> reference guide at https://hbase.apache.org/book.html#amazon_s3_configuration
> (1) the title 'Using Amazon S3 Storage' is confusing.From my point of view, I 
> think this title means that we can use S3 storage to replace HDFS, I really 
> tried this :(,   always give me errors and hbase even can not start, see 
> error mentioned in jira HBASE-11045.
> (2) And the details in this section are more about deploying HBase on Amazon 
> EC2 cluster, which has nothing to do 'using Amazon S3 storage'
> (3) In all, I think we need to remove this section, or at least clarify this 
> section if someone fully test HBase on S3.  see HBASE-15646 for more details 
> about this doc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


  1   2   3   4   5   6   7   >