[
https://issues.apache.org/jira/browse/PHOENIX-2295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14959248#comment-14959248
]
Samarth Jain commented on PHOENIX-2295:
---
+1, LGTM. I will get this committed.
>
[
https://issues.apache.org/jira/browse/PHOENIX-2295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14959366#comment-14959366
]
Samarth Jain commented on PHOENIX-2295:
---
Pushed to 4.x and master branches. Thanks for the patch,
[
https://issues.apache.org/jira/browse/PHOENIX-2295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14947742#comment-14947742
]
Alicia Ying Shu commented on PHOENIX-2295:
--
[~samarthjain] For this particular case, there is
[
https://issues.apache.org/jira/browse/PHOENIX-2295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14946019#comment-14946019
]
Samarth Jain commented on PHOENIX-2295:
---
Patch looks good [~ayingshu]. A few suggestions:
1) Use
[
https://issues.apache.org/jira/browse/PHOENIX-2295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14945380#comment-14945380
]
Alicia Ying Shu commented on PHOENIX-2295:
--
Attached patch.
> NullPointerException when