You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by jp...@apache.org on 2020/02/14 12:42:03 UTC

[lucene-solr] 01/02: LUCENE-9187: remove too-expensive assert from LZ4 HighCompressionHashTable

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

jpountz pushed a commit to branch branch_8x
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git

commit da33e4aa6f92ee622a9605636bc5fe1c105f1f94
Author: Robert Muir <rm...@apache.org>
AuthorDate: Tue Jan 28 11:45:43 2020 -0500

    LUCENE-9187: remove too-expensive assert from LZ4 HighCompressionHashTable
---
 lucene/core/src/java/org/apache/lucene/util/compress/LZ4.java | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/lucene/core/src/java/org/apache/lucene/util/compress/LZ4.java b/lucene/core/src/java/org/apache/lucene/util/compress/LZ4.java
index 1201faa..946ca3a 100644
--- a/lucene/core/src/java/org/apache/lucene/util/compress/LZ4.java
+++ b/lucene/core/src/java/org/apache/lucene/util/compress/LZ4.java
@@ -291,13 +291,6 @@ public final class LZ4 {
       Arrays.fill(chainTable, (short) 0xFFFF);
     }
 
-    private boolean assertReset() {
-      for (int i = 0; i < chainTable.length; ++i) {
-        assert chainTable[i] == (short) 0xFFFF : i;
-      }
-      return true;
-    }
-
     @Override
     void reset(byte[] bytes, int off, int len) {
       FutureObjects.checkFromIndexSize(off, len, bytes.length);
@@ -320,7 +313,6 @@ public final class LZ4 {
         Arrays.fill(hashTable, -1);
         Arrays.fill(chainTable, (short) 0xFFFF);
       }
-      assert assertReset();
       this.bytes = bytes;
       this.base = off;
       this.next = off;