Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Mar 04 13:10
    mysticmind labeled #1747
  • Mar 04 13:10
    mysticmind labeled #1747
  • Mar 04 13:10
    mysticmind labeled #1747
  • Mar 04 13:10
    mysticmind labeled #1747
  • Mar 04 13:09
    oskardudycz commented #1230
  • Mar 04 13:09
    oskardudycz commented #1230
  • Mar 04 13:05
    jedidja opened #1747
  • Mar 04 13:05
    jedidja opened #1747
  • Mar 04 12:59
    jedidja closed #1746
  • Mar 04 12:59
    jedidja closed #1746
  • Mar 04 12:58
    jedidja synchronize #1746
  • Mar 04 12:58
    jedidja synchronize #1746
  • Mar 04 12:50
    jedidja synchronize #1746
  • Mar 04 12:50
    jedidja synchronize #1746
  • Mar 04 12:45
    dnfadmin commented #1746
  • Mar 04 12:45
    dnfadmin commented #1746
  • Mar 04 12:44
    dnfadmin commented #1746
  • Mar 04 12:44
    dnfadmin commented #1746
  • Mar 04 12:44
    jedidja opened #1746
  • Mar 04 12:44
    jedidja opened #1746
Jedidja
@jedidja
Apologies if I missed something in the setup docs.
Jeremy D. Miller
@jeremydmiller
@jedidja You won’t have permission to publish the docs, but to run the docs locally, try npm run docs. @mysticmind just barely got the replacement documentation website for v4 together. And you want to edit what’s in the /docs folder. Sorry for the confusion, but you walked right into a construction zone:)
Jedidja
@jedidja
Ah cool. No problem - just wanted to make sure I was starting from a "good" spot before making any edits.
Babu Annamalai
@mysticmind
You could alternatively provide the change targeting 3.x branch docs, we will pick up the changes into the new v4 branch docs.
If it is an addition/change to the existing docs
@jedidja ^^
Jedidja
@jedidja
@mysticmind It's a fairly minor change to the existing docs, albeit useful for a newbie like me :) So I should probably use the 3.13 branch instead of master?
Babu Annamalai
@mysticmind
We still have to merge few changes from the 3.x docs into v4 so please do on the 3.13 branch, it is easier since your changes will need to be there in both the docs
I will merge the changes in to v4 when I pick up other changes
@jedidja ^^
Jedidja
@jedidja
Will do. Thanks for the confirmation @mysticmind
Babu Annamalai
@mysticmind
:thumbsup:
Babu Annamalai
@mysticmind
@jedidja FYI, with regards to the issues which you were facing running the old docs in master branch, the old docs looks for existence of index.md in each of the documentation folders. In the diagrams folder which was recently added by @jeremydmiller , it was missing the index.md file, added the same. You should be able to run the old docs on master branch as well now.
That said, you wouldn't need to use the old docs though on master/v4 branch
Jedidja
@jedidja
Gotcha. As @jeremydmiller mentioned I picked an odd time to work on docs :)
Jedidja
@jedidja
Huh. Neat. First time I'm seeing this CLA signing mechanism as part of a PR.
Oskar Dudycz
@oskardudycz
@jedidja this is the requirement on project being .NET Foundation member ;)
JT
@Hawxy
Relevant to the above, some nullability annotations might be worth doing within the v4 timeframe :)
Jeremy D. Miller
@jeremydmiller
image.png
Sounds like a good pull request.
Sorry @jedidja, no idea why Gitter decided to stick your avatar in there.
Oskar Dudycz
@oskardudycz
@Hawxy good idea
Kamal S
@kamaltwitts_twitter
Hi all, event store feature in Marten is pretty cool. Thanks @oskardudycz , your repo for event sourcing example was very helpful!. Had couple of questions. 1. If i use Events.Append, is there a way to specify the type of the aggregate. I couldn't find a generic version like StartStream. We are using the Store method that takes in multiple domain models and we use the same method for create/update, so we couldn't use StartStream. 2. Is there a way to store the event and stream on a different database?
Oskar Dudycz
@oskardudycz
Thank you for the kind words! I appreciate that!
  1. Currently it's only possible to provide the stream type during the StartStream. However tbt (in 3.x) it's not used much internally. You can safely use Append.
  1. Not out of the box. You'd need to use different documentstore and documentsession, like e.g. https://github.com/oskardudycz/EventSourcing.NetCore/blob/main/Marten.Integration.Tests/Tenancy/TenancyPerSchema.cs
Kamal S
@kamaltwitts_twitter
cool, thanks for the quick response
Oskar Dudycz
@oskardudycz
:+1:
Oskar Dudycz
@oskardudycz
Btw. I wrote my thoughts about CQRS, tried to explain common myths. I think that for some of you, it can be useful: https://event-driven.io/en/cqrs_facts_and_myths_explained/
Vladislav Plemyannik
@davilko

Hi all, could someone to explain me why compiled query couldn't have nullable fields ? i got such error

Members Nullable<R1.ContractManagement.Domain.RequirementType> RequirementType, Nullable<R1.ContractManagement.Domain.CollectionMethod> CollectionMethod, Nullable<R1.ContractManagement.Domain.ReturnMethod> ReturnMethod, Nullable<System.DateTime> EffectiveDate, Nullable<System.DateTime> ExpiryDate cannot be used as parameters to a compiled query

Oskar Dudycz
@oskardudycz
is it v4?
Jeremy D. Miller
@jeremydmiller
@davilko Because it was extremely hard to support, is why:) You’ll need to use separate compiled queries to get around the Nullable thing.
Vladislav Plemyannik
@davilko
@oskardudycz yes v4-alpha.5
@jeremydmiller ok, thank you for the answer :) Will use not compiled query
Jeremy D. Miller
@jeremydmiller
You can use compiled queries for datetimes, just not DateTime?
Vladislav Plemyannik
@davilko
No, i have a cases when this filed could be null in database and i have to find these records.
For example external service ask to find document by several fields where some of them could be null
Jeremy D. Miller
@jeremydmiller
Gotcha. That’s just not a good case for compiled queries
Vladislav Plemyannik
@davilko
Is there a way to just execute sql script inside session before start querying ? I have an GIN index but postgre thinks it much better to use scan instead, and i want execute SET enable_seqscan = OFF at begin of query
Babu Annamalai
@mysticmind
@davilko possibly, you could do something like below:
await using var session = store.OpenSession();

var cmd = new NpgsqlCommand
{
    CommandText = "SET enable_seqscan = OFF",
    Connection = session.Connection // use the currently open connection from session
};
await cmd.ExecuteNonQueryAsync();

// all other calls on the Marten session
Please give it a shot and let me know how it goes...
Marko Lahma
@lahma
we've started with migration to Marten 4 (still unreleased greenfield project) and we found out that JasperFx/marten#1542 probably never found its way to master, would you like a new PR or is it easier for someone of core maintainers to work their magic?
Oskar Dudycz
@oskardudycz
@lahma I tried to apply that to the current rewritten codebase but I wasn't able to complete that yet
Babu Annamalai
@mysticmind
I will have a look at it @lahma, probably an oversight, there are few things which need to be merged from 3.x, will look at it and keep you posted.
Oskar Dudycz
@oskardudycz
It's not an oversight
The inner mechanism was rewritten and it's not just 1:1 change
it needs to be rewritten to the new patterns
I just wasn't enough onboarded yet on how to do that. We may have also other commits that we didn't backported and we should have a look
Babu Annamalai
@mysticmind
@oskardudycz Sorry, our messages crossed...
Didn't notice you also responded..
Oskar Dudycz
@oskardudycz
:+1:
Marko Lahma
@lahma
thank you @mysticmind and @oskardudycz , great that this is on your radar