Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 13:12
    Aaronontheweb labeled #4099
  • 13:12
    Aaronontheweb labeled #4099
  • 13:12
    Aaronontheweb opened #4099
  • 10:41
    Aaronontheweb commented #4097
  • 08:19
    ismaelhamed synchronize #4097
  • 02:22
    kimbyungeun opened #4098
  • Dec 15 19:47

    Aaronontheweb on dev

    TypeExtensions.TypeQualifiedNam… (compare)

  • Dec 15 19:47
    Aaronontheweb closed #4071
  • Dec 15 19:47
    Aaronontheweb closed #3767
  • Dec 15 19:47
    Aaronontheweb labeled #3767
  • Dec 15 19:47
    Aaronontheweb labeled #3767
  • Dec 15 19:47
    Aaronontheweb milestoned #3767
  • Dec 15 19:44
    Aaronontheweb labeled #4097
  • Dec 15 19:44
    Aaronontheweb milestoned #4097
  • Dec 15 13:23
    Aaronontheweb commented #4096
  • Dec 15 13:22
    Aaronontheweb commented #4093
  • Dec 15 13:16
    ismaelhamed commented #4093
  • Dec 15 13:04
    ismaelhamed edited #4097
  • Dec 15 13:04
    ismaelhamed opened #4097
  • Dec 15 12:50
    ismaelhamed commented #4096
Marc Piechura
@marcpiechura
o_O
Aaron Stannard
@Aaronontheweb
blob
I think I've found the issue
 protected EventFilterTestBase(string config)
            : base(@"akka.loggers = [""" + typeof(ForwardAllEventsTestEventListener).AssemblyQualifiedName + @"""], " + (string.IsNullOrEmpty(config) ? "" : config))
        {
            //We send a ForwardAllEventsTo containing message to the TestEventListenerToForwarder logger (configured as a logger above).
            //It should respond with an "OK" message when it has received the message.
            var initLoggerMessage = new ForwardAllEventsTestEventListener.ForwardAllEventsTo(TestActor);
            // ReSharper disable once DoNotCallOverridableMethodsInConstructor
            SendRawLogEventMessage(initLoggerMessage);
            ExpectMsg("OK");
            //From now on we know that all messsages will be forwarded to TestActor
        }
assertion in the constructor of the test
Marc Piechura
@marcpiechura
Man I really can't remember that I ever touched this code, will check my PR's from this time
Aaron Stannard
@Aaronontheweb
smells to me like a race condition where something isn't started before this command fires
but not otherwise
works fine when I debug it
Marc Piechura
@marcpiechura
Sorry can't look into it at the moment, can you show me in which PR I changed that? I can't find it
Aaron Stannard
@Aaronontheweb
honestly I'm kind of puzzled by this
it's not even the EventFilter stuff that broke this
it's like the testkit initialization is fucked up
ah nope
sure enough
there is a bug here I'm pretty sure
Kris Schepers
@schepersk
@Horusiath I just tried my sharding example again, this time with the new nuget packages.. The issue of the shards not being rebalanced is still there I'm afraid.
Bartosz Sypytkowski
@Horusiath
I'll try to take a look at this during the weekend. I'm feeling that fixing multinode test runner spec for cluster sharding should help to answer/predict this issue.
I assume that's not the reason for this ;)
Aaron Stannard
@Aaronontheweb
ah, yeah probably not
I'm going to revert the changes made to the EventFilter if I can't get this to pass
Marc Piechura
@marcpiechura
we haven't used it anyway if I remember correctly
Aaron Stannard
@Aaronontheweb
I have more important bugs to fix
Aaron Stannard
@Aaronontheweb
can't reproduce on build-server
Kris Schepers
@schepersk
@Horusiath Thanks! In the meanwhile I'll go for a simple coordinator for our first version then..
Aaron Stannard
@Aaronontheweb
re-cloning locally
Bartosz Sypytkowski
@Horusiath
@Aaronontheweb some next bug-fix release planning?
Aaron Stannard
@Aaronontheweb
fix the bugs first
then release the fixes :p
that's my plan
Aaronontheweb 2016
but yeah, sure seems like we need to
are you working on that issue with the default Akka.Persistence config?
because that needs to go out today
or soon
Bartosz Sypytkowski
@Horusiath
I'd like to include this issue @schepersk has submitted to it
Aaron Stannard
@Aaronontheweb
get a PR in
and we'll make it happen
I'm working on making sure we didn't break the TestKit at the moment
good news is that I think the answer is "no"
Bartosz Sypytkowski
@Horusiath
I just run the whole suite
when finished, I'll take a look into akkadotnet/akka.net#1863
Aaron Stannard
@Aaronontheweb
ty sir
I'm working on #1700 and #1845
I think I have #1845 fixe
Zetanova
@Zetanova
Is there some kind of circular buffer in akka, to coorelate some Id's ?
Bartosz Sypytkowski
@Horusiath
@Zetanova I guess not
btw. nice talk about new enterprise stack @Aaronontheweb
Aaron Stannard
@Aaronontheweb
@Zetanova Helios has one internally that you can grab from the soruce