Closed (fixed)
Project:
Drupal core
Version:
5.0-beta1
Component:
database system
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
1 Nov 2006 at 20:22 UTC
Updated:
24 Nov 2006 at 07:31 UTC
Jump to comment: Most recent file
Comments
Comment #1
doq commentednow these errors appear on every page. Just installed, haven't changed anything. Just themes AFAIR.
Comment #2
simeflagging as db issue - sammys?
Comment #3
Jo Wouters commentedI'm no expert, and I don't have a Postgres installation to test this, but I think the problem is in /includes/cache.inc
fills the bytea-field 'data' with a string.
I think we should add
$data = db_encode_blob($data);before we do the update.In cache_get we do a
$cache->data = db_decode_blob($cache->data);The db_encode_blob might have gone lost when cache.inc was moved out of bootstrap.inc (for a while %b was used)
Attached you can find the patch that add this line
(CAUTION: I did not test this)
Comment #4
Jo Wouters commentedComment #5
doq commentedso why just not using %b for 'data'?
Comment #6
havran commentedSame problem here http://drupal.org/node/92503
Comment #7
Jo Wouters commented@doq: you are right: %b is better.
I addded a patch but as I understand from http://drupal.org/node/92503 , this might not be a complete solution ? Is this issue a duplicate from 92503 ?
(caution: Since I don't have Postgres, I couldn't test the patch)
Comment #8
doq commentedPatch is good.
Comment #9
doq commentedPatch is good.
Comment #10
havran commentedI agree patch is good. I have tried patch second time - patch is god and working. But now is here new problem - with negative value which is return from function timer_read('page') in function statistics_init() (in statistics.module).
Comment #11
doq commentedhmmm. Have you tried with/without patch?
I can't imaging that it can break smth in timer_read() :)
Comment #12
havran commentedWithout patch caching don't work but there is no error like this 'read_timer' - with patch caching working perfect, but there is now problem with statistics.module. If i disable this module all seems working ok.
Comment #13
Jo Wouters commented(havran has created a new issue for the timer_read problem: http://drupal.org/node/93491 )
Comment #14
sammys commentedPatch cache_set_pgsql_error_1.patch.txt works ok on PostgreSQL. RTBC!
Comment #15
drummCommitted to HEAD.
Comment #16
(not verified) commented