Bug #86122 for Sphinx-Search: UpdateAttributes() fails to update negative bigint attributes

Thu Jun 13 17:24:48 2013 grinnz [...] gmail.com - Ticket created

Subject:	UpdateAttributes() fails to update negative bigint attributes
Date:	Thu, 13 Jun 2013 17:24:34 -0400
To:	bug-Sphinx-Search [...] rt.cpan.org
From:	Dan Book <grinnz [...] gmail.com>

Bigint attributes should be now possible to update (as well as floats maybe?), however when sending a negative or non-integer value, the function dies with the error "attribute must be numeric" and thus does not even attempt the update.

Sun Jun 30 06:52:32 2013 JJSCHUTZ [...] cpan.org - Correspondence added

On Thu Jun 13 17:24:48 2013, Grinnz wrote: Show quoted text

> Bigint attributes should be now possible to update (as well as floats > maybe?), however when sending a negative or non-integer value, the function > dies with the error "attribute must be numeric" and thus does not even > attempt the update.

I can't reproduce this. Can you send a test case? Thanks, -- Jon Schutz

Sun Jun 30 06:52:32 2013 The RT System itself - Status changed from 'new' to 'open'

Sun Jun 30 11:56:35 2013 grinnz [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #86122] UpdateAttributes() fails to update negative bigint attributes
Date:	Sun, 30 Jun 2013 11:56:18 -0400
To:	bug-Sphinx-Search [...] rt.cpan.org
From:	Dan Book <grinnz [...] gmail.com>

Sure. Using Sphinx 2.0.7 and Sphinx::Search 0.28, and the following source/index config: source src1 { type = mysql sql_host = localhost sql_user = sql_pass = sql_db = test sql_port = 3306 # optional, default is 3306 sql_query = \ SELECT id, text, int_value, bigint_value, float_value FROM test_table sql_attr_uint = int_value sql_attr_bigint = bigint_value sql_attr_float = float_value sql_query_info = SELECT * FROM test_table WHERE id=$id } index test1 { source = src1 path = /var/lib/sphinx/test1 docinfo = extern charset_type = sbcs } And the test script is attached. If either of the $updated_bigint or $updated_float lines are left uncommented the script will die. On Sun, Jun 30, 2013 at 6:52 AM, Jon Schutz via RT < bug-Sphinx-Search@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=86122 > > > On Thu Jun 13 17:24:48 2013, Grinnz wrote:

> > Bigint attributes should be now possible to update (as well as floats > > maybe?), however when sending a negative or non-integer value, the

> function

> > dies with the error "attribute must be numeric" and thus does not even > > attempt the update.

> > > I can't reproduce this. Can you send a test case? > > Thanks, > > -- > > Jon Schutz >

Message body is not shown because sender requested not to inline it.

Sun Jun 30 12:10:20 2013 grinnz [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #86122] UpdateAttributes() fails to update negative bigint attributes
Date:	Sun, 30 Jun 2013 12:10:02 -0400
To:	bug-Sphinx-Search [...] rt.cpan.org
From:	Dan Book <grinnz [...] gmail.com>

By the way, I have "resolved" the issue by using SphinxQL for the updates instead of the API. Just seems like it should be consistent. Thanks for your work on the module. On Sun, Jun 30, 2013 at 11:56 AM, Dan Book <grinnz@gmail.com> wrote: Show quoted text

> > Sure. Using Sphinx 2.0.7 and Sphinx::Search 0.28, and the following > source/index config: > > source src1 > { > type = mysql > > sql_host = localhost > sql_user = > sql_pass = > sql_db = test > sql_port = 3306 # optional, default is 3306 > > sql_query = \ > SELECT id, text, int_value, bigint_value, float_value FROM > test_table > > sql_attr_uint = int_value > sql_attr_bigint = bigint_value > sql_attr_float = float_value > > sql_query_info = SELECT * FROM test_table WHERE id=$id > } > > index test1 > { > source = src1 > path = /var/lib/sphinx/test1 > docinfo = extern > charset_type = sbcs > } > > And the test script is attached. If either of the $updated_bigint or > $updated_float lines are left uncommented the script will die. > > On Sun, Jun 30, 2013 at 6:52 AM, Jon Schutz via RT < > bug-Sphinx-Search@rt.cpan.org> wrote: >

>> <URL: https://rt.cpan.org/Ticket/Display.html?id=86122 > >> >> On Thu Jun 13 17:24:48 2013, Grinnz wrote:

>> > Bigint attributes should be now possible to update (as well as floats >> > maybe?), however when sending a negative or non-integer value, the

>> function

>> > dies with the error "attribute must be numeric" and thus does not even >> > attempt the update.

>> >> >> I can't reproduce this. Can you send a test case? >> >> Thanks, >> >> -- >> >> Jon Schutz >>

> >

Mon Jul 01 07:17:55 2013 jon [...] jschutz.net - Correspondence added

Subject:	Re: [rt.cpan.org #86122] UpdateAttributes() fails to update negative bigint attributes
Date:	Mon, 01 Jul 2013 20:47:26 +0930
To:	bug-Sphinx-Search [...] rt.cpan.org
From:	Jon Schutz <jon [...] jschutz.net>

Thanks for the test case. This appears to be a limitation of the API; UpdateAttributes can only send 32-bit unsigned integers. There are other references to this issue, e.g. http://sphinxsearch.com/forum/view.html?id=11108, http://sphinxsearch.com/forum/view.html?id=3103. So I'll have to let this sit until the API is updated. -- Jon Schutz On 01/07/13 01:26, Dan Book via RT wrote: Show quoted text

> Queue: Sphinx-Search > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=86122 > > > Sure. Using Sphinx 2.0.7 and Sphinx::Search 0.28, and the following > source/index config: > > source src1 > { > type = mysql > > sql_host = localhost > sql_user = > sql_pass = > sql_db = test > sql_port = 3306 # optional, default is 3306 > > sql_query = \ > SELECT id, text, int_value, bigint_value, float_value FROM > test_table > > sql_attr_uint = int_value > sql_attr_bigint = bigint_value > sql_attr_float = float_value > > sql_query_info = SELECT * FROM test_table WHERE id=$id > } > > index test1 > { > source = src1 > path = /var/lib/sphinx/test1 > docinfo = extern > charset_type = sbcs > } > > And the test script is attached. If either of the $updated_bigint or > $updated_float lines are left uncommented the script will die. > > On Sun, Jun 30, 2013 at 6:52 AM, Jon Schutz via RT < > bug-Sphinx-Search@rt.cpan.org> wrote: >

>> <URL: https://rt.cpan.org/Ticket/Display.html?id=86122 > >> >> On Thu Jun 13 17:24:48 2013, Grinnz wrote:

>>> Bigint attributes should be now possible to update (as well as floats >>> maybe?), however when sending a negative or non-integer value, the

>> function

>>> dies with the error "attribute must be numeric" and thus does not even >>> attempt the update.

>> >> I can't reproduce this. Can you send a test case? >> >> Thanks, >> >> -- >> >> Jon Schutz >>