Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 13:32
    phinett opened #2243
  • 13:32
    phinett opened #2243
  • 10:30
    VilleHakli opened #2242
  • 10:30
    VilleHakli opened #2242
  • 09:32
    elexisvenator commented #2146
  • 09:32
    elexisvenator commented #2146
  • 09:30
    elexisvenator commented #2146
  • 09:30
    elexisvenator commented #2146
  • 05:44
    dnfadmin commented #1843
  • 05:44
    dnfadmin commented #1843
  • May 24 16:20

    jeremydmiller on 5.4.0

    (compare)

  • May 24 16:20

    jeremydmiller on 5.4.0

    (compare)

  • May 24 15:46

    jeremydmiller on master

    bumping the ancillary nugets to… (compare)

  • May 24 15:46

    jeremydmiller on master

    bumping the ancillary nugets to… (compare)

  • May 24 15:26

    jeremydmiller on master

    Bumping to 5.4 (compare)

  • May 24 15:26

    jeremydmiller on master

    Bumping to 5.4 (compare)

  • May 24 15:24
    jeremydmiller closed #2234
  • May 24 15:24
    jeremydmiller closed #2234
  • May 24 15:24

    jeremydmiller on master

    wip: upgraded Weasel, big progr… New functionality for writing t… Renamed to ExperimentalMultiStr… and 1 more (compare)

  • May 24 15:24

    jeremydmiller on master

    wip: upgraded Weasel, big progr… New functionality for writing t… Renamed to ExperimentalMultiStr… and 1 more (compare)

Tony Karalis
@tonykaralis
I had no idea ndepend had that feature
Oskar Dudycz
@oskardudycz
@jeremydmiller @mysticmind @jokokko I've made a try to update Marten to npgsql 4.1.0
although it seems to be marked as minor, then imho it contains breaking changes (at least to dependencies)
it'll force us to use at least 4.6.1 .NET and JSON.NET in v12
With JSON.NET v12 we observed some performance degradation
and with the .NET we have right now 4.6
I'd feel more safer to make this v4.0, although pragmatically we need to do this upgrade for our users
Thoughts?
about 4.6.1 upgrade I think that we could even make a step forward to 4.6.2 as afair it's the version that microsoft supports
Jeremy D. Miller
@jeremydmiller
@baronfel had warned me on twitter that we would hit that. They might be trying to retrofit some of the missing stuff
Oskar Dudycz
@oskardudycz
They already did with IsTransactionComplete
So big thanks to @baronfel being on the watch
But 4.1.0 is already released so they won't downgrade the references
So we need to leave with that (although I hate this upgrade process)
After we decide what to do with 4.1.0, I’ll also check the 5.0 pre release. I have foreseen more troubles there - so it's better to start preparation and give us and them the chance to discuss the potential issues.
Babu Annamalai
@mysticmind
@jeremydmiller @oskardudycz @jokokko Looks like Npgsql 4.1.0 have broken semantic versioning... Can we restrict the version of Npgsql being used by Marten to 4.0.x? This way we can avoid users raising issues upgrading Npgsql to 4.1.
Thoughts?
Oskar Dudycz
@oskardudycz
Probably we should, as we did with <4.0 some time ago
Although the problem will still remain
Probably some users might want to use a newer version - eg. when they are using it together with the EF or Dapper
Babu Annamalai
@mysticmind
Yeah, but it atleast reduces the surface area the problems...
Oskar Dudycz
@oskardudycz
Yes, at least it will cut some weird bugs questions from the users as we had last time
I’ll prepare the PR for that
I’ll also check if we can drop using Npgsql.json.net package, that’s forcing the JSON.net upgrade
Babu Annamalai
@mysticmind
Sure
Oskar Dudycz
@oskardudycz
I think that if we move to support 462 or 461 then we could probably also go with the minor update
Although I really don't think comfortable with not following strictly semantic versioning...
It’s even more confusing for the users more than it is for us...
We could, in theory, go with 4.0 just to be prim and proper
but then it won’t be good from the marketing perspective
As we already officially started a discussion on the plans for the v4
Babu Annamalai
@mysticmind
Note that there is already an issue raised by a user who attempted upgrading to Npgsql 4.1.0. JasperFx/marten#1359
Oskar Dudycz
@oskardudycz
Yeah, but it’s from v3
I think that we need to do a better job of being more proactive in the version checking (eg. as @baronfel did for us). I was thinking about some nightly build with the latest prerelease of npgsql
Oskar Dudycz
@oskardudycz
I’ll probably also create issues with that for them, but imho we should try to be closer to them, as it’s our major reference package
Babu Annamalai
@mysticmind
I think the user has incorrectly put details as v3
However close we work with Npgsql versions, any breaking changes will take time to resolve for us
Nightly build with pre-release versions of Npgsql would certainly help us to uncover issues sooner than later
Oskar Dudycz
@oskardudycz
Yes I agree, but if we work closely with them then we can discuss with them turnaround as @baronfel together with me did
In general, I’m trying to find some pragmatic approach
😉
Babu Annamalai
@mysticmind
We depend fully on Npgsql so it imperative for us to figure out a pragmatic approach :-)
Oskar Dudycz
@oskardudycz
Exactly - yelling at clouds won't help 😋
(unfortunately 😉)
Jeremy D. Miller
@jeremydmiller
I say to limit it to 4.0 to <4.1 for right now
Oskar Dudycz
@oskardudycz
Ok, I checked, Npgsql.Json.NET can't be removed
JasperFx/marten#1360
here's the PR
Oskar Dudycz
@oskardudycz
I discussed with @roji on https://gitter.im/npgsql/npgsql the plan for the upgrade. They're willing to revert the JSON.NET upgrade
So the plan would be for this time being we'll:
  • lock the version to <4.1
  • review other potential issues and send the feedback to npgsql team
  • wait for the fixes in 4.1.1
  • make the update