This is an automated email from the ASF dual-hosted git repository.

nightowl888 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/lucenenet.git

commit 2dc1c372a11b08a3f7999a89d1ea3b9dd98ced66
Author: Shad Storhaug <[email protected]>
AuthorDate: Sat Aug 14 05:18:15 2021 +0700

    BUG: 
Lucene.Net.Tests.Index.TestIndexWriterWithThreads::TestRollbackAndCommitWithThreads():
 Restored original NullReferenceException catch blocks that were in Lucene due 
to random test failures.
---
 src/Lucene.Net.Tests/Index/TestIndexWriterWithThreads.cs | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/Lucene.Net.Tests/Index/TestIndexWriterWithThreads.cs 
b/src/Lucene.Net.Tests/Index/TestIndexWriterWithThreads.cs
index 1dd4e13..11c2d95 100644
--- a/src/Lucene.Net.Tests/Index/TestIndexWriterWithThreads.cs
+++ b/src/Lucene.Net.Tests/Index/TestIndexWriterWithThreads.cs
@@ -780,10 +780,10 @@ namespace Lucene.Net.Index
                                 {
                                     // ok
                                 }
-                                //catch (NullReferenceException) // LUCENENET 
specific - NullReferenceException must be allowed to propagate so we can 
defensively avoid it in .NET
-                                //{
-                                //    // ok
-                                //}
+                                catch (NullReferenceException) // LUCENENET 
TODO: - NullReferenceException must be allowed to propagate so we can 
defensively avoid it in .NET
+                                {
+                                    // ok
+                                }
                                 finally
                                 {
                                     commitLock.Unlock();
@@ -803,10 +803,10 @@ namespace Lucene.Net.Index
                                 {
                                     // ok
                                 }
-                                //catch (NullReferenceException) // LUCENENET 
specific - NullReferenceException must be allowed to propagate so we can 
defensively avoid it in .NET
-                                //{
-                                //    // ok
-                                //}
+                                catch (NullReferenceException) // LUCENENET 
TODO: - NullReferenceException must be allowed to propagate so we can 
defensively avoid it in .NET
+                                {
+                                    // ok
+                                }
                                 catch (Exception ae) when 
(ae.IsAssertionError())
                                 {
                                     // ok

Reply via email to