Skip Menu |
 

This queue is for tickets about the Sphinx-Search CPAN distribution.

Report information
The Basics
Id: 86122
Status: open
Priority: 0/
Queue: Sphinx-Search

People
Owner: Nobody in particular
Requestors: grinnz [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



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>
Download (untitled) / with headers
text/plain 237b
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.
Download (untitled) / with headers
text/plain 366b
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
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>
Download (untitled) / with headers
text/plain 1.3k
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 >
Download sphinx_test.pl
text/x-perl 657b

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

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>
Download (untitled) / with headers
text/plain 1.6k
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 >>
> >
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>
Download (untitled) / with headers
text/plain 1.9k
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 >>


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.