[
https://issues.apache.org/jira/browse/HBASE-17239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15718136#comment-15718136
]
Anoop Sam John commented on HBASE-17239:
----------------------------------------
bq.static CodedInputStream newInstance(ByteInput buf, boolean
bufferIsImmutable) {
This method is not really useful now. Just remove it? Can fix on commit. U
will have to remove from 2 files in patch.
+1
> Add UnsafeByteOperations#wrap(ByteInput, int offset, int len) API
> -----------------------------------------------------------------
>
> Key: HBASE-17239
> URL: https://issues.apache.org/jira/browse/HBASE-17239
> Project: HBase
> Issue Type: Improvement
> Components: Protobufs
> Affects Versions: 2.0.0
> Reporter: ramkrishna.s.vasudevan
> Assignee: ramkrishna.s.vasudevan
> Fix For: 2.0.0
>
> Attachments: HBASE-17239_V1.patch, HBASE-17239_V2.patch
>
>
> Instead of exposing the newInstance(ByteInput, boolean) in CIS we could just
> add a UnsafeByteOperations#wrap(ByteInput, offset, len). And we could just
> call that and do a #newcodedInput() over that. So internally we do return a
> immutable version of the ByteInput only. This way we can avoid
> CIS#newInstance(ByteInput) exposure and can keep it package private as done
> in COS.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
