[ https://issues.apache.org/jira/browse/HBASE-18554?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124539#comment-16124539 ]
Reid Chan edited comment on HBASE-18554 at 8/12/17 10:50 AM: ------------------------------------------------------------- Remove {code@SuppressWarnings("unchecked")} for it's no longer unchecked; add a row check for a passed cell and ignore it if fails the check; no unit test provided. was (Author: reidchan): Remove {code:@SuppressWarnings("unchecked")} for it's no longer unchecked; add a row check for a passed cell and ignore it if fails the check; no unit test provided. > Append#add doesn't check the row of passed cell > ----------------------------------------------- > > Key: HBASE-18554 > URL: https://issues.apache.org/jira/browse/HBASE-18554 > Project: HBase > Issue Type: Bug > Reporter: Chia-Ping Tsai > Labels: beginner > Attachments: HBASE-18554.master.001.patch > > > {code} > @SuppressWarnings("unchecked") > public Append add(final Cell cell) { > // Presume it is KeyValue for now. > byte [] family = CellUtil.cloneFamily(cell); > List<Cell> list = this.familyMap.get(family); > if (list == null) { > list = new ArrayList<>(1); > } > // find where the new entry should be placed in the List > list.add(cell); > this.familyMap.put(family, list); > return this; > } > {code} -- This message was sent by Atlassian JIRA (v6.4.14#64029)