View Issue Details

IDProjectCategoryView StatusLast Update
14918Bug reports[All Projects] Installationpublic2019-05-22 12:10
ReporterDenisChenuAssigned ToDenisChenu 
PrioritynoneSeverityminor 
Status resolvedResolutionfixed 
Product Version4.0.0-beta 
Target VersionFixed in Version4.0.0-beta 
Summary14918: Unable to install with pgsql
Description

Trying to install : broke with unknow options

Steps To Reproduce

Install

Additional Information

I didn't install on pgsql, i use https://github.com/Shnoulle/Test-LS-checkDbForUpgrade
And broke with pgsql : https://github.com/Shnoulle/Test-LS-checkDbForUpgrade/commit/12c05f0631943ea9b10cc7e4ba7a78b0c785e39d#diff-3c8b45db8a2db228701055d312df025dR140

Introcude here : https://github.com/LimeSurvey/LimeSurvey/commit/4f97fb02e96c2526fab1ef6db5e9cc73eebc1943#diff-3aec9285e40974fd598d382116b29ece

I can fix , but i want to understand the reason for adding this (maybe upgrade_db must set this options on old table ?)

TagsNo tags attached.
Complete LimeSurvey version number (& build)4.0.0 github
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database & DB-Versionpgsql 9
Server OS (if known)debian/linux
Webserver software & version (if known)Apache
PHP Versionphp5.6

Activities

TonisOrmisson

TonisOrmisson

2019-05-21 09:37

developer   ~52000

@DenisChenu reason for adding what?

Feel free to fix it. I am quite over-occupied at this point, thanks

DenisChenu

DenisChenu

2019-05-21 10:23

developer   ~52004

Why adding $options = 'ROW_FORMAT=DYNAMIC'; ?

TonisOrmisson

TonisOrmisson

2019-05-21 10:34

developer   ~52005

it is related with the same issue why we need to set the the

innodb_large_prefix = 1
innodb_file_format = Barracuda;
innodb_file_format_max = Barracuda;

see eg
https://www.google.com/search?q=innodb+ROW_FORMAT%3DDYNAMIC+Specified+key+was+too+long

DenisChenu

DenisChenu

2019-05-21 11:57

developer   ~52007

Then need only if config are set to innodb ? Not for isam ?

TonisOrmisson

TonisOrmisson

2019-05-21 12:05

developer   ~52008

probably, yes

DenisChenu

DenisChenu

2019-05-22 12:03

developer   ~52032

@TonisOrmisson I let you review if ROW_FORMAT=DYNAMIC is needed only for innodb

TonisOrmisson

TonisOrmisson

2019-05-22 12:10

developer   ~52034

not able to do it, busy :(

Issue History

Date Modified Username Field Change
2019-05-21 09:12 DenisChenu New Issue
2019-05-21 09:12 DenisChenu Assigned To => TonisOrmisson
2019-05-21 09:12 DenisChenu Status new => assigned
2019-05-21 09:37 TonisOrmisson Note Added: 52000
2019-05-21 09:37 TonisOrmisson Assigned To TonisOrmisson => DenisChenu
2019-05-21 10:23 DenisChenu Note Added: 52004
2019-05-21 10:34 TonisOrmisson Note Added: 52005
2019-05-21 11:57 DenisChenu Note Added: 52007
2019-05-21 12:05 TonisOrmisson Note Added: 52008
2019-05-22 12:03 DenisChenu Status assigned => resolved
2019-05-22 12:03 DenisChenu Resolution open => fixed
2019-05-22 12:03 DenisChenu Fixed in Version => 4.0.0-beta
2019-05-22 12:03 DenisChenu Note Added: 52032
2019-05-22 12:10 TonisOrmisson Note Added: 52034