[geeklog-devtalk] geeklog-devel digest, Vol 1 #294 - 2 msgs

[geeklog-devel-request at lists.geeklog.net]

Send geeklog-devel mailing list submissions to
	geeklog-devel at lists.geeklog.net

To subscribe or unsubscribe via the World Wide Web, visit
	http://lists.geeklog.net/listinfo/geeklog-devel
or, via email, send a message with subject or body 'help' to
	geeklog-devel-request at lists.geeklog.net

You can reach the person managing the list at
	geeklog-devel-admin at lists.geeklog.net

When replying, please edit your Subject line so it is more specific
than "Re: Contents of geeklog-devel digest..."


Today's Topics:

   1. Re: [Fwd: Re: Geeklog optimalisations] (Dirk Haun)
   2. Re: [Fwd: Re: Geeklog optimalisations] (Tony Bibbs)

--__--__--

Message: 1
From: "Dirk Haun" <dirk at haun-online.de>
To: <geeklog-devel at lists.geeklog.net>
Subject: Re: [geeklog-devel] [Fwd: Re: Geeklog optimalisations]
Date: Sun, 21 Mar 2004 22:30:39 +0100
Organization: Terra Software Systems
Reply-To: geeklog-devel at lists.geeklog.net

Niels Leenheer wrote:


>> Next is the database layer. The mysql_fetch_array() function has two

>> arguments. The second determines what the function returns. Either an

>> associative array, a numbered array or both. By default the function

returns

>> both. This is what Geeklog does. In most of the code only the associative

>> array is used. Only in a couple of small instances the code requires an

>> numbered array. What we have done is to instruct the mysql_fetch_array()

>> function to return only an associative array by default. Only when the code

>> requires a numbered array we request both. This should lower the amount of

>> memory needed by Geeklog.


This is now implemented in CVS. Per default, DB_fetchArray will only
return the associative array indices. If you need the numeric ones as
well, you'll have to pass a new (optional) second parameter:
DB_fetchArray ($result, true);

Thanks to Niels for helping me find all the places where the second
parameter is needed.

I've also fixed the first batch of bugs that have come up (from docs/
history - Tony, any news on CVS notifications?)

- Removed extra quotes in SQL statements in admin/block.php to ensure
  compatibility with old MySQL versions.
- Fixed SQL error in COM_showTopics when users tried to exclude topics in
their
  preferences.
- In admin/database.php, if the is_executable function is not available (e.g.
  on Windows), do at least a file_exists to check if the mysqldump executable
  exists in the path given in config.php.

bye, Dirk


-- 
http://www.haun-online.de/
http://geeklog.info/


--__--__--

Message: 2
Date: Mon, 22 Mar 2004 08:45:15 -0600
From: Tony Bibbs <tony at tonybibbs.com>
To: geeklog-devel at lists.geeklog.net
Subject: Re: [geeklog-devel] [Fwd: Re: Geeklog optimalisations]
Reply-To: geeklog-devel at lists.geeklog.net

News on CVS notification is this.  I'm only a week or so away from 
moving the site back to the primary server (which now has 1GB of RAM and 
two new 18.1GB drives to add to the 3 existing ones which will be 
RAID5'd).  Instead of fixing the problem on the current server (which 
has been futile) I'm going to wait until I cut back over to the primary 
server (at the new ISP) and go from there.

--Tony

Dirk Haun wrote:

> Niels Leenheer wrote:

> 

> 

>>>Next is the database layer. The mysql_fetch_array() function has two

>>>arguments. The second determines what the function returns. Either an

>>>associative array, a numbered array or both. By default the function

> 

> returns

> 

>>>both. This is what Geeklog does. In most of the code only the associative

>>>array is used. Only in a couple of small instances the code requires an

>>>numbered array. What we have done is to instruct the mysql_fetch_array()

>>>function to return only an associative array by default. Only when the code

>>>requires a numbered array we request both. This should lower the amount of

>>>memory needed by Geeklog.

> 

> 

> This is now implemented in CVS. Per default, DB_fetchArray will only

> return the associative array indices. If you need the numeric ones as

> well, you'll have to pass a new (optional) second parameter:

> DB_fetchArray ($result, true);

> 

> Thanks to Niels for helping me find all the places where the second

> parameter is needed.

> 

> I've also fixed the first batch of bugs that have come up (from docs/

> history - Tony, any news on CVS notifications?)

> 

> - Removed extra quotes in SQL statements in admin/block.php to ensure

>   compatibility with old MySQL versions.

> - Fixed SQL error in COM_showTopics when users tried to exclude topics in

> their

>   preferences.

> - In admin/database.php, if the is_executable function is not available (e.g.

>   on Windows), do at least a file_exists to check if the mysqldump executable

>   exists in the path given in config.php.

> 

> bye, Dirk

> 

> 



--__--__--

_______________________________________________
geeklog-devel mailing list
geeklog-devel at lists.geeklog.net
http://lists.geeklog.net/listinfo/geeklog-devel


End of geeklog-devel Digest