Opened 16 years ago

Closed 16 years ago

#362 closed defect (fixed)

Serialise termfreqs and reltermfreqs together in network protocol

Reported by: Richard Boulton Owned by: Richard Boulton
Priority: normal Milestone: 1.1.1
Component: Backend-Remote Version: SVN trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Operating System: All

Description

When the term statistics are serialised (in serialise_stats() in net/serialise.cc), first the mapping of termfreqs is serialised, followed by the mapping of reltermfreqs. This involves serialising all the terms in the query twice. It would be more efficient, and now match with the structure used to hold these in Weight::Internal, to store each term, followed by the two termfreqs for that term. I've added a FIXME (in r12597) to note that we should do this next time we break the remote protocol.

Change History (3)

comment:1 by Richard Boulton, 16 years ago

Milestone: 1.1.41.1.1

Might as well fix now.

comment:2 by Richard Boulton, 16 years ago

Owner: changed from Olly Betts to Richard Boulton

comment:3 by Richard Boulton, 16 years ago

Resolution: fixed
Status: newclosed

Fixed in r12642.

Note: See TracTickets for help on using tickets.