JetBrains / lincheck

Framework for testing concurrent data structures
Mozilla Public License 2.0
564 stars 33 forks source link

Deprecate`logLevel` and introduce an internal `lincheck.debug` system property instead #157

Closed ndkoval closed 1 year ago

ndkoval commented 1 year ago

The logLevel property is used internally when debugging Lincheck. It should be separate from the API and is rarely needed, even when developing Lincheck itself. Let's deprecate the logLevel property and introduce the lincheck.debug system property instead. Thus, to print the debug info, one needs to run the process with -Dlincheck.debug=true. This will be an internal property for internal use.

eupp commented 1 year ago

Also, maybe we can just use some existing logging library, like kotlin-logging?

ndkoval commented 1 year ago

We don't need such a library, as this property should enable the debug output for us, the developers.

ndkoval commented 1 year ago

There is a better option: we will keep it as a part of the internal API. Therefore, I'm closing the issue.