[OpenSER-Devel] Fwd: [ openser-Bugs-1836601 ] db_berkeley don't work

[Henning Westerholt]

On Monday 26 November 2007, William Quan wrote:
> H,
> good catch on the path on line 53.
>
> The db_load at 392 should not use the LOAD_CMD.
> The db_load is a Berkeley utility that can support two use cases:
> 1. load data from a file into a db
> 2. reset internal db attributes: fileid or log sequence number (lsn)
>
> In most places in openserdbct.db_berkeley we  use-case 1 (e.g LOAD_CMD).
> The only use-case #2 occurs at line 392.
> The 'append' operation that occurs around line 392 takes a copy of the
> db, and it is important to reset the db internal attributes to prevent
> corruption.

Good morning Will,

hmm... Ok, the problem is that on debian the db_load tools is not available, 
its name is e.g. 'db46_load'. So it would be nice if the openserdbctl for 
db_berkeley would work after i set the LOAD_CMD variable. Perhaps you could 
use the "plain" command in the variable, and add the switches later?

> Variable STANDARD_TABLES is defined in openserdbctl.dbtext and
> openserdbctl.db_berkeley to identify the tables that are required.
> Variable STANDARD_MODULES is defined in openserdbctl.base to identify
> the module names (not table names).
>
> Maybe we can move these 3 vars to .base so dbtext and db_berkeley can
> share them?
> STANDARD_TABLES="version acc dbaliases lcr domain grp uri speed_dial gw pdt
>            subscriber location re_grp trusted address missed_calls
>            usr_preferences aliases gw_grp silo dispatcher"
>
> EXTRA_TABLES="imc_members imc_rooms cpl sip_trace domainpolicy"
>
> PRESENCE_TABLES="presentity active_watchers watchers xcap pua
> rls_presentity rls_watchers"
>
> And of course remove them from both dbtext and db_berkeley.
> If you want I can make these changes.

Ah, sure, you're right. Yesterday it was already a little bit to late  for 
work.. ;-) Please do this change, move this defines to the .base file.

Thank you,

Hening