[
https://issues.apache.org/jira/browse/KYLIN-2612?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16008565#comment-16008565
]
Ted Yu commented on KYLIN-2612:
-------------------------------
Similar check should be added in LegacyAclService as well.
> Potential NPE accessing familyMap in AclTableMigrationTool#getAllAceInfo
> ------------------------------------------------------------------------
>
> Key: KYLIN-2612
> URL: https://issues.apache.org/jira/browse/KYLIN-2612
> Project: Kylin
> Issue Type: Bug
> Reporter: Ted Yu
> Priority: Minor
>
> Here is related code:
> {code}
> NavigableMap<byte[], byte[]> familyMap =
> result.getFamilyMap(Bytes.toBytes(AclHBaseStorage.ACL_ACES_FAMILY));
> for (Map.Entry<byte[], byte[]> entry : familyMap.entrySet()) {
> {code}
> Good practice is to check whether familyMap is null / empty first.
> See the following class in hbase for example:
> hbase-client/src/main/java/org/apache/hadoop/hbase/quotas/QuotaTableUtil.java
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
