Skip to content

Conversation

Arkatufus
Copy link
Contributor

No description provided.

Copy link
Member

@Aaronontheweb Aaronontheweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, looks great to me but we need to clarify some things in the documentation

clean: false # whether to fetch clean each time
submodules: recursive # set to 'true' for a single level of submodules or 'recursive' to get submodules of submodules
persistCredentials: true
- task: UseDotNet@2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

vmImage: 'ubuntu-16.04'
scriptFileName: ./build.sh
scriptArgs: all
- template: azure-pipeline.template.yaml
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

displayName: 'NuGet Pack'
vmImage: 'windows-2019'
scriptFileName: build.cmd
scriptArgs: CreateNuget nugetprerelease=dev
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


<PropertyGroup>
<TargetFrameworks>netcoreapp2.1</TargetFrameworks>
<TargetFrameworks>$(NetFrameworkTestVersion);$(NetTestVersion);$(NetCoreTestVersion)</TargetFrameworks>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


namespace Akka.Persistence.MongoDb
{
public class MongoDbPersistenceSetup : Setup
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need some XML-DOC magic here

else
{
client = new MongoClient(setupOption.Value.SnapshotConnectionSettings);
snapshot = client.GetDatabase(setupOption.Value.SnapshotDatabaseName);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

var snapshotDatabaseName = "theSnapshotDatabase"
var journalDatabaseName = "theJournalDatabase"
var setup = BootstrapSetup.Create()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need some more examples:

  1. Journal only
  2. Snapshot store only
  3. Both (which you have included here)

Also need to explain that you still need to set the mongodb plugin in akka.persistence.journal and akka.persistence.snapshot-store in HOCON even with these settings passed in.

Copy link
Member

@Aaronontheweb Aaronontheweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Aaronontheweb Aaronontheweb enabled auto-merge (squash) April 23, 2021 20:22
@Aaronontheweb Aaronontheweb merged commit 85f3d38 into akkadotnet:dev Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants