java - Spring: circular dependencies, @PostConstruct and order imposed by @DependsOn

Question

I was expecting Spring to take @DependsOn into account when calling @PostConstruct methods, but seems like it's not the case in presence of circular (auto-wired) dependencies.

Consider two beans (code below), BeanB @DependsOn BeanA. When field BeanA#b has it's @Autowired commented out, post-construct methods are called in expected order: first A, then B. But with @Autowired in effect for A, I have B's post called first, then A's post.

I understand this is a bad design (actually, it's minimal demo of very big @Autowired ... code-base), but I was expecting Spring to finish injection of @Autowired fields and then starting to call lifecycle callbacks, honoring @DependsOn, but Spring seems to ignore @DependsOn order when there are circular deps.

Spring version is 4.1.5.

So, is this my misunderstanding or undocumented behavior or can it be considered a Spring bug (or, perhaps, feature request)?

@Component
class BeanA {

    // @Autowired
    private BeanB b;

    void f() {
        System.out.println(this);
    }

    @PostConstruct
    void post() {
        System.out.println("A done");
    }

    @Override
    public String toString() {
        return "Bean{" +
                "b=" + (b == null ? null : b.getClass()) +
                '}';
    }
}
// ---------------------
@Component
@DependsOn("beanA")
class BeanB {

    @Autowired
    private BeanA a;

    void f() {
        System.out.println(this);
    }

    @PostConstruct
    void post() {
        System.out.println("B done");
    }

    @Override
    public String toString() {
        return "BeanB{" +
                "a=" + (a == null ? null : a.getClass()) +
                '}';
    }
}

Answer 1

In the chapter about Initialization callbacks, the Spring documentation states

[@PostConstruct and other methods] allows a bean to perform initialization work after all necessary properties on the bean have been set by the container.

With your commented code, the following happens: beanA is instantiated and saved. The container sees that all necessary properties have been set and it invokes the init (@PostConstruct) method. Then it goes to beanB which it initializes, saves, sees an @Autowired, retrieves the saved beanA, injects it, the runs beanB's @PostConstruct since all its properties have been set.

In your uncommented code, you have a case of circular dependencies. beanA gets instantiated first and is saved. The container notices it has an injection target of type BeanB. To perform this injection, it needs the beanB bean. It therefore instantiates the bean, saves it, sees that it has a dependency on a beanA as an injection target. It retrieves the beanA (which was saved earlier), injects it, then beanB's properties are all set and its @PostConstruct method is invoked. Finally, this initialized beanB bean is injected into beanA, whose @PostConstruct method is then invoked since all its properties have been set.

This second has case beanB being constructed while beanA is being constructed. This is how Spring solves the following

class A {
    private B b;
}

class B {
    private A a;
}

An instance of each has to be created before either can be injected into the other.

---

If you get rid of the @DependsOn, you'll get the same behavior (but just because of the default ordering of classpath scanning, which seems to be alphabetical). If you renamed BeanA to BeanZ, for example, the beanB will be instantiated first, then beanZ would get instantiated, initialized, and returned to be injected into beanB.

@DependsOn is really only necessary if you have side effects that you'd like to happen before a bean is initialized.