[GitHub] [lucene] dweiss opened a new pull request, #11766: Retry gradle wrapper download on http 500 and 503.

2022-09-13 Thread GitBox
dweiss opened a new pull request, #11766: URL: https://github.com/apache/lucene/pull/11766 We have more and more frequent CI failures due to http 500 and 503 when downloading the gradle wrapper. I added a simple retry mechanism (compatible with JDK11 for ease of backport) - perhaps it'll li

[GitHub] [lucene] rmuir commented on a diff in pull request #11766: Retry gradle wrapper download on http 500 and 503.

2022-09-13 Thread GitBox
rmuir commented on code in PR #11766: URL: https://github.com/apache/lucene/pull/11766#discussion_r969252828 ## buildSrc/src/main/java/org/apache/lucene/gradle/WrapperDownloader.java: ## @@ -90,18 +95,34 @@ public void run(Path destination) throws IOException, NoSuchAlgorithmEx

[GitHub] [lucene] dweiss commented on a diff in pull request #11766: Retry gradle wrapper download on http 500 and 503.

2022-09-13 Thread GitBox
dweiss commented on code in PR #11766: URL: https://github.com/apache/lucene/pull/11766#discussion_r969270375 ## buildSrc/src/main/java/org/apache/lucene/gradle/WrapperDownloader.java: ## @@ -90,18 +95,34 @@ public void run(Path destination) throws IOException, NoSuchAlgorithmE

[GitHub] [lucene] dweiss merged pull request #11766: Retry gradle wrapper download on http 500 and 503.

2022-09-13 Thread GitBox
dweiss merged PR #11766: URL: https://github.com/apache/lucene/pull/11766 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apac

[GitHub] [lucene] shaie commented on a diff in pull request #11764: GITHUB#11742: MatchingFacetSetsCounts#getTopChildren now returns top children instead of all children

2022-09-13 Thread GitBox
shaie commented on code in PR #11764: URL: https://github.com/apache/lucene/pull/11764#discussion_r969152978 ## lucene/facet/src/java/org/apache/lucene/facet/facetset/MatchingFacetSetsCounts.java: ## @@ -156,7 +157,43 @@ public FacetResult getAllChildren(String dim, String... p

[GitHub] [lucene] iverase opened a new issue, #11767: Does the method #cureLocalIntersections on the Tessellator makes any sense?

2022-09-13 Thread GitBox
iverase opened a new issue, #11767: URL: https://github.com/apache/lucene/issues/11767 ### Description I have always been confused by this method which claims to fix self-intersections of the polygon but we require in the Tessellator java docs that polygons should not have self-inter

[GitHub] [lucene] gsmiller opened a new pull request, #11768: Fix tie-break bug in various Facets implementations

2022-09-13 Thread GitBox
gsmiller opened a new pull request, #11768: URL: https://github.com/apache/lucene/pull/11768 ### Description There are a number of places in `Facets` implementations where `getTopChildren` is incorrectly handling count/value ties. The behavior should prefer smaller ordinals when coun

[GitHub] [lucene] gsmiller merged pull request #11764: GITHUB#11742: MatchingFacetSetsCounts#getTopChildren now returns top children instead of all children

2022-09-13 Thread GitBox
gsmiller merged PR #11764: URL: https://github.com/apache/lucene/pull/11764 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.ap

[GitHub] [lucene] mayya-sharipova opened a new issue, #11769: TestKnnVectorQuery.testScoreEuclidean fails

2022-09-13 Thread GitBox
mayya-sharipova opened a new issue, #11769: URL: https://github.com/apache/lucene/issues/11769 ### Description Reproducibly fails on `main` branch. ### Gradle command to reproduce ./gradlew test --tests TestKnnVectorQuery.testScoreEuclidean -Dtests.seed=1DAEE5C9217129EB

[GitHub] [lucene] mayya-sharipova commented on issue #11769: TestKnnVectorQuery.testScoreEuclidean fails

2022-09-13 Thread GitBox
mayya-sharipova commented on issue #11769: URL: https://github.com/apache/lucene/issues/11769#issuecomment-1245545277 @msokolov I've noticed that in https://github.com/apache/lucene/pull/1054 we don't force merge index for this test anymore, and may end up with several segments, which leads

[GitHub] [lucene] gsmiller closed issue #11742: MatchingFacetSetsCounts doesn't properly implement getTopChildren

2022-09-13 Thread GitBox
gsmiller closed issue #11742: MatchingFacetSetsCounts doesn't properly implement getTopChildren URL: https://github.com/apache/lucene/issues/11742 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [lucene] msokolov commented on issue #11769: TestKnnVectorQuery.testScoreEuclidean fails

2022-09-13 Thread GitBox
msokolov commented on issue #11769: URL: https://github.com/apache/lucene/issues/11769#issuecomment-1245892443 sorry - did I break that? I think I replaced RandomIndexWriter with IndexWriter and took control over flushing and merging? But maybe I misremembered/messed up -- This is an aut

[GitHub] [lucene] LuXugang opened a new issue, #11770: Optimization for time series data

2022-09-13 Thread GitBox
LuXugang opened a new issue, #11770: URL: https://github.com/apache/lucene/issues/11770 ### Description Hi, recently I read a [paper](https://www.vldb.org/pvldb/vol15/p3472-yu.pdf) from [VLDB](https://vldb.org/2022/?paper-session) said it gains significant performance improvements ag