The only part that makes a circular dependency is including the MiniAccumuloRunner in the Main.class. I'm not sure if that warrants needing to rearchitect the runner, since it was made to give users the ability to interact with the Miniaccumulocluster as a single node accumulo. It was also made to make the maven plugin much easier and standardize the interface. Seems like two options are to remove the runner option from the Main.class or move it to the start module.
Personally, I'd opt for moving the runner to the start module. On May 20, 2013 8:12 AM, "David Medinets" <david.medin...@gmail.com> wrote: > Combine this work with Dave Marion's work and put MiniAccumuloRunner into > an add-on script? > > > On Mon, May 20, 2013 at 7:49 AM, Corey Nolet <cjno...@gmail.com> wrote: > > > I think the ability to run "./bin/accumulo mini" may have introduced this > > circular dependency. Perhaps the MiniAccumuloRunner should be moved > > somewhere else. > > On May 20, 2013 12:07 AM, "Christopher" <ctubb...@apache.org> wrote: > > > > > What do you mean there isn't a way to know this? Doesn't the server > > > POM express the dependency on start, explicitly? > > > > > > How are you executing MiniAccumuloCluster when you get this error? I'm > > > guessing you have a project with a dependency on "accumulo-server"? > > > > > > Can you elaborate more on this circular dependency risk you mention? > > > I'm not sure how "accumulo-server" depending on "accumulo-start" is a > > > circular dependency. Are you suggesting "accumulo-start" also has a > > > dependency on "accumulo-server"? Because... it shouldn't... > > > > > > -- > > > Christopher L Tubbs II > > > http://gravatar.com/ctubbsii > > > > > > > > > On Sun, May 19, 2013 at 10:10 PM, Jim Klucar <klu...@gmail.com> wrote: > > > > MiniAccumuloCluster moved to the 'server' module, which is fine. > > > However, I > > > > now have to have a dependency on accumulo-start and accumulo-server > to > > be > > > > able to use the MiniAccumuloCluster. There isn't really any way of > > know > > > > that until you fire up a MiniAccumloCluster object and call .start() > > and > > > > get a ClassNotFoundException on the Main class from accumulo-start. > > > Seems > > > > like depending on accumulo-server should pull in accumulo-start > > (creates > > > > circular dependency?) or some other solution should be found. > > > > > >
