These are chat archives for akkadotnet/AkkaStreams

4th
Dec 2018
Arjen Smits
@Danthar
Dec 04 2018 14:29
Hey has the SetHandler method from a TimerGraphStageLogic been removed ?
Since Akka.Streams 1.3.7 ?
Marc Piechura
@marcpiechura
Dec 04 2018 14:32
I only remember that we’ve made it more type safety in the past, maybe that’s the reason why you get an compile error ?
Arjen Smits
@Danthar
Dec 04 2018 14:32
Even better
getting a missing method exception
Using the azure CloudQueue integration
and upgraded to latest akka.stream
s
but it seems its not compatible yet
to fair, that integration hasn't been updates in a while
so im not suprised
This line borks: this.SetHandler((Outlet) source.Out, new Action(this.PullQueue), (Action) null);
So its probably some new signature thing
ill create an issue and downgrade for now
Done. AkkaNetContrib/Alpakka#46 Probably an easy fix. Ill see if i can whip up a PR
anything major that changed between 1.3.7 and now that i need to be aware of ?
Marc Piechura
@marcpiechura
Dec 04 2018 14:37
If I remember correctly it now needs Outlet<T> instead of outlet
Arjen Smits
@Danthar
Dec 04 2018 14:38
lol. Ah yeah that (Outlet) cast would sure ruin that
Marc Piechura
@marcpiechura
Dec 04 2018 14:39
Yup, and you could cast Outlet<string> to Outlet while your handler expects an int ;-)
Results in some nasty type cast exceptions at runtime
Arjen Smits
@Danthar
Dec 04 2018 14:40
funny thing is. Its already an Outlet<CloudQueueMessage>
the downcast is completely bull :P so its a good thing that that interface has been 'fixed'
Marc Piechura
@marcpiechura
Dec 04 2018 14:41
Yup ^^ was probably left over from the „just make it work“ times ;-)
Arjen Smits
@Danthar
Dec 04 2018 14:43
good times
We also got some PR's open on the Alpakka repo
Arjen Smits
@Danthar
Dec 04 2018 14:53
oh, its on akka.streams 1.3.1 even
Arjen Smits
@Danthar
Dec 04 2018 15:00
LOL
i already updated it AkkaNetContrib/Alpakka#36
So in short. Alpakka is in need of a release :D
Marc Piechura
@marcpiechura
Dec 04 2018 17:34
😂