[jira] [Commented] (GEODE-9221) Remove ByteArrayWrapper Usage From RedisString

["ASF subversion and git services (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GEODE-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17344635#comment-17344635 ] 

ASF subversion and git services commented on GEODE-9221:
--------------------------------------------------------

Commit cbda688f15c2e6a270a2f6041a70d648d58aa748 in geode's branch refs/heads/develop from Donal Evans
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=cbda688 ]

GEODE-9221: Remove uses of ByteArrayWrapper from RedisString (#6464)

 - Replace uses of ByteArrayWrapper in RedisString and NullRedisString
 with byte[]
 - Remove unused methods from ByteArrayWrapper
 - Populate RedisString.bitcountTable at initialization time via static method
 - Added integration tests for RedisString.bitpos
 - Refactor to remove warnings

Authored-by: Donal Evans <do...@vmware.com>

> Remove ByteArrayWrapper Usage From RedisString
> ----------------------------------------------
>
>                 Key: GEODE-9221
>                 URL: https://issues.apache.org/jira/browse/GEODE-9221
>             Project: Geode
>          Issue Type: Sub-task
>          Components: redis
>            Reporter: Wayne
>            Assignee: Donal Evans
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.15.0
>
>
> The classes RedisString and NullRedisString should be refactored to eliminate the use of ByteArrayWrapper and use byte[] instead.
>  
> When used as a key in a GeodeRegion, the new RedisKey class should be used as a key instead of byte[].
>   
> +Acceptance Criteria+
> After this refactor, we must verify that all existing regression tests run without failure.  Using the Redis Benchmarking tools, we expect an improvement in transaction throughput.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)