You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datasketches.apache.org by le...@apache.org on 2019/12/24 07:11:38 UTC

[incubator-datasketches-java] 03/10: restore to no checkReadOnly on DirectHllArray / putAuxHashMap().

This is an automated email from the ASF dual-hosted git repository.

leerho pushed a commit to branch NewHllIterator
in repository https://gitbox.apache.org/repos/asf/incubator-datasketches-java.git

commit b042a2e384304c48d81d8ffaf7cdca157be32b22
Author: Lee Rhodes <le...@users.noreply.github.com>
AuthorDate: Thu Dec 19 12:16:28 2019 -0800

    restore to no checkReadOnly on DirectHllArray / putAuxHashMap().
---
 src/main/java/org/apache/datasketches/hll/DirectHllArray.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/datasketches/hll/DirectHllArray.java b/src/main/java/org/apache/datasketches/hll/DirectHllArray.java
index 60e7566..60ebdae 100644
--- a/src/main/java/org/apache/datasketches/hll/DirectHllArray.java
+++ b/src/main/java/org/apache/datasketches/hll/DirectHllArray.java
@@ -189,7 +189,6 @@ abstract class DirectHllArray extends AbstractHllArray {
 
   @Override
   void putAuxHashMap(final AuxHashMap auxHashMap, final boolean compact) {
-    checkReadOnly(wmem);
     if (auxHashMap instanceof HeapAuxHashMap) {
       if (compact) {
         this.auxHashMap = auxHashMap; //heap and compact
@@ -202,7 +201,7 @@ abstract class DirectHllArray extends AbstractHllArray {
       }
     } else { //DirectAuxHashMap
       assert !compact; //must not be compact
-      this.auxHashMap = auxHashMap;
+      this.auxHashMap = auxHashMap; //In case of read-only this works.
     }
   }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datasketches.apache.org
For additional commands, e-mail: commits-help@datasketches.apache.org