Last modified: 2013-07-25 07:04:47 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T43987, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 41987 - Echo doesn't work on SQLite
Echo doesn't work on SQLite
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Echo (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Erik Bernhardson
: schema-change
Depends on:
Blocks: sqlite
  Show dependency treegraph
 
Reported: 2012-11-10 21:15 UTC by Alex Monk
Modified: 2013-07-25 07:04 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Alex Monk 2012-11-10 21:15:53 UTC
Today Hashar enabled Jenkins tests on the Echo repository, and it failed, so has been disabled.
I did some testing and found that this is because Echo does not work with SQLite, which Jenkins relies on to run the tests.

E.g. this error:
https://integration.mediawiki.org/ci/job/Ext-Echo/20/console
20:32:21      [exec] "CREATE INDEX user_timestamp ON echo_notification (notification_user,notification_timestamp)
20:32:21      [exec] "
20:32:21      [exec] from within function "DatabaseBase::sourceFile( /var/lib/jenkins/jobs/Ext-Echo/workspace/extensions/Echo/echo.sql )".
20:32:21      [exec] Database returned error "1: index user_timestamp already exists"

It also looks like all the files in db_patches/ will fail because of no ALTER TABLE x CHANGE COLUMN. https://www.sqlite.org/omitted.html
Comment 1 bsitu 2012-11-13 18:53:12 UTC
Looks like index name in SQLite must be unique across database
Comment 2 Dereckson 2012-11-28 19:05:14 UTC
Adding schema-change keyword.
Comment 3 Dereckson 2012-12-18 00:01:50 UTC
Taking this bug.
Comment 4 Alex Monk 2012-12-23 16:43:44 UTC
Another Jenkins test has been activated which causes issues because of this bug - see Gerrit change #40090.
Comment 5 Dereckson 2012-12-23 18:50:18 UTC
I'm currently fixing Ia2782d64 and I then can prepare the database patch for the Echo index.
Comment 6 Dereckson 2012-12-24 01:47:45 UTC
Gerrit change #40095.

Please open a new bug for the comment #4 if it's a different issue.
Comment 7 Erik Bernhardson 2013-05-24 21:59:51 UTC
Related URL: https://gerrit.wikimedia.org/r/64594 (Gerrit Change
 I9b6468221ba6fe501b15c563f3301694262eec65 )
Comment 8 Erik Bernhardson 2013-06-14 18:37:17 UTC
change merged, tests now run on jenkins

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links