[Groonga-commit] [Errored] groonga/groonga#2867 (default-weight-1 - 382ff07)

Back to archive index

Travis CI notif****@travi*****
Wed Mar 5 00:02:01 JST 2014


Build Update for groonga/groonga
-------------------------------------

Build: #2867
Status: Errored

Duration: 6 minutes and 11 seconds
Commit: 382ff07 (default-weight-1)
Author: Kouhei Sutou
Message: Change the default internal weight to 1 from 0

If the default internal weight is 0, "weight 10" produces "score
11". It is strange.

The following chunks in ii.c may be wrong:

    @@ -5705,7 +5705,7 @@ grn_ii_similar_search(grn_ctx *ctx, grn_ii *ii,
             while (grn_ii_cursor_next(ctx, c)) {
               pos = c->post;
               if ((w2 = get_weight(ctx, s, pos->rid, pos->sid, wvm, optarg))) {
    -            res_add(ctx, s, (grn_rset_posinfo *) pos, *w1 * w2 * (pos->tf + pos->weight), op);
    +            res_add(ctx, s, (grn_rset_posinfo *) pos, *w1 * w2 * pos->tf * pos->weight, op);
               }
             }
           }

    @@ -5926,6 +5926,7 @@ grn_ii_select(grn_ctx *ctx, grn_ii *ii, const char *string, unsigned int string_
                   if (max - min <= max_interval) {
                     if (rep) { pi.pos = min; res_add(ctx, s, &pi, weight, op); }
                     noccur++;
    +                tscore += ti->p->weight; /* FIXME: Is it right? */
                     if (ti->pos == max + 1) {
                       break;
                     }

    @@ -5950,14 +5951,14 @@ grn_ii_select(grn_ctx *ctx, grn_ii *ii, const char *string, unsigned int string_
    ...
                 }
               }
             }
    -        if (noccur && !rep) { res_add(ctx, s, &pi, (noccur + tscore) * weight, op); }
    +        if (noccur && !rep) { res_add(ctx, s, &pi, (noccur * tscore) * weight, op); }

View the changeset: https://github.com/groonga/groonga/compare/f18445d1db27...382ff0751a06

View the full build log and details: https://travis-ci.org/groonga/groonga/builds/20058982

--

You can configure recipients for build notifications in your .travis.yml file. See http://about.travis-ci.org/docs/user/build-configuration


-------------- next part --------------
An HTML attachment was scrubbed...
Download 



More information about the Groonga-commit mailing list
Back to archive index