Last modified: 2014-02-14 12:57:37 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 T58591, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 56591 - het-deploy relies on dba_* functions existing
het-deploy relies on dba_* functions existing
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
wmf-deployment
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: hhvm
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-04 22:39 UTC by Chad H.
Modified: 2014-02-14 12:57 UTC (History)
1 user (show)

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


Attachments

Description Chad H. 2013-11-04 22:39:28 UTC
Right now het deploy relies on dba_* functions existing so we can read wikiversions.cdb. This shouldn't be completely necessary since we've got CdbReader_PHP in core.

This makes it impossible to run maintenance scripts under hhvm currently. While hhvm should support the functionality, it would be nice to use our own fallbacks here.
Comment 1 Sam Reed (reedy) 2013-11-04 22:48:36 UTC
The code as is sorta suggests (or reads like, at least) we shouldn't be using the php version...

public static function open( $fileName ) {
		if ( self::haveExtension() ) {
			return new CdbReader_DBA( $fileName );
		} else {
			wfDebug( "Warning: no dba extension found, using emulation.\n" );
			return new CdbReader_PHP( $fileName );
		}
	}


Cdb_PHP.php looks standalone enough, so we can presumably reuse that (without having to initialize mediawiki to read a cdb, which would lead to some nice recursion - what version of mediawiki should we load?)...

Is it worth us copying this into multiversion? Or just use require/include ../php/includes/Cdb.php and ../php/includes/Cdb_PHP.php
Comment 2 Gerrit Notification Bot 2013-11-04 23:54:20 UTC
Change 93622 had a related patch set uploaded by Chad:
Fix up multiversion to not require dba_* functions

https://gerrit.wikimedia.org/r/93622
Comment 3 Gerrit Notification Bot 2013-12-04 23:04:20 UTC
Change 93622 merged by jenkins-bot:
Fix up multiversion to not require dba_* functions

https://gerrit.wikimedia.org/r/93622
Comment 4 Andre Klapper 2014-02-14 12:57:37 UTC
Assuming this bug is FIXED as per comment 3. If that is not the case: Please reopen and elaborate what is left to do here to get this report fixed.

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


Navigation
Links