Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Issue / Domain Layer / Performance Comparison #88

Merged
merged 34 commits into from
Dec 29, 2023
Merged

Issue / Domain Layer / Performance Comparison #88

merged 34 commits into from
Dec 29, 2023

Conversation

dncsvr
Copy link
Member

@dncsvr dncsvr commented Dec 22, 2023 •
edited by cihandeniz
Loading

Create a benchmark project and test getting typeinfo from assembly vs
getting typeinfo from a prebuilt static field. Main purpose is to decide if
it is worth to create a DomainModel.cs instead of using reflection.

Tasks

  • Setup a project for performing benchmark tests
    • add model records
  • Setup a baseline benchmark using reflection
    • Add T4 template for generating entites, query and operation object classes
    • Add benchmark method which uses reflection
  • Setup a benchmark using domain model approach
    • Add source generator poject for generating domain model
    • Add benchmark method which uses domain model
  • Refine benchmark tests
    • check for same conventions, make sure generated api models are equal
    • Add a unit test project for validating created ControllerModel
    • Add IDomainModel implementation which is built from reflection
    • Reorganize solution
    • Add a research documentation and fine tune benchmarks tests

    reflection is currently faster than domain model approach, try to speed up domain model

@dncsvr
add setup test console app for performance benchmark
5f998ae 
- setup project for designing performance tests
- add `EntityTemplate.tt` for generating entity classes
- add `BenchmarkSpec` for testing do
- add `Build` test class to contain methods for benchmark
@dncsvr dncsvr self-assigned this Dec 22, 2023
@dncsvr dncsvr changed the base branch from main to epic/domain-layer December 22, 2023 11:05
@dncsvr
remove benchmark project from do
65d0d62
@dncsvr
add DomainModelOverReflection research project
9da24da 
- add `research.sln`
@dncsvr
add model records
57a57cb
@dncsvr
setup benchmark test for using reflection case
c87129f
@dncsvr
edit reflection test
c9cd844 
- add `OperationObject` template
- improve `ApiModel build
@dncsvr
updates for controller model generation from domain model
3f9fdaf 
- Add helper methods for using model dtos
- Add a seperate class library for domain objects
@dncsvr
setup a benchmark test using domain model
cafe770 
- Add source generator project for building domain model
- Restructure project
@dncsvr
minor edits
056877b
@cihandeniz cihandeniz mentioned this pull request Dec 27, 2023
Merged
9 tasks
dncsvr added 18 commits December 28, 2023 10:59
@dncsvr
edit domain model generation and template files
8eaec36
@dncsvr
add unit test project for validating controller model
a6b5d6c 
- exclude generated entity and operation object files
- minor edits
@dncsvr
minor edit
a2d6d58
@dncsvr
setup building controller model test cases and implementation
8531710 
- move api model dtos to models project
@dncsvr
edit test expected result
79acd63 
- add a test operation object
- minor edits
@dncsvr
refactoring domain model
e6fdbb5 
- convert domain dtos to classes
- change Type Type to string Type
- change Lists to Arrays for generated
for generated domain model
@dncsvr
refactor domain model
f1a15d4 
- change dtos to structs
- give arrays fixed size
@dncsvr
include generated entity and operation object classes
a0ef22d
@dncsvr
run benchmark tests with net 8
42813f0
@dncsvr
remove .g suffix from templates !!! BUILD FAILING !!!
dcbab20
@dncsvr
update Domain.csproj
9369839
@dncsvr
edit directory.build.props
7c398ec
@dncsvr
edit directory.build.props
693f9dc
@dncsvr
add domain model with reflection and runtime reflection cases
2e95b88 
- reduce generated domain model overhead with nullable arrays
- reduce generated domain model overhead with excluding implicitly declared methods
@dncsvr
remove properties from proj files to use directory.build.props
5a22491 
- minor edits
@dncsvr
rename projects
2097690
@dncsvr
add a research documentation
90c54ca
@dncsvr
add benchmark result images to docs
1ab01d9
@dncsvr dncsvr marked this pull request as ready for review December 29, 2023 14:27
@dncsvr dncsvr requested a review from cihandeniz December 29, 2023 14:29
@dncsvr dncsvr merged commit 1cd7791 into mouseless:epic/domain-layer Dec 29, 2023
4 checks passed
@dncsvr dncsvr deleted the issue/domain-layer/performance-comparison branch December 29, 2023 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cihandeniz cihandeniz cihandeniz approved these changes

Assignees

@dncsvr dncsvr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dncsvr @cihandeniz