Ticket #8401 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

[ICAT] Update query language used for searching

Reported by: Jay Rainey Owned by: Jay Rainey
Priority: major Milestone: Release 3.1
Component: Framework Keywords: ICAT, ICAT4
Cc: Blocked By:
Blocking: #8230 Tester: Gesner Passos

Description

In order to implement paging I will need to update the querying language used for searching to use the ICAT 4.3 JPQL syntax.

Change History

comment:1 Changed 7 years ago by Jay Rainey

  • Blocking 8230 added
  • Blocked By 8230 removed

comment:2 Changed 7 years ago by Jay Rainey

  • Status changed from new to inprogress

Updated ICAT4.3 query syntax to JSQL. Refs #8401.

Changeset: f9080f58827c3423d9ea0783b8e9d39c22f5d051

Last edited 7 years ago by Jay Rainey (previous) (diff)

comment:3 Changed 7 years ago by Jay Rainey

Updated debug messages in ICat4Catalog. Refs #8401.

Changeset: 179bc3daf97b4a936394903dac1499c7d3982132

Last edited 7 years ago by Jay Rainey (previous) (diff)

comment:4 Changed 7 years ago by Jay Rainey

Fixed searching by sample name. Refs #8401.

Changeset: f0841cd42bfbdacc1e5fc9143dae0a333421cfa6

Last edited 7 years ago by Jay Rainey (previous) (diff)

comment:5 Changed 7 years ago by Jay Rainey

Allow search by investigator's name in "my data". Refs #8401.

Changeset: c06ec21183b054b28058366dd16a31df5f8383a1

Last edited 7 years ago by Jay Rainey (previous) (diff)

comment:6 Changed 7 years ago by Jay Rainey

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:7 Changed 7 years ago by Jay Rainey

This ticket dealt with converting the old ICAT custom query language to ICAT4.3 JSQL like language.

To test

  1. Log into the catalog and open the Search GUI.
  2. Perform multiple searches with each input field, and several at a time. They should all function as expected.
Last edited 7 years ago by Jay Rainey (previous) (diff)

comment:8 Changed 7 years ago by Gesner Passos

  • Status changed from verify to verifying
  • Tester set to Gesner Passos

comment:9 Changed 7 years ago by Gesner Passos

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8401_icat_update_query_lang'

Full changeset: ​ac231bdbdab49618adfd75cd02e8a697b62371d6

comment:10 Changed 7 years ago by Gesner Passos

I've performed many different queries and it proceeded nicely.

I just found strange that adding terms to your query may slow down your research. For example, I put Instrument to SANS2D and investigation title to Sample. I got a fast answer. But, when I added run range, it took almost 2 minutes to have an answer.

It seems that by joining results you end up with too big data set to select from.

comment:11 Changed 7 years ago by Jay Rainey

Updated ICAT4.3 query syntax to JSQL. Refs #8401.

Changeset: 09aee2526edda16b6ababcde6e444e8c42fec540

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​9246

Note: See TracTickets for help on using tickets.