Re: [Sqlrelay-discussion] Zope adapter limit of 1001 records?

[Cameron L. <cla...@gm...>]

Many thanks, again, for your help with this.  You can imagine what a help it
is to have correct results, rather than ones truncated after 1001 records.

Can you effectively use my help updating the ZSQLRelayDA adapter?  I
certainly owe you.

We have identified a few errors that we can work around for now.  One,
however, is both puzzling and apparently crucial to work I'm preparing.  Let
me know if you'd like me to report this in the mailing list or elsewhere:

Briefly,
    sudo -u test-user sqlrsh $HOST 9005 "" $ACCOUNT $PASSWORD $SCRIPT
segfaults.  "test-user" is an unremarkable account, with pedestrian
privileges.  I can
    su test-user
    sqlrsh $HOST 9005 "" $ACCOUNT $PASSWORD $SCRIPT
and everything goes fine.

Have you come across this before?  What do you recommend?  Shall I generate
sqlrsh with debugging symbols, so I can get a readable backtrace?


On Tue, Jul 7, 2009 at 10:57 PM, David Muse <dav...@fi...>wrote:

> Hey Cameron,
>
> I figured out the 1001 rows issue.  By default, a ZSQLMethod has a limit
> of 1000 rows and passes this limit into the ZSQLRelayDA adapter when it
> runs the query.  To change this, click the Advanced tab of the
> ZSQLMethod and set the "Maximum rows to retrieve" setting to a larger
> number.
>
> If you have a gazillion ZSQLMethods, then you can probably modify
> lib/python/Shared/DC/ZRDB/DA.py and change the line:
>
> max_rows_=1000
>
> to something else.
>
> Arguably the ZSQLRelayDA adapter does have a bug that's causing it to
> retrieve 1001 rather than 1000 rows.
>
>
> Aside from that though, the adapter doesn't work with Zope 2.11 or
> higher because it uses several deprecated objects and attributes.  So,
> that does need to be fixed, ideally in a back-compatible way.
>    ...