samtools / htsjdk

A Java API for high-throughput sequencing data (HTS) formats.
http://samtools.github.io/htsjdk/
276 stars 244 forks source link

Remove low-value progress logging message #1659

Closed nh13 closed 1 year ago

nh13 commented 1 year ago

Description

I think printing read names for non-coordinate sorted data is useful, but the log message that "Seen many non-increasing record positions. Printing Read-names as well" is not that useful confusing to both developers and users, especially in an API. I'd be fine with it being at the debug level. I just don't want to see this message in my logs every time I use a progress logger that doesn't feed coordinate-sorted data.