Problem/Motivation

Issue fork cloud-3293949

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yas created an issue. See original summary.

yas opened merge request !1153

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented
Title: Set the MySQL transaction isolation level (READ COMMITTED) » Refactor to set the MySQL transaction isolation level (READ COMMITTED)
Status: Needs work » Needs review

@baldwinlouie

Can you please review the patch? Thanks

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented
Status: Needs review » Reviewed & tested by the community

@yas, this looks good to me.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented

@baldwinlouie

Thank you for your review. I'll merge the patch to 4.x and 5.x, and close this issue as Fixed.

  • yas committed 9876475 on 5.x 
    Issue #3293949 by yas, baldwinlouie: Refactor to set the MySQL...

yas opened merge request !1160

  • yas committed 0efd9f4 on 4.x 
    Issue #3293949 by yas, baldwinlouie: Refactor to set the MySQL...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented
Status: Reviewed & tested by the community » Fixed

yas opened merge request !1166

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented
Status: Fixed » Needs review

Fixing a bug for .tugboat/config.xml since the config.xml uses SQLite, not using MySQL

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented
Status: Needs review » Reviewed & tested by the community

@all I'll merge the patch to 4.x and 5.x, and close this issue as Fixed.

  • yas committed 9475708 on 5.x 
    Issue #3293949 by yas, baldwinlouie: Hotfix - Refactor to set the MySQL...

  • yas committed 72420e0 on 4.x 
    Issue #3293949 by yas, baldwinlouie: Hotfix - Refactor to set the MySQL...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.