Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-24 Thread via GitHub
javanna commented on PR #12689: URL: https://github.com/apache/lucene/pull/12689#issuecomment-1777505662 Thanks for the review @jpountz ! -- 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 s

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-24 Thread via GitHub
javanna merged PR #12689: URL: https://github.com/apache/lucene/pull/12689 -- 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.apa

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-23 Thread via GitHub
javanna commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1369190050 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-19 Thread via GitHub
jpountz commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1365756941 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-19 Thread via GitHub
javanna commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1365648915 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-19 Thread via GitHub
jpountz commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1365380977 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-18 Thread via GitHub
javanna commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1363627307 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-18 Thread via GitHub
javanna commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1363626228 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-18 Thread via GitHub
jpountz commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1363513095 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-17 Thread via GitHub
javanna commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1362621950 ## lucene/core/src/test/org/apache/lucene/search/TestTaskExecutor.java: ## @@ -43,7 +47,8 @@ public class TestTaskExecutor extends LuceneTestCase { public static vo

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-17 Thread via GitHub
javanna commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1362621950 ## lucene/core/src/test/org/apache/lucene/search/TestTaskExecutor.java: ## @@ -43,7 +47,8 @@ public class TestTaskExecutor extends LuceneTestCase { public static vo

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-17 Thread via GitHub
javanna commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1362621063 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task

Re: [PR] TaskExecutor to cancel all tasks on exception [lucene]

2023-10-17 Thread via GitHub
javanna commented on code in PR #12689: URL: https://github.com/apache/lucene/pull/12689#discussion_r1362620375 ## lucene/core/src/java/org/apache/lucene/search/TaskExecutor.java: ## @@ -64,64 +67,124 @@ public final class TaskExecutor { * @param the return type of the task