You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucenenet.apache.org by sy...@apache.org on 2014/12/30 07:14:42 UTC

[1/3] lucenenet git commit: throw a more informative exception to track down TeamCity test failures

Repository: lucenenet
Updated Branches:
  refs/heads/master cb4bbe094 -> 2bf20847f


throw a more informative exception to track down TeamCity test failures


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

Branch: refs/heads/master
Commit: a57d7201e74ba53784dffb2e18be2e2646618da3
Parents: cb4bbe0
Author: Laimonas Simutis <la...@gmail.com>
Authored: Mon Dec 29 20:34:20 2014 -0500
Committer: Laimonas Simutis <la...@gmail.com>
Committed: Mon Dec 29 20:34:20 2014 -0500

----------------------------------------------------------------------
 src/Lucene.Net.TestFramework/Util/Paths.cs | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucenenet/blob/a57d7201/src/Lucene.Net.TestFramework/Util/Paths.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.TestFramework/Util/Paths.cs b/src/Lucene.Net.TestFramework/Util/Paths.cs
index 108efe1..2685305 100644
--- a/src/Lucene.Net.TestFramework/Util/Paths.cs
+++ b/src/Lucene.Net.TestFramework/Util/Paths.cs
@@ -139,10 +139,23 @@ namespace Lucene.Net.Util
                     // where [Section] is either core, demo, or contrib, and [Build] is either Debug or Release.
                     string assemblyLocation = AssemblyDirectory;
                     int index = -1;
+
+                    var buildPathPart = Path.DirectorySeparatorChar + "build" + Path.DirectorySeparatorChar;
+                    var binPathPart = Path.DirectorySeparatorChar + "bin" + Path.DirectorySeparatorChar;
+
                     if (assemblyLocation.IndexOf("build", StringComparison.InvariantCultureIgnoreCase) > -1)
-                        index = assemblyLocation.IndexOf(Path.DirectorySeparatorChar + "build" + Path.DirectorySeparatorChar, StringComparison.InvariantCultureIgnoreCase);
+                    {
+                        index = assemblyLocation.IndexOf(buildPathPart, StringComparison.InvariantCultureIgnoreCase);
+                    }
                     else
-                        index = assemblyLocation.IndexOf(Path.DirectorySeparatorChar + "bin" + Path.DirectorySeparatorChar, StringComparison.InvariantCultureIgnoreCase);
+                    {
+                        index = assemblyLocation.IndexOf(binPathPart, StringComparison.InvariantCultureIgnoreCase);
+                    }
+
+                    if (index < 0)
+                    {
+                        throw new ArgumentOutOfRangeException("Could not locate project root directory in " + assemblyLocation + ", checked " + buildPathPart + " and " + binPathPart);
+                    }
 
                     int difference = assemblyLocation.Substring(index).Count(o => o == Path.DirectorySeparatorChar);
 


[2/3] lucenenet git commit: fixes for several dictionary lookups

Posted by sy...@apache.org.
fixes for several dictionary lookups


Project: http://git-wip-us.apache.org/repos/asf/lucenenet/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucenenet/commit/6f88e8aa
Tree: http://git-wip-us.apache.org/repos/asf/lucenenet/tree/6f88e8aa
Diff: http://git-wip-us.apache.org/repos/asf/lucenenet/diff/6f88e8aa

Branch: refs/heads/master
Commit: 6f88e8aa2ff096b8d8e2059ee15e20f6a40921ab
Parents: cb4bbe0
Author: Laimonas Simutis <la...@gmail.com>
Authored: Mon Dec 29 21:17:13 2014 -0500
Committer: Laimonas Simutis <la...@gmail.com>
Committed: Mon Dec 29 21:17:13 2014 -0500

----------------------------------------------------------------------
 src/Lucene.Net.Core/Codecs/Lucene3x/Lucene3xTermVectorsReader.cs | 4 ++--
 src/Lucene.Net.Core/Codecs/Lucene40/Lucene40TermVectorsReader.cs | 4 ++--
 src/Lucene.Net.Core/Codecs/Perfield/PerFieldPostingsFormat.cs    | 3 +--
 src/Lucene.Net.Core/Index/TieredMergePolicy.cs                   | 4 ++--
 src/Lucene.Net.Core/Store/RateLimitedDirectoryWrapper.cs         | 4 +++-
 src/Lucene.Net.Core/Util/Fst/FST.cs                              | 4 ++--
 6 files changed, 12 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucenenet/blob/6f88e8aa/src/Lucene.Net.Core/Codecs/Lucene3x/Lucene3xTermVectorsReader.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.Core/Codecs/Lucene3x/Lucene3xTermVectorsReader.cs b/src/Lucene.Net.Core/Codecs/Lucene3x/Lucene3xTermVectorsReader.cs
index ae4796e..7b474be 100644
--- a/src/Lucene.Net.Core/Codecs/Lucene3x/Lucene3xTermVectorsReader.cs
+++ b/src/Lucene.Net.Core/Codecs/Lucene3x/Lucene3xTermVectorsReader.cs
@@ -326,8 +326,8 @@ namespace Lucene.Net.Codecs.Lucene3x
                     return null;
                 }
 
-                int fieldIndex = FieldNumberToIndex[fieldInfo.Number];
-                if (fieldIndex == null)
+                int fieldIndex;
+                if (!FieldNumberToIndex.TryGetValue(fieldInfo.Number, out fieldIndex))
                 {
                     // Term vectors were not indexed for this field
                     return null;

http://git-wip-us.apache.org/repos/asf/lucenenet/blob/6f88e8aa/src/Lucene.Net.Core/Codecs/Lucene40/Lucene40TermVectorsReader.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.Core/Codecs/Lucene40/Lucene40TermVectorsReader.cs b/src/Lucene.Net.Core/Codecs/Lucene40/Lucene40TermVectorsReader.cs
index 4c4b4d8..4e9352c 100644
--- a/src/Lucene.Net.Core/Codecs/Lucene40/Lucene40TermVectorsReader.cs
+++ b/src/Lucene.Net.Core/Codecs/Lucene40/Lucene40TermVectorsReader.cs
@@ -310,8 +310,8 @@ namespace Lucene.Net.Codecs.Lucene40
                     return null;
                 }
 
-                int fieldIndex = FieldNumberToIndex[fieldInfo.Number];
-                if (fieldIndex == null)
+                int fieldIndex;
+                if (!FieldNumberToIndex.TryGetValue(fieldInfo.Number, out fieldIndex))
                 {
                     // Term vectors were not indexed for this field
                     return null;

http://git-wip-us.apache.org/repos/asf/lucenenet/blob/6f88e8aa/src/Lucene.Net.Core/Codecs/Perfield/PerFieldPostingsFormat.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.Core/Codecs/Perfield/PerFieldPostingsFormat.cs b/src/Lucene.Net.Core/Codecs/Perfield/PerFieldPostingsFormat.cs
index f21de45..60f90fe 100644
--- a/src/Lucene.Net.Core/Codecs/Perfield/PerFieldPostingsFormat.cs
+++ b/src/Lucene.Net.Core/Codecs/Perfield/PerFieldPostingsFormat.cs
@@ -121,8 +121,7 @@ namespace Lucene.Net.Codecs.Perfield
                     // First time we are seeing this format; create a new instance
 
                     // bump the suffix
-                    Suffixes.TryGetValue(formatName, out suffix);
-                    if (suffix == null)
+                    if (!Suffixes.TryGetValue(formatName, out suffix))
                     {
                         suffix = 0;
                     }

http://git-wip-us.apache.org/repos/asf/lucenenet/blob/6f88e8aa/src/Lucene.Net.Core/Index/TieredMergePolicy.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.Core/Index/TieredMergePolicy.cs b/src/Lucene.Net.Core/Index/TieredMergePolicy.cs
index 0482c2c..f453c89 100644
--- a/src/Lucene.Net.Core/Index/TieredMergePolicy.cs
+++ b/src/Lucene.Net.Core/Index/TieredMergePolicy.cs
@@ -685,8 +685,8 @@ namespace Lucene.Net.Index
             bool? segmentIsOriginal = false;
             foreach (SegmentCommitInfo info in infos.Segments)
             {
-                bool? isOriginal = segmentsToMerge[info];
-                if (isOriginal != null)
+                bool? isOriginal;
+                if (segmentsToMerge.TryGetValue(info, out isOriginal))
                 {
                     segmentIsOriginal = isOriginal;
                     if (!merging.Contains(info))

http://git-wip-us.apache.org/repos/asf/lucenenet/blob/6f88e8aa/src/Lucene.Net.Core/Store/RateLimitedDirectoryWrapper.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.Core/Store/RateLimitedDirectoryWrapper.cs b/src/Lucene.Net.Core/Store/RateLimitedDirectoryWrapper.cs
index 793d20b..8ac68b1 100644
--- a/src/Lucene.Net.Core/Store/RateLimitedDirectoryWrapper.cs
+++ b/src/Lucene.Net.Core/Store/RateLimitedDirectoryWrapper.cs
@@ -97,7 +97,9 @@ namespace Lucene.Net.Store
                 throw new System.ArgumentException("Context must not be null");
             }
             //int ord = context.ordinal();
-            RateLimiter limiter = ContextRateLimiters[context];
+            RateLimiter limiter;
+            ContextRateLimiters.TryGetValue(context, out limiter);
+
             if (mbPerSec == null)
             {
                 if (limiter != null)

http://git-wip-us.apache.org/repos/asf/lucenenet/blob/6f88e8aa/src/Lucene.Net.Core/Util/Fst/FST.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.Core/Util/Fst/FST.cs b/src/Lucene.Net.Core/Util/Fst/FST.cs
index 4f5596b..06a0446 100644
--- a/src/Lucene.Net.Core/Util/Fst/FST.cs
+++ b/src/Lucene.Net.Core/Util/Fst/FST.cs
@@ -1901,8 +1901,8 @@ namespace Lucene.Net.Util.Fst
                             bool doWriteTarget = TargetHasArcs(arc) && (flags & BIT_TARGET_NEXT) == 0;
                             if (doWriteTarget)
                             {
-                                int ptr = topNodeMap[(int)arc.Target];
-                                if (ptr != null)
+                                int ptr;
+                                if (topNodeMap.TryGetValue((int)arc.Target, out ptr))
                                 {
                                     absPtr = ptr;
                                 }


[3/3] lucenenet git commit: Merge remote-tracking branch 'laimis/dictionary_fixes'

Posted by sy...@apache.org.
Merge remote-tracking branch 'laimis/dictionary_fixes'


Project: http://git-wip-us.apache.org/repos/asf/lucenenet/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucenenet/commit/2bf20847
Tree: http://git-wip-us.apache.org/repos/asf/lucenenet/tree/2bf20847
Diff: http://git-wip-us.apache.org/repos/asf/lucenenet/diff/2bf20847

Branch: refs/heads/master
Commit: 2bf20847ff7616883be0c9d23fafcc7fd611745e
Parents: a57d720 6f88e8a
Author: Itamar Syn-Hershko <it...@code972.com>
Authored: Tue Dec 30 08:13:38 2014 +0200
Committer: Itamar Syn-Hershko <it...@code972.com>
Committed: Tue Dec 30 08:13:38 2014 +0200

----------------------------------------------------------------------
 src/Lucene.Net.Core/Codecs/Lucene3x/Lucene3xTermVectorsReader.cs | 4 ++--
 src/Lucene.Net.Core/Codecs/Lucene40/Lucene40TermVectorsReader.cs | 4 ++--
 src/Lucene.Net.Core/Codecs/Perfield/PerFieldPostingsFormat.cs    | 3 +--
 src/Lucene.Net.Core/Index/TieredMergePolicy.cs                   | 4 ++--
 src/Lucene.Net.Core/Store/RateLimitedDirectoryWrapper.cs         | 4 +++-
 src/Lucene.Net.Core/Util/Fst/FST.cs                              | 4 ++--
 6 files changed, 12 insertions(+), 11 deletions(-)
----------------------------------------------------------------------