StackOverflowError in IDEA

[axonframework]

StackOverflowError in IDEA

[klauss42]

Just to confirm: New version 0.8.5 now works in IDEA

[CodeDrivenMitch]

@klauss42 Can you share with me your aggregate structure? It's seems there is a recursivity somewhere in the model, and we get an infinite loop in our analysis. Of course I can make an iteration guard, but I'd like to see if there's a possibility I missed in the structure resolver.

[klauss42]

Hi Mitchell,
we have a lot aggregates in our project, but as you mentioned recursion, here is a simplified extract from an aggregate, that has been introduced recently. Maybe this kind of structure breaks the plugin?

The OrganizationalUnitAggregate holds a tree of organizational units to represent an arbitrary organization structure.

data class OrganizationalUnitId(val id: String = UUID.randomUUID().toString())

// Represents the root-OU of an OU-hierarchy-tree
@Aggregate
internal class OrganizationalUnitAggregate : OrganizationalUnitNode<OrganizationalUnitEntity> {

  @AggregateIdentifier
  override lateinit var organizationalUnitId: OrganizationalUnitId

  @AggregateMember
  override val children = mutableListOf<OrganizationalUnitEntity>()

}

// Represents a non-root OU in the OU-hierarchy-tree
internal class OrganizationalUnitEntity(
  @EntityId
  override var organizationalUnitId: OrganizationalUnitId,
) : OrganizationalUnitNode<OrganizationalUnitEntity> {

  @AggregateMember
  override val children: MutableList<OrganizationalUnitEntity> = mutableListOf()
}

// Represents any node of an OU-hierarchy-tree, either the root (Aggregate) or any
// other non-root-node (Entity); T = Type of children
internal interface OrganizationalUnitNode<T> where T : OrganizationalUnitNode<T> {

  var organizationalUnitId: OrganizationalUnitId
  val children: MutableList<T>

  fun addChild(child: T) {
    children.add(child)
  }

  fun removeChild(childOrganizationalUnitId: OrganizationalUnitId) {
    children.removeIf { node -> node.organizationalUnitId == childOrganizationalUnitId }
  }
}

Klaus

[CodeDrivenMitch]

Yep that will be the recursion! I made a PR that will resolve this. I will release 0.8.5 as soon as that's merged.