Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 01:09
    billgao87 starred exceptionless/Exceptionless
  • Sep 20 13:20

    niemyjski on elastic7

    Fixed more unit tests (compare)

  • Sep 20 12:11
    niemyjski commented #447
  • Sep 20 07:42
    ZhiqiangTao commented #447
  • Sep 19 22:56

    niemyjski on elastic7

    bumped major and deps (fixed te… (compare)

  • Sep 19 14:20

    niemyjski on elastic7

    Fixed all issues with the event… (compare)

  • Sep 19 07:04
  • Sep 19 04:49
  • Sep 19 03:32
  • Sep 18 12:46
    LydellM commented #448
  • Sep 18 12:38
    niemyjski labeled #448
  • Sep 18 12:37
    niemyjski closed #448
  • Sep 18 12:37
    niemyjski commented #448
  • Sep 18 12:37
    niemyjski commented #448
  • Sep 18 12:35
    niemyjski commented #447
  • Sep 18 07:09
    ZhiqiangTao closed #447
  • Sep 18 05:31
    leehomlee starred exceptionless/Exceptionless
  • Sep 18 03:17
  • Sep 18 02:13

    niemyjski on elastic7

    More ES7 Changes (compare)

  • Sep 17 20:34
    LydellM opened #448
Blake Niemyjski
@niemyjski
just added a quick comment
just came across that for pandora :)
now I only need to run chrome for debugging js
and it’s oss
Blake Niemyjski
@niemyjski
I just added a comment that I think we should trim the scope and check for isnullorwhitespace
then ship it
if we apply it to the Check to Scope then we could use that for the key prefix and they’d be in sync
Scope = !String.IsNullOrWhitespace(scope) ? scope.Trim() : String.Empty;
_keyPrefix = String.Concat(Scope, ":");
crap, still doesn’t fix it where your keyprefix could just be :
are we fine with it just being :
I’d personally not have a : if it’s empty or null
Jamie Clarke
@jamie94bc
Scope = !String.IsNullOrWhiteSpace(scope) ? scope.Trim() : null;

            _keyPrefix = scope != null ? String.Concat(scope, ":") : String.Empty;
Blake Niemyjski
@niemyjski
yeah on the bottom one use Scope
because it’s already been trimmed
or null
Jamie Clarke
@jamie94bc
oh yeah haha :)
Blake Niemyjski
@niemyjski
then ship it :)
Jamie Clarke
@jamie94bc
damn casing - this is why i tend to use this.
but i tried not to since to match your coding style
Blake Niemyjski
@niemyjski
hehe yeah
I don’t think we ever use this.
nice find on that :)
Jamie Clarke
@jamie94bc
yeah - was getting some logs that didn't make sense!
Blake Niemyjski
@niemyjski
:)
Blake Niemyjski
@niemyjski
@frankebersoll have you ever saw this with bower
bower exceptionless#1.3.2 install exceptionless#1.3.2
bower skipped exceptionless was not installed because there is already a non-bower directory with that name in the components directory (bower_components/exceptionless). You can force installation with --force.
I’ve always installed it via bower
so not sure why I get that
Eric J. Smith
@ejsmith
so another flakey test?
Jamie Clarke
@jamie94bc
yeah!
Eric J. Smith
@ejsmith
yay
at least the logging helps a lot for tracking those down.
Blake Niemyjski
@niemyjski
just thought of something
var valueMap = new Dictionary<string, CacheValue<T>>();
we could put the old dictionary length in the constructor so if it’s a huge existing dictionary we won’t be changing the size internally every 2^nth
Eric J. Smith
@ejsmith
where is that for?
Blake Niemyjski
@niemyjski
in the scoped cache client
I’m going to make that change real quick
Eric J. Smith
@ejsmith
that only works for in memory implementations, no?
Blake Niemyjski
@niemyjski
no
all of them
hmm
ToDictionary overload doesn’t look at the count
when creating it
Eric J. Smith
@ejsmith
how would you know how many items are in reds?
redis
Blake Niemyjski
@niemyjski
it does a getall
and then iterates over that..
Eric J. Smith
@ejsmith
huh???