Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dataset Record 500 when should be 400: add aliasqueryaref #46

Open
aurreco-uga opened this issue Apr 21, 2024 · 3 comments
Open

Dataset Record 500 when should be 400: add aliasqueryaref #46

aurreco-uga opened this issue Apr 21, 2024 · 3 comments
Assignees

Comments

@aurreco-uga
Copy link
Member

aurreco-uga commented Apr 21, 2024

(found URL in error emails)
check teh difference between:
https://microbiomedb.org/mbio/app/record/dataset/DS_1c4b2eeecb
and
https://microbiomedb.org/mbio/app/record/dataset/bbbbbbbb

@ryanrdoherty
Copy link
Member

This has nothing to do with the client. The issue is similar to what we've seen recently with alias queries. In mbio, the alias query for datasets is:

        <sql includeProjects="ClinEpiDB,MicrobiomeDB">
        <![CDATA[
                  select dataset_presenter_id as dataset_id,
                         dataset_presenter_id as old_dataset_id
                  from apidbTuning.DatasetPresenter
                    UNION
          SELECT d.dataset_stable_id, d.dataset_stable_id
          FROM apidbuserdatasets.installeduserdataset i, 
               apidbuserdatasets.datasetattributes d
          WHERE i.user_dataset_id = d.user_dataset_id
        ]]>
        </sql>

This SQL returns a row for DS_1c4b2eeecb, indicating that it is a real ID, but not for bbbbbbbb, so bbbbbbbb throws 404. Then when we try to join DS_1c4b2eeecb to the attribute query, DatasetAttributes.ClinepiStudyAttributes, there's no data so we get 500. Makes sense to me that a table named ClinepiStudyAttributes would not have data for an mbio study.

@aurreco-uga aurreco-uga transferred this issue from VEuPathDB/web-monorepo Apr 21, 2024
@aurreco-uga
Copy link
Member Author

we need the aliasqueryref..

@aurreco-uga aurreco-uga changed the title WDK record not found sometimes throws 500 Dataset Record 500 when should be 400: add aliasqueryaref Apr 21, 2024
@ryanrdoherty
Copy link
Member

@aurreco-uga No, that's not what I was saying. Not sure why mbio datasets are trying to join to DatasetAttributes.ClinepiStudyAttributes, but that is the problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants