On Tuesday 12 August 2008 16:50, Florent Daigni?re wrote:
> * Matthew Toseland <toad at amphibian.dyndns.org> [2008-08-12 14:47:30]:
>
> > We need better handling of this sort of option ... at the very least, the
node
> > should accept the change, write it to disk, and then tell the user that
they
> > need to restart their node, with a button to do so. There has been a bug
> > filed for this since very early on... probably all we need is a custom
> > exception, handling for it in the config toadlet and the FCP code, and
some
> > changes in the callbacks...
> >
>
> :@ top-posting
>
> How urgent is it? imho it's only a minor-inconvenience to developpers. We
don't want
> users to change port-numbers, datastore types, ... do we?
Hmmm, I suppose...
>
> > On Sunday 10 August 2008 12:32, j16sdiz at freenetproject.org wrote:
> > > Author: j16sdiz
> > > Date: 2008-08-10 11:32:45 +0000 (Sun, 10 Aug 2008)
> > > New Revision: 21704
> > >
> > > Modified:
> > >
> >
branches/saltedhashstore/freenet/src/freenet/l10n/freenet.l10n.en.properties
> > > branches/saltedhashstore/freenet/src/freenet/node/Node.java
> > > Log:
> > > Allow changing datastore type from ConfigToadlet
> > >
> > > Modified:
> >
branches/saltedhashstore/freenet/src/freenet/l10n/freenet.l10n.en.properties
> > > ===================================================================
> > > ---
> >
branches/saltedhashstore/freenet/src/freenet/l10n/freenet.l10n.en.properties
> > 2008-08-10 11:14:09 UTC (rev 21703)
> > > +++
> >
branches/saltedhashstore/freenet/src/freenet/l10n/freenet.l10n.en.properties
> > 2008-08-10 11:32:45 UTC (rev 21704)
> > > @@ -705,7 +705,7 @@
> > > Node.storeSize=Store size in bytes
> > > Node.storeSizeLong=Store size in bytes
> > > Node.storeType=Store type (LEAVE THIS ALONE)
> > > -Node.storeTypeLong=Datastore type. Currently this can be bdb-index (use
a
> > BerkeleyDBFreenetStore to store the index, and keep the data in files on
> > disk), or ram (keep the index and the data in RAM). Only use ram if you
know
> > what you are doing and have enough RAM to store all your data (and note it
> > will not be saved on shutdown)!
> > > +Node.storeTypeLong=Datastore type. Currently this can be salt-hash (use
a
> > salted on-disk hashtable with bloom filter), bdb-index (use a
> > BerkeleyDBFreenetStore to store the index, and keep the data in files on
> > disk), or ram (keep the index and the data in RAM). Only use ram if you
know
> > what you are doing and have enough RAM to store all your data (and note it
> > will not be saved on shutdown)! Changes will not take effect until Freenet
> > has been restarted.
> > > Node.swapRInterval=Swap request send interval (ms)
> > > Node.swapRIntervalLong=Interval between swap attempting to send swap
> > requests in milliseconds. Leave this alone!
> > > Node.throttleLocalTraffic=Throttle local traffic?
> > >
> > > Modified: branches/saltedhashstore/freenet/src/freenet/node/Node.java
> > > ===================================================================
> > > --- branches/saltedhashstore/freenet/src/freenet/node/Node.java
2008-08-10
> > 11:14:09 UTC (rev 21703)
> > > +++ branches/saltedhashstore/freenet/src/freenet/node/Node.java
2008-08-10
> > 11:32:45 UTC (rev 21704)
> > > @@ -178,7 +178,8 @@
> > > }
> > >
> > > public void set(String val) throws InvalidConfigValueException {
> > > - throw new InvalidConfigValueException("Store type
> > > cannot be changed
on
> > the fly");
> > > + // need restart to effect
> > > + config.get("node").set("storeType", val);
> > > }
> > >
> > > public String[] getPossibleValues() {
> > >
> > > _______________________________________________
> > > cvs mailing list
> > > cvs at freenetproject.org
> > > http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs
> > >
> > >
>
>
>
> > _______________________________________________
> > Devl mailing list
> > Devl at freenetproject.org
> > http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL:
<https://emu.freenetproject.org/pipermail/devl/attachments/20080812/64784707/attachment.pgp>