[comixed] Re: ComiXed 0.7.0-rc2 is now available

  • From: "Darryl L. Pierce" <mcpierce@xxxxxxxxx>
  • To: comixed@xxxxxxxxxxxxx
  • Date: Wed, 16 Sep 2020 18:37:01 -0400

Cool, this is what I need to look into the issue.Just a first glance
(I'll do more later) I see errors like the following happening:

org.comixedproject.task.model.WorkerTaskException: Could not rename
original comic file
        at 
org.comixedproject.task.model.MoveComicWorkerTask.startTask(MoveComicWorkerTask.java:66)
~[comixed-tasks-0.7.0-rc2.jar!/:0.7.0-rc2]
        at 
org.comixedproject.task.runner.TaskManager.lambda$runTask$0(TaskManager.java:71)
~[comixed-tasks-0.7.0-rc2.jar!/:0.7.0-rc2]
        at 
java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130)
~[na:na]
        at 
java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630)
~[na:na]
        at java.base/java.lang.Thread.run(Thread.java:832) ~[na:na]
Caused by: java.io.FileNotFoundException: Source
'/media/server01h/Comics/comixed_test/DC Comics/Batman/2016/Batman
v2016 #22 (Jul 2017).cbz' does not exist
        at org.apache.commons.io.FileUtils.moveFile(FileUtils.java:2995)
~[commons-io-2.6.jar!/:2.6]
        at 
org.comixedproject.task.model.MoveComicWorkerTask.renameOriginalFile(MoveComicWorkerTask.java:104)
~[comixed-tasks-0.7.0-rc2.jar!/:0.7.0-rc2]
        at 
org.comixedproject.task.model.MoveComicWorkerTask.startTask(MoveComicWorkerTask.java:64)
~[comixed-tasks-0.7.0-rc2.jar!/:0.7.0-rc2]
        ... 4 common frames omitted

So there it's trying to move a comic but the comic it's trying to move
(filename=/media/server01h/Comics/comixed_test/DC
Comics/Batman/2016/Batman v2016 #22 (Jul 2017).cbz) isn't found.

I also see it trying twice to delete the same task:

2020-09-16 18:07:47.392 DEBUG 114052 --- [       Jarvis-3]
o.c.service.task.TaskService             : Deleting task: id=533
...
2020-09-16 18:07:57.365 DEBUG 114052 --- [       Jarvis-4]
o.c.service.task.TaskService             : Deleting task: id=533

which is strange behavior since tasks are only deleted after they've
been executed and on the same thread (worker threads are named
"Jarvis"). And here we see two separate threads trying to delete the
same task. I wonder if it's not the case of two Jarvises (Jarvii?)
grabbing the same task.

I'll see if I can replicate this locally and what I can determine is a
fix for it.

On Wed, Sep 16, 2020 at 5:36 PM Jim Dinda <jim@xxxxxxxxx> wrote:


Update:

Same behavior as before.

imported 84 comics.

Tried to organize the library.

51 comics were moved into the library directory

33 comics were left behind in the import directory

It seems like the app went into a loop trying to move those same 51 comics 
over and over.

Loop continues even if I restart the app.

See trace attached (zipped because the raw text file was too big)














On Wed, Sep 16, 2020 at 1:39 PM Jim Dinda <jim@xxxxxxxxx> wrote:

I realized that I moved the comics out of the library folder without 
deleting the database so I could start over.

I deleted the database/settings and I am going through the whole process 
again and will report the results.

On Wed, Sep 16, 2020 at 11:39 AM Darryl L. Pierce <mcpierce@xxxxxxxxx> wrote:

You should be able to just copy the content of the stack trace and
paste it into a reply to this thread.

On Wed, Sep 16, 2020 at 2:10 PM Jim Dinda <jim@xxxxxxxxx> wrote:

Here is the last page of the Task log through the web interface as a PDF.

Please let me know if you are looking for something else.  Thank you.

On Wed, Sep 16, 2020 at 8:17 AM Darryl L. Pierce <mcpierce@xxxxxxxxx> 
wrote:

On Tue, Sep 15, 2020 at 4:22 PM Jim Dinda <jim@xxxxxxxxx> wrote:
<snip>
Website comes up, everything seems to be working, but when I try to do 
the following, it does not work as expected:

1. start with a clean/new database
2. have one folder with comics in them organized by Comixed previously 
(default folder structure).
3. have another folder with other comics just flat (no folder 
structure).

My goal is to:

1. import each of the two existing folders
2. organize the library into a new folder with the default folder 
structure
3. in the future, import more comics from an import folder and then, 
organize the library again to put them all into the library folder 
organized using the default structure.

However ... this is what happens:

1. import the 1st folder described above.  Import process works 
correctly and comics are added to the database (but not moved)
2. import the 2nd folder described above.  The import process is a 
little weird.  At 1st, when I click import nothing happens, but if I 
go to "comics", and then back to import, the "import" starts.

There can be a slight lag depending on how many comics are being
imported from any directory. The long and short of it is, for each
comic being imported, a task is created in the database to start that
process. This is done by the TaskManager which, currently, can only do
five tasks in parallel. So while it's doing the task of enqueueing the
import tasks for each comic, it can't actually start importing.

3. once the 2nd import completes, I go to the "Library" page, put in 
the path of the library folder and click "Organize library"
4. for some reason, all of the comics in the 1st folder imported are 
moved into the default folder structure in the new library folder, but 
NONE of the comics in the 2nd folder are moved (all are still 
browsable in the app).
5. when I look at the java messages running on the server, I can see 
that it seems to be looping around and trying to move the folders in 
the 1st folder over and over again (even though they are already 
moved).  Even if I kill the process and relaunch, it seems to be 
looping around and "moving" the files again and again.

Can you paste the task audit log entry for this task in a reply for
me, please? If a task fails, it shouldn't be getting rerun. The task
manager is supposed to delete all tasks from the database after
they're run, even if they fail and throw an exception. But if it's an
unhandled exception then it might be getting past that cleanup.

I hope that is clear.  Please let me know if I can provide any more 
details/log files/etc.

Yes, just a few questions for clarification but otherwise well described.

--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie



-- 
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie

Other related posts: