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.

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

Bug: BulkInsert doesn't use PropertyHandler #1008

Open
Garios opened this issue Dec 13, 2021 · 5 comments
Open

Bug: BulkInsert doesn't use PropertyHandler #1008

Garios opened this issue Dec 13, 2021 · 5 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request known-bug A known-bug that has been a part of the release. pre-discussed Pre-discussed and/or pre-collaborated with the reported todo Things to be done in the future

Comments

@Garios
Copy link

Garios commented Dec 13, 2021

Bug Description

I wrote a PropertyHandler to convert TimeSpan-(Ticks) to (db-)bigint.

On InsertAsync() PropertyHandlers works fine.
On BulkInsertAsync() PropertyHandlers aren't used.

FluentMapper.Entity<TABLE>().PropertyHandler<PropertyHandlerTimeSpan>(p => p.Duration);

Example: RepoDb v1.12.9 (also 1.12.10-beta4) and RepoDb.SqlServer v1.1.4 (1.1.5-beta3) and SqlServer.BulkOperations 1.1.5 (also 1.1.6-beta1)

@Garios Garios added the bug Something isn't working label Dec 13, 2021
@mikependon
Copy link
Owner

Somebody place this issue on our RepoDB.NET site.
Referencing: mikependon/RepoDB.NET#41

@mikependon
Copy link
Owner

Yes, this is not triggered on the Bulk Operations yet (both SQL Server and PostgreSQL). This is on our timeline.

@Garios
Copy link
Author

Garios commented Dec 13, 2021

oh, I didn't realized that there is another repo. Thank you!

@mikependon
Copy link
Owner

This is the right place, the other repo is for the documentation site.

@mikependon mikependon added enhancement New feature or request known-bug A known-bug that has been a part of the release. pre-discussed Pre-discussed and/or pre-collaborated with the reported todo Things to be done in the future labels Jan 25, 2022
@mikejpappas
Copy link

Would be great to have this enhancement. I ran into it a few moments ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request known-bug A known-bug that has been a part of the release. pre-discussed Pre-discussed and/or pre-collaborated with the reported todo Things to be done in the future
Projects
None yet
Development

No branches or pull requests

3 participants