Page MenuHomePhabricator

PostgreSQL Logging table has old column definitions
Closed, ResolvedPublic

Description

Author: matt

Description:
(Using 1.4b5):

When I try to upload a file, I get the following error. The pg table definition
is probably outdated:

Warning: pg_query(): Query failed: ERROR: column "log_params" of relation
"logging" does not exist . in /home/www/zenko/includes/DatabasePostgreSQL.php on
line 99
A database error has occurred Query: INSERT INTO "logging"
(log_type,log_action,log_timestamp,log_user,log_namespace,log_title,log_comment,log_params)
VALUES ('upload','upload','2005-01-23
19:43:43','4','6','Bubblebobble.png','BubBob','') Function: LogPage::saveContent
Error: 1 ERROR: column "log_params" of relation "logging" does not exist

Backtrace:

  • DatabasePostgreSQL.php line 419 calls wfdebugdiebacktrace()
  • Database.php line 297 calls databasepostgresql::reportqueryerror()
  • Database.php line 873 calls databasepostgresql::query()
  • DatabasePostgreSQL.php line 223 calls database::insert()
  • LogPage.php line 64 calls databasepostgresql::insert()
  • LogPage.php line 189 calls logpage::savecontent()
  • Image.php line 790 calls logpage::addentry()
  • SpecialUpload.php line 270 calls wfrecordupload()
  • SpecialUpload.php line 112 calls uploadform::processupload()
  • SpecialUpload.php line 19 calls uploadform::execute()
  • SpecialPage.php line 309 calls wfspecialupload()
  • SpecialPage.php line 220 calls specialpage::execute()
  • index.php line 94 calls specialpage::executepath()

Version: 1.4.x
Severity: major
OS: Linux
Platform: PC

Details

Reference
bz1391
TitleReferenceAuthorSource BranchDest Branch
Draft: Update section-topics pipeline argumentsrepos/data-engineering/airflow-dags!584mlitnT339129_2main
Draft: Periodically invoke a bunch of section-topics scriptsrepos/data-engineering/airflow-dags!583mlitnT339129_1main
Draft: Accept denylist as parquetrepos/structured-data/image-suggestions!39mlitnT339129main
Draft: Consume parquets instead of static filesrepos/structured-data/section-topics!30mlitnT339129_2main
Alter scripts/*.py to write their outputs to parquetsrepos/structured-data/section-topics!29mlitnT339129_1main
HISTORY: prep for 2.0.0 releaserepos/ci-tools/commit-message-validator!11bd808work/bd808/2.0.0main
toolforge-cd: adapt to the decided flowsrepos/cloud/cicd/gitlab-ci!11dcarouse_new_flowsmain
Add fallback values for entityType and riskTyperepos/mediawiki/services/ipoid!13stranadd-property-fallbacksmain
Prepare to release new versions via GitLab CI and taggingrepos/ci-tools/commit-message-validator!2bd808work/bd808/automated-releasesmain
AutosuggestSitelink.js: Remove the animation on Popuprepos/commtech/autosuggest-sitelink!37s-mukutiT339169main
Migrate to GitLabrepos/ci-tools/commit-message-validator!1bd808work/bd808/gitlab-migrationmain
Show related patches Customize query in GitLab

Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.
StatusSubtypeAssignedTask
InvalidNone
ResolvedNone

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 8:09 PM
bzimport set Reference to bz1391.
bzimport added a subscriber: Unknown Object (MLST).

matt wrote:

The fix for this is pretty simple, add this line to the logging table definition:

log_params text DEFAULT ''::text NOT NULL

(sorry, I don't have a CVS tree checked out so I can't give you a patch)

  • This bug has been marked as a duplicate of 1295 ***
Jdforrester-WMF subscribed.

Migrating from the old tracking task to a tag for PostgreSQL-related tasks.