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

MySQL GUID case sensitive #2078

Merged
merged 8 commits into from Oct 25, 2018
Merged

Conversation

Alkarex
Copy link
Member

@Alkarex Alkarex commented Oct 24, 2018

latin1_bin
#2077

@Alkarex Alkarex added the Work in progress 🚧 Wait before merging label Oct 24, 2018
@Alkarex
Copy link
Member Author

Alkarex commented Oct 24, 2018

The current approach is good for cron updates, but not good for Ajax updates (which perform several requests in parallel). I will find another approach.
Fixed

@Alkarex Alkarex added Work in progress 🚧 Wait before merging and removed Work in progress 🚧 Wait before merging labels Oct 25, 2018
@Alkarex Alkarex removed the Work in progress 🚧 Wait before merging label Oct 25, 2018
@Alkarex Alkarex merged commit 307e699 into FreshRSS:dev Oct 25, 2018
@Alkarex Alkarex deleted the mysql-guid-sensitive branch October 25, 2018 20:43
@Alkarex
Copy link
Member Author

Alkarex commented Oct 25, 2018

@Dan-Q Merged in /dev . Tests welcome

Alkarex added a commit that referenced this pull request Oct 25, 2018
javerous pushed a commit to javerous/FreshRSS that referenced this pull request Jan 20, 2020
* MySQL GUID case sensitive

latin1_bin
FreshRSS#2077

* Prepare update for existing bases

* Perform DB update during actualize

* Reduce frequency slightly

* No optimize at the same time

* Take advantage of the SQL modifications in 1.12

* Move higher up

* Move to purge, which all users can manually call
javerous pushed a commit to javerous/FreshRSS that referenced this pull request Jan 20, 2020
mdemoss pushed a commit to mdemoss/FreshRSS that referenced this pull request Mar 25, 2021
* MySQL GUID case sensitive

latin1_bin
FreshRSS#2077

* Prepare update for existing bases

* Perform DB update during actualize

* Reduce frequency slightly

* No optimize at the same time

* Take advantage of the SQL modifications in 1.12

* Move higher up

* Move to purge, which all users can manually call
mdemoss pushed a commit to mdemoss/FreshRSS that referenced this pull request Mar 25, 2021
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Sep 10, 2023
Should help with some DB lock issues.

Complete FreshRSS#3558 after FreshRSS#5625 already cherry-picked from it.

* Removed auto-update of MySQL GUID case sensitivity
FreshRSS#2078
  * Contributed to a DB lock in FreshRSS#5008

Also removed the following non-problematic auto-updates, simply because they were older than the above ones
* Auto-create custom labels (1.12.0) FreshRSS#2027
* Auto-add JSON column for feeds (1.11.0) FreshRSS#1838
* Auto-create temporary tables (1.7.0) FreshRSS#1470
Alkarex added a commit that referenced this pull request Sep 12, 2023
Should help with some DB lock issues.

Complete #3558 after #5625 already cherry-picked from it.

* Removed auto-update of MySQL GUID case sensitivity
#2078
  * Contributed to a DB lock in #5008

Also removed the following non-problematic auto-updates, simply because they were older than the above ones
* Auto-create custom labels (1.12.0) #2027
* Auto-add JSON column for feeds (1.11.0) #1838
* Auto-create temporary tables (1.7.0) #1470
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.

None yet

1 participant