Add support to programmatically provide TableField values to RecordStreams
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 35/100
Research direction
Start with the model XML handling of queryRef and the RecordStream implementations that currently read table rows from ResultSet. Trace how paged, sorted ID SQL and table columns reach those streams, then determine the plugin entry point described in the issue. Done means Java-provided TableField values work alongside SQL-backed values for the same record and paging behavior.
Written by the indexing model from the issue text.
Description
This is to support use cases where table data comes from an outside source (files or services) or needs to be generated programmatically (vs pulled in directly from the configured appDb with SQL).
Design plan is as follows:
- In the model XML, add option to replace the queryRef attribute with a javaClass value which directs WDK to instantiate and call a Java plugin class to provide table data instead of an SQL query.
- This plugin will take paged, sorted ID SQL and provide a Stream/Iterator of String[] containing the PK columns for the recordclass and the columns of the table (same columns table SQL provides).
- RecordStream implementations must check to see what kind of table field provider this is (SQL vs Java) and branch there, providing rows either from the ResultSet or from the plugin (probably will create a plugin impl that takes SQL and call that).
- Dominant language
- Java
- Stars
- 1
- Forks
- 4
- Avg merge
- 1m
- Merged PRs (30d)
- 2
Getting set up
- Ships a Dockerfile or Docker Compose file
- No pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from VEuPathDB/WDK
-
Difficulty 3/5 1-2 days Newbie friendliness 62/100
-
Difficulty 4/5 3-5 days Newbie friendliness 38/100
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
Feature request: Add redirectUrl query parameter support to `/service/logout` endpointPossibly taken A pull request linked to this issue is open or already merged. Openenhancement
Difficulty 3/5 1-2 days Newbie friendliness 42/100
-
Refine login API to get rid of cookies and automatic guestsPossibly taken @ryanrdoherty claimed this 685 days ago. Open
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
openhab/openhab-addons#21921 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
jellyfin/jellyfin-androidtv#5885 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
objectionary/eo-graphs#85 ·
-
XtYaml throws a bare ClassCastException when sheets or asserts is a scalar or holds a non-stringOpen
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
objectionary/xax#246 ·
Maintainers usually reply within 1 day