wForget commented on code in PR #3686: URL: https://github.com/apache/datafusion-comet/pull/3686#discussion_r2937832303
########## docs/source/contributor-guide/debugging.md: ########## @@ -33,8 +33,14 @@ _Caveat: The steps here have only been tested with JDK 11_ on Mac (M1) ## Debugging for Advanced Developers -Add a `.lldbinit` to comet/core. This is not strictly necessary but will be useful if you want to -use advanced `lldb` debugging. +Add a `.lldbinit` to comet/native. This is not strictly necessary but will be useful if you want to +use advanced `lldb` debugging. For example, we can ignore some exceptions and signals that are not relevant +to our debugging and would otherwise cause the debugger to stop. + +``` Review Comment: I tend to think it is recommended, these exceptions/signals don't seem to need to be considered during local debugging. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
