Skip to content

Commit 79c1b4b

Browse files
committed
Reduce logging
1 parent f98d6e4 commit 79c1b4b

File tree

1 file changed

+24
-2
lines changed

1 file changed

+24
-2
lines changed

SequenceAnalysis/src/org/labkey/sequenceanalysis/run/util/RnaSeQCWrapper.java

Lines changed: 24 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@
2828
*/
2929
public class RnaSeQCWrapper extends AbstractCommandWrapper
3030
{
31+
final static int MAX_WARNINGS = 100;
32+
3133
public RnaSeQCWrapper(@Nullable Logger logger)
3234
{
3335
super(logger);
@@ -122,12 +124,32 @@ public File execute(List<File> inputBams, List<String> sampleIds, @Nullable List
122124
}
123125
else if (!line.contains("transcript_id"))
124126
{
125-
getLogger().info("skipping GTF line " + lineNo + " because it lacks transcript_id");
127+
if (filteredLines <= MAX_WARNINGS)
128+
{
129+
if (filteredLines == MAX_WARNINGS)
130+
{
131+
getLogger().info("skipping GTF line " + lineNo + " because it lacks transcript_id. No additional warnings will be printed");
132+
}
133+
else
134+
{
135+
getLogger().info("skipping GTF line " + lineNo + " because it lacks transcript_id");
136+
}
137+
}
126138
filteredLines++;
127139
}
128140
else if (!line.contains("gene_id"))
129141
{
130-
getLogger().info("skipping GTF line " + lineNo + " because it lacks gene_id");
142+
if (filteredLines <= MAX_WARNINGS)
143+
{
144+
if (filteredLines == MAX_WARNINGS)
145+
{
146+
getLogger().info("skipping GTF line " + lineNo + " because it lacks gene_id. No additional warnings will be printed");
147+
}
148+
else
149+
{
150+
getLogger().info("skipping GTF line " + lineNo + " because it lacks gene_id");
151+
}
152+
}
131153
filteredLines++;
132154
}
133155
else

0 commit comments

Comments
 (0)