Fixed.
I have also added a Javadoc comment to the run()
method in the class.
If a watched directory is deleted, JBoss will not give
a NULLPointerException, it will *ignore* the
directory. The next check will undeploy all EJBs from
that directoy. The directory will still remain in the
watched list - so if a user deletes the dir by
mistake, he can undelete / recreate it and get on with
life :-)
Let me know if this is ok !
Another thing, I noticed is that the Autodeployer is
set to 3 seconds - hardcoded ! If the user wants to
change that, he would have to modify the source. I
think, we shold move that out of source and into a
config file !
I am ready to commit. (need rw access) !
Ajay
--- marc fleury <[EMAIL PROTECTED]> wrote:
> aren't you supposed to do RMI-IIOP??? :)
>
> give me some RMI IIOP and I give you RW
>
> right
>
> marcf
>
> PS: seriously, b is the option, let me know when you
> are ready to commit
>
>
> |-----Original Message-----
> |From: [EMAIL PROTECTED]
>
|[mailto:[EMAIL PROTECTED]]On
> Behalf Of Ajay
> |Gautam
> |Sent: Wednesday, June 13, 2001 5:47 PM
> |To: [EMAIL PROTECTED]
> |Subject: Re: [JBoss-dev] [ jboss-Bugs-432844 ]
> NullPointerException in
> |AutoDeployer
> |
> |
> |Hey Guys
> |
> |Just to find my way around the code and to get
> |acquainted with the build / deploy process, I took
> on
> |this bug to fix.
> |
> |I am able to replicate / locate / fix the problem.
> |Right now, this error stops the auto deployer
> thread
> |(running = false;) !
> |
> |One possibility could be to remove this bug from
> the
> |bugs database as its lame ! otherwise:
> |
> |What should JBoss do when someone deletes a
> "watched"
> |directory ?
> | a. Create the directory again, and continue -
> in
> |this case, JBoss auto un-deployes EJBs from that
> |directory !
> | b. Undeploy all EJBs from that directory and
> |remove directory from watched list.
> | c. Quit / Halt !
> |I would personally prefer b ! that looks the
> cleanest.
> |The problem with a and b - if someone deleted the
> |"deploy" directory, jbosspool-jdbc.rar and
> jms-ra.rar
> |are undeployed ! I was wondering if deploy/lib is a
> |good place for these rars !
> |
> |I also found that a URL based EJB is not undeployed
> if
> |it is not accessible - it is assumed that the URL
> will
> |exist! Is this the behavior we want ?
> |
> |I do not have write access to CVS.
> |
> |Thanks
> |
> |Ajay
> |
> |--- [EMAIL PROTECTED] wrote:
> |> Bugs item #432844, was updated on 2001-06-13
> 10:53
> |> You can respond by visiting:
> |>
>
|http://sourceforge.net/tracker/?func=detail&atid=376685&aid=432844&;
> |group_id=22866
> |>
> |> Category: JBossServer
> |> Group: v2.2.1 (stable)
> |> Status: Open
> |> Resolution: None
> |> Priority: 5
> |> Submitted By: Dirk Hoffmann (dirkey)
> |> Assigned to: Nobody/Anonymous (nobody)
> |> Summary: NullPointerException in AutoDeployer
> |>
> |> Initial Comment:
> |> If the deploy directory is deleted (by a stupid
> user
> |> ;-
> |> )), the Autodeployer thread stops with a
> |> NullPointerException. After restoring the
> directory
> |> the Autodeployer is not running anymore.
> |>
> |> [Auto deploy] java.lang.NullPointerException
> |> [Auto deploy] at org.jboss.ejb.AutoDeployer.run
> |> (AutoDeployer.java:135)
> |> [Auto deploy] at java.lang.Thread.run
> |> (Thread.java:484)
> |>
> |> Regards,
> |> Dirk
> |>
> |>
>
|----------------------------------------------------------------------
> |>
> |> You can respond by visiting:
> |>
>
|http://sourceforge.net/tracker/?func=detail&atid=376685&aid=432844&;
> |group_id=22866
> |>
__________________________________________________
Do You Yahoo!?
Spot the hottest trends in music, movies, and more.
http://buzz.yahoo.com/
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development
