[
https://issues.apache.org/jira/browse/GEODE-680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15062971#comment-15062971
]
ASF GitHub Bot commented on GEODE-680:
--------------------------------------
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-geode/pull/61
> Chunk class refactoring encountered while writing unit tests.
> -------------------------------------------------------------
>
> Key: GEODE-680
> URL: https://issues.apache.org/jira/browse/GEODE-680
> Project: Geode
> Issue Type: Improvement
> Components: core
> Reporter: Sai Boorlagadda
> Assignee: Sai Boorlagadda
>
> Good to have Chunk refactoring encountered while writing unit tests
> 1. Promote REF_COUNT_OFFSET to at-least protected access level and use it in
> GemFireChunkJUnitTest - search by REF_COUNT_OFFSET in the comments.
> 2. Promote MAX_REF_COUNT to at-least protected access level and use it in
> GemFireChunkJUnitTest.retainShouldThrowExceptionAfterMaxNumberOfTimesRetained
> 3. Promote IS_SERIALIZED_BIT & IS_COMPRESSED_BIT to at-least protected level
> and use it in GemFireChunkJUnitTest - search by IS_SERIALIZED_BIT in the
> comments
> 4. Chunk.getAddressForReading - can delete to Chunk.getUnsafeAddress, as both
> the methods are similar and `getUnsafeAddress` has more assert checks.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
