Skip to content

Sail#320

Merged
dianat merged 4 commits into
masterfrom
sail
Mar 25, 2024
Merged

Sail#320
dianat merged 4 commits into
masterfrom
sail

Conversation

@gspinaci
Copy link
Copy Markdown
Collaborator

Why

The SQLSail business core was outdated and integrated only in the branch build-docker/archipelago. With the SQLSail integrated in the master, we can merge all the modifications Villa i Tatti did on semantic-map.

What

In MpRepositoryVocabulary.java, I added the required namespaces. AbstractServiceWrappingSailConfig.java and AbstractServiceWrappingSailConnection.java have been generalised. The latter with another method to override to handle other type of results, not only streams.

I created the classes SQLSail.java, SQLSailConfig.java, SQLSailFactory.java, and SQLSailConnection.java where the business logic for SQLSail is contained.

Video / Gif / Screenshot

No changes in the UI

Meta

The code was required to add another abstract method in AbstractServiceWrappingSailConnection.java to have other classes (RestSail, and SQLSail) extend it without worrying for different types.

How To Test

@aindlq we should define a common way to create Unit Test for SQL connections.

Signed-off-by: gspinaci <gianmarco.spinaci@gmail.com>
Signed-off-by: gspinaci <gianmarco.spinaci@gmail.com>
Signed-off-by: gspinaci <gianmarco.spinaci@gmail.com>
@gspinaci
Copy link
Copy Markdown
Collaborator Author

@giancrys84 I have the help page ready. Where should I add it?

@giancrys84
Copy link
Copy Markdown
Member

Hi @gspinaci, for the time being add the help page here https://drive.google.com/drive/folders/1gNF44hd8k4l9YIZT-CmxNc0qR6xxtvdr
Thanks!

@gspinaci
Copy link
Copy Markdown
Collaborator Author

@dianat dianat self-requested a review March 13, 2024 15:47
Signed-off-by: Diana Tanase <diana@kartography.org>
@dianat
Copy link
Copy Markdown
Collaborator

dianat commented Mar 14, 2024

Hi,

I followed your documentation (thank you), but I kept getting a null error. I've traced it back to the way the output parameter was set, I think I've corrected the problem, but please have a look as well @gspinaci

Hi all, the documentation is online: https://docs.google.com/document/d/1lz6vcVqVXbYsJLewqhooCYN23HBlvl090PwKtqIPqDQ/edit?usp=sharing

@gspinaci
Copy link
Copy Markdown
Collaborator Author

Hi @dianat! Thanks for testing! I am currently working on providing better documentation and a test/default app.

@gspinaci
Copy link
Copy Markdown
Collaborator Author

Hi @dianat, Thanks for fixing the bug. It works perfectly fine with this fix. Moreover, I updated the documentation and created a custom app (https://github.com/villaitatti/RS_SQLSAIL_test) for testing purposes.

Also, in the documentation folder, you can find the dump of the artworks table in CSV format.

@dianat
Copy link
Copy Markdown
Collaborator

dianat commented Mar 25, 2024

Documentation:
https://docs.google.com/document/d/1lz6vcVqVXbYsJLewqhooCYN23HBlvl090PwKtqIPqDQ/edit?usp=sharing

Example custom app (https://github.com/villaitatti/RS_SQLSAIL_test) for testing purposes.

Also, in the documentation folder, you can find the dump of the artworks table in CSV format.

@dianat dianat merged commit dce1dc4 into master Mar 25, 2024
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

Successfully merging this pull request may close these issues.

3 participants