You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by kw...@apache.org on 2018/03/29 14:37:53 UTC

[1/2] lucene-solr:master: LUCENE-8227: Under pressure, had to @Ignore tests that caused intermittent failures. This means no further work on Geo3D until these tests can be re-enabled.

Repository: lucene-solr
Updated Branches:
  refs/heads/master 668b81721 -> 358e59596


LUCENE-8227: Under pressure, had to @Ignore tests that caused intermittent failures.  This means no further work on Geo3D until these tests can be re-enabled.


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

Branch: refs/heads/master
Commit: 5b429df56f36031c8423803d5157029c59bcbb3d
Parents: b151b2c
Author: Karl Wright <Da...@gmail.com>
Authored: Thu Mar 29 10:37:28 2018 -0400
Committer: Karl Wright <Da...@gmail.com>
Committed: Thu Mar 29 10:37:28 2018 -0400

----------------------------------------------------------------------
 .../apache/lucene/spatial3d/TestGeo3DPoint.java | 10 ++++
 .../lucene/spatial3d/geom/GeoPolygonTest.java   | 60 ++++++++++++++++++++
 2 files changed, 70 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/5b429df5/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/TestGeo3DPoint.java
----------------------------------------------------------------------
diff --git a/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/TestGeo3DPoint.java b/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/TestGeo3DPoint.java
index 861e26e..38b4114 100644
--- a/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/TestGeo3DPoint.java
+++ b/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/TestGeo3DPoint.java
@@ -81,6 +81,8 @@ import org.apache.lucene.util.NumericUtils;
 import org.apache.lucene.util.StringHelper;
 import org.apache.lucene.util.TestUtil;
 
+import org.junit.Ignore;
+
 import com.carrotsearch.randomizedtesting.generators.RandomNumbers;
 
 public class TestGeo3DPoint extends LuceneTestCase {
@@ -188,6 +190,8 @@ public class TestGeo3DPoint extends LuceneTestCase {
   }
 
   /** Tests consistency of GeoArea.getRelationship vs GeoShape.isWithin */
+  //@AwaitsFix("https://issues.apache.org/jira/browse/LUCENE-8227")
+  @Ignore
   public void testGeo3DRelations() throws Exception {
 
     int numDocs = atLeast(1000);
@@ -467,16 +471,22 @@ public class TestGeo3DPoint extends LuceneTestCase {
     }
   }
 
+  //@AwaitsFix("https://issues.apache.org/jira/browse/LUCENE-8227")
+  @Ignore
   public void testRandomTiny() throws Exception {
     // Make sure single-leaf-node case is OK:
     doTestRandom(10);
   }
 
+  //@AwaitsFix("https://issues.apache.org/jira/browse/LUCENE-8227")
+  @Ignore
   public void testRandomMedium() throws Exception {
     doTestRandom(10000);
   }
 
   @Nightly
+  //@AwaitsFix("https://issues.apache.org/jira/browse/LUCENE-8227")
+  @Ignore
   public void testRandomBig() throws Exception {
     doTestRandom(50000);
   }

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/5b429df5/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/geom/GeoPolygonTest.java
----------------------------------------------------------------------
diff --git a/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/geom/GeoPolygonTest.java b/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/geom/GeoPolygonTest.java
index 2625ba7..ebfb0f4 100755
--- a/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/geom/GeoPolygonTest.java
+++ b/lucene/spatial3d/src/test/org/apache/lucene/spatial3d/geom/GeoPolygonTest.java
@@ -22,6 +22,7 @@ import java.util.BitSet;
 import java.util.Collections;
 
 import org.junit.Test;
+import org.junit.Ignore;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
@@ -1204,4 +1205,63 @@ shape:
     Collections.reverse(points);
     polygon  = GeoPolygonFactory.makeGeoPolygon(PlanetModel.SPHERE, points);
   }
+  
+  /*
+   [lat=-0.63542308910253, lon=0.9853722928232957([X=0.4446759777403525, Y=0.6707549854468698, Z=-0.5934780737681111])], 
+  [lat=0.0, lon=0.0([X=1.0011188539924791, Y=0.0, Z=0.0])], 
+  [lat=0.45435018176633574, lon=3.141592653589793([X=-0.8989684544372841, Y=1.1009188402610632E-16, Z=0.4390846549572752])], 
+  [lat=-0.375870856827283, lon=2.9129132647718414([X=-0.9065744420970767, Y=0.21100590938346708, Z=-0.36732668582405886])], 
+  [lat=-1.2205765069413237, lon=3.141592653589793([X=-0.3424714964202101, Y=4.194066218902145E-17, Z=-0.9375649457139603])]}}
+  
+   [junit4]   1>       unquantized=[lat=-3.1780051348770987E-74, lon=-3.032608859187692([X=-0.9951793580358298, Y=-0.1088898762907205, Z=-3.181560858610375E-74])]
+   [junit4]   1>       quantized=[X=-0.9951793580415914, Y=-0.10888987641797832, Z=-2.3309121299774915E-10]
+  */
+  @Test
+  @Ignore
+  public void testLUCENE8227() throws Exception {
+    List<GeoPoint> points = new ArrayList<>();
+    points.add(new GeoPoint(PlanetModel.WGS84, -0.63542308910253, 0.9853722928232957));
+    points.add(new GeoPoint(PlanetModel.WGS84, 0.0, 0.0));
+    points.add(new GeoPoint(PlanetModel.WGS84, 0.45435018176633574, 3.141592653589793));
+    points.add(new GeoPoint(PlanetModel.WGS84, -0.375870856827283, 2.9129132647718414));
+    points.add(new GeoPoint(PlanetModel.WGS84, -1.2205765069413237, 3.141592653589793));
+    GeoPolygonFactory.PolygonDescription pd = new GeoPolygonFactory.PolygonDescription(points);
+    
+    for (int i = 0; i < points.size(); i++) {
+      System.out.println("Point "+i+": "+points.get(i));
+    }
+
+    final GeoPoint unquantized = new GeoPoint(PlanetModel.WGS84, -3.1780051348770987E-74, -3.032608859187692);
+    final GeoPoint quantized = new GeoPoint(-0.9951793580415914, -0.10888987641797832, -2.3309121299774915E-10);
+    
+    final GeoPoint negativeX = new GeoPoint(PlanetModel.WGS84, 0.0, Math.PI);
+    final GeoPoint negativeY = new GeoPoint(PlanetModel.WGS84, 0.0, -Math.PI * 0.5);
+    
+    // Construct a standard polygon first to see what that does
+    GeoPolygon standard = GeoPolygonFactory.makeGeoPolygon(PlanetModel.WGS84, pd);
+    
+    System.out.println("Standard polygon: "+standard);
+    
+    // This shows y < 0 hemisphere is all in-set
+    //assertTrue(standard.isWithin(negativeY));
+    // This should be in-set too, but isn't!!
+    assertTrue(standard.isWithin(negativeX));
+    
+/*
+    final XYZBounds standardBounds = new XYZBounds();
+    standard.getBounds(standardBounds);
+    final XYZSolid standardSolid = XYZSolidFactory.makeXYZSolid(PlanetModel.WGS84, standardBounds);
+
+    System.out.println("Standard bounds: "+standardBounds);
+    
+    assertFalse(standardSolid.isWithin(quantized));
+    assertFalse(standardSolid.isWithin(unquantized));
+*/
+    // Now, both points should also not be in the poly
+    assertFalse(standard.isWithin(unquantized));
+    assertFalse(standard.isWithin(quantized));
+
+
+  }
+  
 }


[2/2] lucene-solr:master: Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/lucene-solr

Posted by kw...@apache.org.
Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/lucene-solr


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

Branch: refs/heads/master
Commit: 358e59596d17ba34452ea923e048afee6233d597
Parents: 5b429df 668b817
Author: Karl Wright <Da...@gmail.com>
Authored: Thu Mar 29 10:37:45 2018 -0400
Committer: Karl Wright <Da...@gmail.com>
Committed: Thu Mar 29 10:37:45 2018 -0400

----------------------------------------------------------------------
 dev-tools/scripts/crawl.maven.release.dist.sh   |  4 +-
 dev-tools/scripts/prep-solr-ref-guide-rc.sh     | 20 +++----
 dev-tools/scripts/smokeTestRelease.py           | 44 +++++++--------
 lucene/CHANGES.txt                              |  9 +++
 lucene/common-build.xml                         | 10 ++--
 .../lucene/index/IndexDeletionPolicy.java       | 18 +++---
 .../apache/lucene/index/NoDeletionPolicy.java   |  6 --
 solr/CHANGES.txt                                | 14 ++++-
 solr/build.xml                                  |  2 +-
 .../handler/dataimport/DataImportHandler.java   |  6 +-
 .../cloud/LeaderInitiatedRecoveryThread.java    |  6 +-
 .../org/apache/solr/core/CoreContainer.java     | 45 ++++++++-------
 .../apache/solr/core/HdfsDirectoryFactory.java  |  8 +--
 .../solr/core/IndexDeletionPolicyWrapper.java   |  6 --
 .../src/java/org/apache/solr/core/SolrCore.java | 41 ++++++++------
 .../java/org/apache/solr/core/SolrInfoBean.java |  2 +-
 .../apache/solr/handler/ReplicationHandler.java | 44 +++++++--------
 .../apache/solr/handler/RequestHandlerBase.java | 12 ++--
 .../solr/handler/admin/CoreAdminHandler.java    |  4 +-
 .../component/HttpShardHandlerFactory.java      |  9 ++-
 .../solr/handler/component/SearchComponent.java |  2 +-
 .../handler/component/SuggestComponent.java     | 11 +++-
 .../solr/highlight/HighlightingPluginBase.java  |  6 +-
 .../solr/metrics/SolrCoreMetricManager.java     |  7 ++-
 .../apache/solr/metrics/SolrMetricManager.java  | 58 ++++++++++++++++++--
 .../apache/solr/metrics/SolrMetricProducer.java |  7 ++-
 .../metrics/reporters/solr/SolrReporter.java    |  2 +-
 .../solr/search/ExtendedDismaxQParser.java      |  2 +-
 .../org/apache/solr/search/FastLRUCache.java    |  9 ++-
 .../java/org/apache/solr/search/LFUCache.java   |  8 ++-
 .../java/org/apache/solr/search/LRUCache.java   | 10 +++-
 .../apache/solr/search/SolrFieldCacheBean.java  |  4 +-
 .../apache/solr/search/SolrIndexSearcher.java   | 37 +++++++------
 .../apache/solr/servlet/SolrDispatchFilter.java | 26 ++++++---
 .../apache/solr/store/blockcache/Metrics.java   |  8 ++-
 .../solr/store/hdfs/HdfsLocalityReporter.java   |  8 ++-
 .../solr/update/DirectUpdateHandler2.java       | 28 ++++++----
 .../java/org/apache/solr/update/PeerSync.java   |  2 +-
 .../org/apache/solr/update/SolrIndexWriter.java | 34 +++++++-----
 .../java/org/apache/solr/update/UpdateLog.java  | 14 +++--
 .../apache/solr/update/UpdateShardHandler.java  |  6 +-
 .../processor/AtomicUpdateProcessorFactory.java | 26 ++++++---
 .../stats/InstrumentedHttpRequestExecutor.java  |  7 ++-
 ...entedPoolingHttpClientConnectionManager.java | 19 +++++--
 .../solr/collection1/conf/schema12.xml          | 12 ++++
 .../test/org/apache/solr/CursorPagingTest.java  |  5 +-
 .../test/org/apache/solr/SolrInfoBeanTest.java  |  2 +-
 .../org/apache/solr/cloud/ZkSolrClientTest.java | 30 ++++++++--
 .../sim/SimClusterStateProvider.java            |  6 +-
 .../solr/core/ExitableDirectoryReaderTest.java  |  7 ++-
 .../solr/core/HdfsDirectoryFactoryTest.java     |  4 +-
 .../test/org/apache/solr/core/MockInfoBean.java |  8 ++-
 .../core/MockQuerySenderListenerReqHandler.java |  6 +-
 .../apache/solr/core/TestJmxIntegration.java    |  4 +-
 .../org/apache/solr/metrics/JvmMetricsTest.java |  2 +-
 .../solr/metrics/SolrCoreMetricManagerTest.java |  2 +-
 .../solr/metrics/SolrMetricManagerTest.java     | 10 ++--
 .../solr/metrics/SolrMetricTestUtils.java       |  2 +-
 .../solr/search/TestExtendedDismaxParser.java   | 19 ++++++-
 .../apache/solr/search/TestFastLRUCache.java    | 14 ++---
 .../org/apache/solr/search/TestLFUCache.java    |  6 +-
 .../org/apache/solr/search/TestLRUCache.java    |  4 +-
 .../solr/search/TestReRankQParserPlugin.java    |  3 +-
 .../apache/solr/search/TestSolr4Spatial2.java   |  3 +-
 .../solr/search/TestSolrFieldCacheBean.java     |  8 +--
 .../apache/solr/search/TestSolrQueryParser.java |  9 +--
 .../apache/solr/search/join/BJQParserTest.java  |  9 +--
 .../solr/search/join/TestScoreJoinQPScore.java  |  3 +-
 .../solr/store/blockcache/BufferStoreTest.java  |  4 +-
 .../AtomicUpdateProcessorFactoryTest.java       |  1 -
 solr/solr-ref-guide/src/meta-docs/publish.adoc  |  8 ++-
 71 files changed, 524 insertions(+), 318 deletions(-)
----------------------------------------------------------------------