You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sa...@apache.org on 2018/07/17 21:13:04 UTC

[29/48] lucene-solr:jira/lucene-2562: LUCENE-8398: TieredMergePolicy.getMaxMergedSegmentMB has rounding error

LUCENE-8398: TieredMergePolicy.getMaxMergedSegmentMB has rounding error


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/f5c8af1e
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/f5c8af1e
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/f5c8af1e

Branch: refs/heads/jira/lucene-2562
Commit: f5c8af1e873ea61c428401757f2da792dbfa7aa8
Parents: f7da358
Author: Erick Erickson <er...@apache.org>
Authored: Mon Jul 16 08:12:35 2018 -0700
Committer: Steve Rowe <sa...@apache.org>
Committed: Tue Jul 17 12:22:31 2018 -0400

----------------------------------------------------------------------
 lucene/CHANGES.txt                                                | 3 +++
 .../core/src/java/org/apache/lucene/index/TieredMergePolicy.java  | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/f5c8af1e/lucene/CHANGES.txt
----------------------------------------------------------------------
diff --git a/lucene/CHANGES.txt b/lucene/CHANGES.txt
index b5335c8..f6958fd 100644
--- a/lucene/CHANGES.txt
+++ b/lucene/CHANGES.txt
@@ -160,6 +160,9 @@ Bug Fixes:
   TokenStream if a token consisting entirely of delimiter characters was 
   encountered, but preserve_original was set. (Alan Woodward)
 
+
+* LUCENE-8398: TieredMergePolicy.getMaxMergedSegmentMB has rounding error (Erick Erickson)
+
 Changes in Runtime Behavior:
 
 * LUCENE-7976: TieredMergePolicy now respects maxSegmentSizeMB by default when executing

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/f5c8af1e/lucene/core/src/java/org/apache/lucene/index/TieredMergePolicy.java
----------------------------------------------------------------------
diff --git a/lucene/core/src/java/org/apache/lucene/index/TieredMergePolicy.java b/lucene/core/src/java/org/apache/lucene/index/TieredMergePolicy.java
index 7f917bb..c60f578 100644
--- a/lucene/core/src/java/org/apache/lucene/index/TieredMergePolicy.java
+++ b/lucene/core/src/java/org/apache/lucene/index/TieredMergePolicy.java
@@ -192,7 +192,7 @@ public class TieredMergePolicy extends MergePolicy {
    *
    * @see #setMaxMergedSegmentMB */
   public double getMaxMergedSegmentMB() {
-    return maxMergedSegmentBytes/1024/1024.;
+    return maxMergedSegmentBytes/1024.0/1024.0;
   }
 
   /** Controls how aggressively merges that reclaim more