search

apache / pekko

Build highly concurrent, distributed, and resilient message-driven applications using Java/Scala
https://pekko.apache.org/
Apache License 2.0
1.22k stars 148 forks source link

Scala 3.3 docs.dispatcher.DispatcherDocSpec java.lang.IllegalArgumentException: no matching constructor found on #1081

Open He-Pin opened 9 months ago

He-Pin commented 9 months ago 
[01-30 07:45:26.559] [info] - using Identify *** FAILED *** (0 milliseconds)
[01-30 07:45:26.559] [info]   java.lang.IllegalArgumentException: no matching constructor found on class docs.actor.ActorDocSpec$$anon$32$Follower for arguments [class docs.actor.ActorDocSpec$$anon$32]
[01-30 07:45:26.559] [info]   at org.apache.pekko.util.Reflect$.error$1(Reflect.scala:98)
[01-30 07:45:26.559] [info]   at org.apache.pekko.util.Reflect$.findConstructor(Reflect.scala:122)
[01-30 07:45:26.559] [info]   at org.apache.pekko.actor.ArgsReflectConstructor.<init>(IndirectActorProducer.scala:109)
[01-30 07:45:26.559] [info]   at org.apache.pekko.actor.IndirectActorProducer$.apply(IndirectActorProducer.scala:74)
[01-30 07:45:26.559] [info]   at org.apache.pekko.actor.Props.producer(Props.scala:141)
[01-30 07:45:26.560] [info]   at org.apache.pekko.actor.Props.<init>(Props.scala:154)
[01-30 07:45:26.560] [info]   at org.apache.pekko.actor.Props$.apply(Props.scala:123)

refs: https://github.com/apache/incubator-pekko/actions/runs/7707379313/job/21004465786#step:7:35225

pjfanning commented 9 months ago

This also affects the 1.0.x branch - i.e this is not a new issue

pjfanning commented 8 months ago

Can we leave this open until we fix 1.0.x branch? But we should wait a day or 2 before merging to 1.0.x. The issue is this code is not properly tested in PR builds - so we need the nightly build to run to test this.

He-Pin commented 8 months ago

I tested with manually trigger, it works , but genjavadoc does not work on scala 3.

OK, let's leave it open before backported to 1.0.x

pjfanning commented 8 months ago

looks like the main branch nightly build is better so we should backport this change to 1.0.x