DICE-UNC / jargon

Jargon core libraries
Other
28 stars 31 forks source link

MySQL case error makes Jargon mad with mysql on user group membership #272

Open michael-conway opened 6 years ago

michael-conway commented 6 years ago

iRODS is mad about this Sql query


Nov  6 13:13:06 pid:8088 remote addresses: 192.168.56.1 ERROR: Agent [8088] exiting with status = -4000
Nov  6 13:13:06 pid:5254  ERROR: Agent process [8088] exited with status [96]
Nov  6 13:17:18 pid:8220 NOTICE: bindVar1=testAddUserToGroupAsGroupAdminBug255
Nov  6 13:17:18 pid:8220 NOTICE: cllExecSqlWithResultBV: SQLExecDirect error: -1, sql:select count(user_id) from r_user_group where  group_user_id != user_id and group_user_id in (select user_id from r_user_main where user_name=? and user_type_name='rodsgroup') limit 1
Nov  6 13:17:18 pid:8220 NOTICE: SQLSTATE: S0002
Nov  6 13:17:18 pid:8220 NOTICE: SQLCODE: 1146
Nov  6 13:17:18 pid:8220 NOTICE: SQL Error message: [unixODBC][MySQL][ODBC 5.1 Driver][mysqld-5.5.58-0ubuntu0.14.04.1]Table 'ICAT.r_user_group' doesn't exist

it seems to be case, as this is happy


select
    count( user_id )
from
    R_USER_GROUP
where
    group_user_id != user_id
    and group_user_id in(
        select
            user_id
        from
            R_USER_MAIN
        where
            user_type_name = 'rodsgroup'
    ) limit 1;

NB that is happy because I changed the case of the table names.

michael-conway commented 6 years ago

a reversion of https://github.com/irods/irods/issues/2098 ?

trel commented 6 years ago

irods/irods#3717 now fixed - this should be clear in jargon now

trel commented 6 years ago

and #271 as well?

michael-conway commented 6 years ago

OK will retest

On Mon, Feb 5, 2018 at 12:08 AM, Terrell Russell notifications@github.com wrote:

and #271 https://github.com/DICE-UNC/jargon/issues/271 as well?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/DICE-UNC/jargon/issues/272#issuecomment-362983064, or mute the thread https://github.com/notifications/unsubscribe-auth/ABC-LfN7YYM8T80fRKDoEN0cc7RsvPh0ks5tRozpgaJpZM4QU0ci .