[
https://issues.apache.org/jira/browse/HBASE-18995?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16226094#comment-16226094
]
stack commented on HBASE-18995:
-------------------------------
The use case [~mdrob] ran into was CellUtil.createFirstOnRow. Its not in
CellUtil anymore [~ram_krish]
Here are other uses of CellUtil by Crunch:
{code}
./crunch-hbase/src/it/java/org/apache/crunch/io/hbase/HFileTargetIT.java:
Cell cell = CellUtil.createCell(Bytes.toBytes(w), Bytes.toBytes(c));
./crunch-hbase/src/main/java/org/apache/crunch/io/hbase/HFileOutputFormatForCrunch.java:
InetSocketAddress preferredNodeForRow =
regionLocationTable.getPreferredNodeForRow(CellUtil.cloneRow(cell));
./crunch-hbase/src/main/java/org/apache/crunch/io/hbase/HFileUtils.java:
return ByteBuffer.wrap(CellUtil.cloneRow(input));
./crunch-hbase/src/main/java/org/apache/crunch/io/hbase/HFileUtils.java:
LOG.debug("split row: " + Bytes.toString(CellUtil.cloneRow(kv)));
./crunch-hbase/src/main/java/org/apache/crunch/io/hbase/HFileUtils.java:
emitter.emit(ByteBuffer.wrap(CellUtil.cloneRow(foundCell)));
./crunch-spark/src/it/java/org/apache/crunch/SparkHFileTargetIT.java:
Cell cell = CellUtil.createCell(Bytes.toBytes(w), Bytes.toBytes(c));
{code}
> Move methods that are for internal usage from CellUtil to Private util class
> ----------------------------------------------------------------------------
>
> Key: HBASE-18995
> URL: https://issues.apache.org/jira/browse/HBASE-18995
> Project: HBase
> Issue Type: Sub-task
> Affects Versions: 2.0.0-alpha-3
> Reporter: ramkrishna.s.vasudevan
> Assignee: ramkrishna.s.vasudevan
> Priority: Critical
> Fix For: 2.0.0-alpha-4
>
> Attachments: HBASE-18995-branch-2.002.patch,
> HBASE-18995-branch-2.patch, HBASE-18995-branch-2_1.patch,
> HBASE-18995-branch-2_1.patch, HBASE-18995-branch-2_1.patch,
> HBASE-18995-branch-2_1.patch, HBASE-18995-branch-2_2.patch,
> HBASE-18995_002-branch-2.patch, HBASE-18995_002-branch-2.patch,
> HBASE-18995_003-branch-2.patch, HBASE-18995_1.patch, HBASE-18995_2.patch,
> HBASE-18995_2.patch
>
>
> This was brought up long time back. We need to move some of the public APIs
> from CellUtil to internal private Util class because they are used in some
> internal flow and does not make sense to have it in a @public exposed Util
> class.
> The topic again came in HBASE-18945 RB comments also.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
