Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 04:03

    ejsmith on mappings-refactor

    (compare)

  • 04:02
    ejsmith synchronize #452
  • 04:02

    ejsmith on elastic7

    Change a bunch of index field s… Updated dependencies. Revert to 2201-pre and 6 more (compare)

  • 04:02
    ejsmith closed #470
  • 04:02
    ejsmith edited #470
  • 03:59
    ejsmith opened #470
  • 03:14

    niemyjski on mappings-refactor

    Updated to latest foundatio (compare)

  • 02:59

    niemyjski on mappings-refactor

    Tweaked the event index mapping… (compare)

  • 01:53

    ejsmith on mappings-refactor

    Update cache client list method… (compare)

  • 01:19

    ejsmith on mappings-refactor

    More index work (compare)

  • Dec 10 13:26
    niemyjski commented #468
  • Dec 10 13:26
    niemyjski closed #469
  • Dec 10 13:26
    niemyjski commented #469
  • Dec 10 09:27
    xuyi100200 starred exceptionless/Exceptionless
  • Dec 10 08:55
  • Dec 10 05:34

    ejsmith on mappings-refactor

    Some indexing changes... still … (compare)

  • Dec 09 13:10
    niemyjski commented #469
  • Dec 07 03:15
    wuzongwen commented #469
  • Dec 06 13:39
    niemyjski commented #469
  • Dec 06 10:46
    wuzongwen opened #469
Eric J. Smith
@ejsmith
don’t know what you mean
Blake Niemyjski
@niemyjski
look at the above log message
Blake Niemyjski
@niemyjski
updated the unit test and it’s failing
Blake Niemyjski
@niemyjski
            c.UseLogger(new XunitExceptionlessLog(_writer));
:D
got awesome logging now in our client unit tests
Blake Niemyjski
@niemyjski
Checking event: with hash: 738490750
Checking event: with hash: -717263284
Enqueueing event with hash:-717263284 to cache.
Checking event: with hash: -717263284
Enqueueing event with hash:-717263284 to cache.
Checking event: with hash: -717263284
Enqueueing event with hash:-717263284 to cache.
Enqueueing event with hash:738490750 to cache.
Checking event: with hash: -717263284
Checking event: with hash: -717263284
def having a threading issue
with the plugin
changed the forloop to a parallel 4 and everything broke loose
Blake Niemyjski
@niemyjski
@srijken would you be willing to help me look into why the concurrent ones are failing? I added unit tests with improved logging
I think we might need to add some locking code in there
which is both scary and sucks
Blake Niemyjski
@niemyjski
I Added a lock around all of it
and it increased the run time quite a bit
I’m also seeing a weird issue where 1 out of 4 runs fails with invalid hashcode
Enqueueing event with hash:-1799816918 to cache.
Adding event with hash:-1799816918 to cache.
Enqueueing event with hash:825815748 to cache.
Ignoring duplicate event with hash:-1799816918
Ignoring duplicate event with hash:-1799816918
Ignoring duplicate event with hash:-1799816918
Ignoring duplicate event with hash:-1799816918
Ignoring duplicate event with hash:-1799816918
Ignoring duplicate event with hash:-1799816918
Ignoring duplicate event with hash:-1799816918
same code paths and event being generated
Blake Niemyjski
@niemyjski

DeduplicationBenchmarks_IdenticalExceptions, 116 runs

DeduplicationBenchmarks_IdenticalExceptions - 0.01ms

DeduplicationBenchmarks_LargeEventsFromFiles, 103 runs

DeduplicationBenchmarks_LargeEventsFromFiles - 0.30ms

DeduplicationBenchmarks_RandomExceptions, 110 runs

DeduplicationBenchmarks_RandomExceptions - 0.01ms

that’s what it was before my change
lets see what it is on the build server now
@ejsmith @srijken feedback on this is greatly appreciated: exceptionless/Exceptionless.Net@9a34941

DeduplicationBenchmarks_IdenticalExceptions, 114 runs

DeduplicationBenchmarks_IdenticalExceptions - 0.01ms

DeduplicationBenchmarks_LargeEventsFromFiles, 214 runs

DeduplicationBenchmarks_LargeEventsFromFiles - 0.31ms

DeduplicationBenchmarks_RandomExceptions, 112 runs

DeduplicationBenchmarks_RandomExceptions - 0.01ms

didn’t seem to slow it down any
Blake Niemyjski
@niemyjski
@frankebersoll https://github.com/sergeyt/karma-typescript-preprocessor/issues/30#issuecomment-99655328 looks like the tooling is starting to support typescript a bit better :)
Sander Rijken
@srijken
Hi, reading
agree on removing those properties from gethashcode
locking thing.. it’s weird that it’s necessary?
concurrent queue should fix concurrency issues
Sander Rijken
@srijken

DuplicateCheckerPlugin: Checking event: with hash: -1999577676
DuplicateCheckerPlugin: Checking event: with hash: -25846723
DuplicateCheckerPlugin: Checking event: with hash: -25846723
DuplicateCheckerPlugin: Checking event: with hash: -688792890
DuplicateCheckerPlugin: Checking event: with hash: -25846723
DuplicateCheckerPlugin: Checking event: with hash: -1999577676
DuplicateCheckerPlugin: Enqueueing event with hash:-1999577676 to cache.
DuplicateCheckerPlugin: Enqueueing event with hash:-25846723 to cache.
DuplicateCheckerPlugin: Enqueueing event with hash:-1999577676 to cache.
DuplicateCheckerPlugin: Enqueueing event with hash:-25846723 to cache.
DuplicateCheckerPlugin: Enqueueing event with hash:-688792890 to cache.
DuplicateCheckerPlugin: Enqueueing event with hash:-25846723 to cache.

^^ What’s wrong with that?

except for the hash calculations
Blake Niemyjski
@niemyjski
yeah
well those are all called concurrently
and they are all enqueued
when some of those should be ignored
Sander Rijken
@srijken
that’s because the hashes don’t match up
Blake Niemyjski
@niemyjski
yeah
look
Sander Rijken
@srijken
so we must make sure they do match?
Blake Niemyjski
@niemyjski
-2584673 all of those were enqueued and were the same number
same for the -1999577676
yeah there is something goofy going on
Sander Rijken
@srijken
oh wait
it’s enqueueing them too often
Blake Niemyjski
@niemyjski
yeah
idealy no lock would be good but if you look at it without one it’s chaos
Sander Rijken
@srijken
can you add timestamps to the log output?
I’m pretty convinced it should work without the lock
the lock doesn’t fix the issue either right?
Blake Niemyjski
@niemyjski
yeah we could do that by updating the xunit base class