Skip to content

Conversation

@soujay
Copy link
Contributor

@soujay soujay commented Apr 7, 2025

This PR handles the RegexMatchTimeoutException by returning the GetNonStandardClassification.

Fixes #4937

@soujay soujay self-assigned this Apr 7, 2025
…ypeAndStackTraceFailureClassifier.cs

Co-authored-by: Nick Gallegos <nicholas.gallegos@gmail.com>
@soujay soujay marked this pull request as ready for review April 7, 2025 21:24
@bording bording merged commit 13c7852 into master Apr 8, 2025
32 checks passed
@bording bording deleted the handle-stacktrace-parser-regex-exception branch April 8, 2025 21:51
DavidBoike pushed a commit that referenced this pull request Apr 9, 2025
* handle the RegexMatchTimeoutException

* Update src/ServiceControl/Recoverability/Grouping/Groupers/ExceptionTypeAndStackTraceFailureClassifier.cs

Co-authored-by: Nick Gallegos <nicholas.gallegos@gmail.com>

---------

Co-authored-by: Nick Gallegos <nicholas.gallegos@gmail.com>
DavidBoike pushed a commit that referenced this pull request Apr 9, 2025
* handle the RegexMatchTimeoutException

* Update src/ServiceControl/Recoverability/Grouping/Groupers/ExceptionTypeAndStackTraceFailureClassifier.cs

Co-authored-by: Nick Gallegos <nicholas.gallegos@gmail.com>

---------

Co-authored-by: Nick Gallegos <nicholas.gallegos@gmail.com>
DavidBoike added a commit that referenced this pull request Apr 9, 2025
* handle the RegexMatchTimeoutException

* Update src/ServiceControl/Recoverability/Grouping/Groupers/ExceptionTypeAndStackTraceFailureClassifier.cs



---------

Co-authored-by: Jayanthi <jayanthi.sourirajan@particular.net>
Co-authored-by: Nick Gallegos <nicholas.gallegos@gmail.com>
DavidBoike added a commit that referenced this pull request Apr 9, 2025
* handle the RegexMatchTimeoutException

* Update src/ServiceControl/Recoverability/Grouping/Groupers/ExceptionTypeAndStackTraceFailureClassifier.cs



---------

Co-authored-by: Jayanthi <jayanthi.sourirajan@particular.net>
Co-authored-by: Nick Gallegos <nicholas.gallegos@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error ingestion can throw RegexMatchTimeoutException

5 participants