Skip to content

Commit 4af7b79

Browse files
committed
Expand orphan pipeline job tests
1 parent d6df820 commit 4af7b79

File tree

1 file changed

+15
-8
lines changed

1 file changed

+15
-8
lines changed

SequenceAnalysis/src/org/labkey/sequenceanalysis/pipeline/OrphanFilePipelineJob.java

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -396,11 +396,10 @@ public void getOrphanFilesForContainer(Container c, User u, Set<File> orphanFile
396396
}
397397
}
398398

399-
//TODO: look for .deleted and /archive
400399
File deletedDir = new File(root.getRootPath().getParentFile(), ".deleted");
401400
if (deletedDir.exists())
402401
{
403-
messages.add("## .deleted dir found: " + deletedDir.getPath());
402+
getJob().getLogger().warn("WARNING: .deleted dir found: " + deletedDir.getPath());
404403
}
405404

406405
File assayData = new File(root.getRootPath(), "assaydata");
@@ -513,12 +512,6 @@ private void getOrphanFilesForDirectory(Set<Integer> knownExpDatas, Map<URI, Set
513512

514513
for (File f : arr)
515514
{
516-
//skipped for perf reasons. extremely unlikely
517-
//if (Files.isSymbolicLink(f.toPath()))
518-
//{
519-
// continue;
520-
//}
521-
522515
if (f.isDirectory())
523516
{
524517
if (f.getName().endsWith(".gdb"))
@@ -535,6 +528,20 @@ private void getOrphanFilesForDirectory(Set<Integer> knownExpDatas, Map<URI, Set
535528
{
536529
//iterate possible issues:
537530

531+
// extremely large log (200 MB):
532+
if (f.getName().endsWith(".log") & f.length() > (200*1024*1024))
533+
{
534+
getJob().getLogger().warn("WARNING: Extremely large log file: " + f.getPath() + ", " + FileUtils.byteCountToDisplaySize(f.length()));
535+
}
536+
537+
if (f.getName().endsWith(".rds"))
538+
{
539+
if (!dataMap.containsKey(f.toURI()))
540+
{
541+
getJob().getLogger().warn("WARNING: Unknown RDS file: " + f.getPath());
542+
}
543+
}
544+
538545
//orphan index
539546
if (f.getPath().toLowerCase().endsWith(".bai") || f.getPath().toLowerCase().endsWith(".tbi") || f.getPath().toLowerCase().endsWith(".idx") || f.getPath().toLowerCase().endsWith(".crai"))
540547
{

0 commit comments

Comments
 (0)