Bug and Fix for DistributedSearch$Client

[Heiko Dietze]

Hi,

during my tests with the nutch-0.8-dev
(daily checkout from svn repository)
I encountered a bug in the constructor
for the class DistributedSearch$Client.
Can someone please correct the order of
the statements and put it in the svn:

old version:

public Client(InetSocketAddress[] addresses, Configuration conf) throws 
IOException {
      this.defaultAddresses = addresses;
      updateSegments();
      setDaemon(true);
      start();
      this.conf = conf;
    }

fixed version:

public Client(InetSocketAddress[] addresses, Configuration conf) throws 
IOException {
      this.defaultAddresses = addresses;
      this.conf = conf;
      updateSegments();
      setDaemon(true);
      start();
    }

During the updateSegments() the this.conf is required or an
NullPointerException is thrown.

Thanks for your help in advance.

with best regards,

Heiko Dietze