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

Lazy loading should not overwrite a property value set by user #988

Closed
ataverasp opened this Issue Apr 21, 2017 · 7 comments

Comments

Projects
None yet
3 participants
@ataverasp

ataverasp commented Apr 21, 2017

MyBatis version

3.4.4

Database vendor and version

Oracle 12c

Test case or example project

If I call setter with my own value, and call getter later.

I am expecting I get my own value back, i.e. lazy-loading shouldn't kick in at all.

However, the change makes the lazy-loading kick-in when I call getter, and erase my own value.

Steps to reproduce

  • Load an entity with lazy load enabled
  • Call a setter from a lazy loadable property
  • Call get on that property

Expected result

I should get the value that I set back

Actual result

I'm getting the value from the lazy loading/overriding my value

@harawata

This comment has been minimized.

Show comment
Hide comment
@harawata

harawata Apr 22, 2017

Member

@ataverasp , @lyiu18 , @viruscamp
Please try the snapshot and verify it works as you guys expect.

Member

harawata commented Apr 22, 2017

@ataverasp , @lyiu18 , @viruscamp
Please try the snapshot and verify it works as you guys expect.

@lyiu18

This comment has been minimized.

Show comment
Hide comment
@lyiu18

lyiu18 Apr 23, 2017

works great

lyiu18 commented Apr 23, 2017

works great

@ataverasp

This comment has been minimized.

Show comment
Hide comment
@ataverasp

ataverasp Apr 25, 2017

it works good for me too

ataverasp commented Apr 25, 2017

it works good for me too

@harawata

This comment has been minimized.

Show comment
Hide comment
@harawata

harawata Apr 25, 2017

Member

@lyiu18 @ataverasp Thank you for the clarification!

Member

harawata commented Apr 25, 2017

@lyiu18 @ataverasp Thank you for the clarification!

@lyiu18

This comment has been minimized.

Show comment
Hide comment
@lyiu18

lyiu18 May 8, 2017

@harawata do we have a schedule when 3.4.5 will be released?

lyiu18 commented May 8, 2017

@harawata do we have a schedule when 3.4.5 will be released?

@harawata

This comment has been minimized.

Show comment
Hide comment
@harawata

harawata May 10, 2017

Member

@lyiu18 ,
I'm sorry for the inconvenience.
Usually, we release new version every 3-4 months, but because we are volunteers working on our spare time, it's not possible for us to make a release schedule and follow it.

It would be great if you could try the latest snapshot on your development environment.
Then we can detect and fix issues like this before final release.

Regards,
Iwao

Member

harawata commented May 10, 2017

@lyiu18 ,
I'm sorry for the inconvenience.
Usually, we release new version every 3-4 months, but because we are volunteers working on our spare time, it's not possible for us to make a release schedule and follow it.

It would be great if you could try the latest snapshot on your development environment.
Then we can detect and fix issues like this before final release.

Regards,
Iwao

@lyiu18

This comment has been minimized.

Show comment
Hide comment
@lyiu18

lyiu18 May 10, 2017

@harawata, no worries, thanks for the response.

lyiu18 commented May 10, 2017

@harawata, no worries, thanks for the response.

@harawata harawata changed the title from Lazy loading is overriding value post setting my own value to Lazy loading should not overwrite a property value set by user Aug 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment