Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Mention SQL logger impact on batch processing#7849

Merged
lcobucci merged 2 commits into
doctrine:2.6from
axi:patch-1
Oct 2, 2019
Merged

Mention SQL logger impact on batch processing#7849
lcobucci merged 2 commits into
doctrine:2.6from
axi:patch-1

Conversation

@axi
Copy link
Copy Markdown
Contributor

@axi axi commented Oct 1, 2019

Looking for a way to improve one of our bulk update treatment, I went back to this page then found elsewhere that setting logger to null was a really effective way to improve time and memory consumption. Might be a right place to state it ? Don't know if my edit style is ok

Looking for a way to improve one of our bulk update treatment, I went back to this page then found elsewhere that setting logger to null was a really effective way to improve time and memory consumption. Might be a right place to state it ? Don't know if my edit style is ok
Copy link
Copy Markdown
Member

@lcobucci lcobucci left a comment

Choose a reason for hiding this comment

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

@axi thanks for your contribution, what do you think about this?

Comment thread docs/en/reference/batch-processing.rst
@lcobucci lcobucci added this to the 2.6.5 milestone Oct 1, 2019
@lcobucci lcobucci changed the title Update batch-processing.rst Mention SQL logger impact on batch processing Oct 1, 2019
Copy link
Copy Markdown
Member

@lcobucci lcobucci left a comment

Choose a reason for hiding this comment

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

@axi thanks for your contribution 👍

@lcobucci lcobucci merged commit b226043 into doctrine:2.6 Oct 2, 2019
@lcobucci lcobucci assigned lcobucci and unassigned axi Oct 2, 2019
@axi
Copy link
Copy Markdown
Contributor Author

axi commented Oct 2, 2019

I knew I would do something for doctrine someday ;-) Keep going !

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants