Added memory connector to trino

[pdmthorsrud]

Closes/fixes/resolves issue(s)?

Closes #121

What was added/changed/fixed?

Memory connector properties file

Related issue(s)? [Optional]

Others [Optional]

Checklist (after created PR)

• Added reviewers
• Added assignee(s)
• Added label(s)
• Added to project
• Added to milestone

[pdmthorsrud]

Pipeline gets an error, but it does not seem to be related to my changes. Also runs fine locally

edit: created issue #124

[fredrikhgrelland]

Looks good, but some changes required.

There must be links to documentation added, and possibly also a simple example on how/why to use?

[pdmthorsrud]

Looks good, but some changes required.

There must be links to documentation added, and possibly also a simple example on how/why to use?

I was thinking its own section would be too much, considering there is no mention of the other connectors (Trino and PostgreSQL), except in the input section.

If we add doc for the memory connector we should add doc for all connectors under a "Connectors" section. I think that is a good idea, WDYT?

edit: god damn I miss github, this site is so clear and concise

[fredrikhgrelland]

Looks good, but some changes required.
There must be links to documentation added, and possibly also a simple example on how/why to use?

I was thinking its own section would be too much, considering there is no mention of the other connectors (Trino and PostgreSQL), except in the input section.

If we add doc for the memory connector we should add doc for all connectors under a "Connectors" section. I think that is a good idea, WDYT?

edit: god damn I miss github, this site is so clear and concise

Yep - a simple and short connectors section with doc links would do it

[fredrikhgrelland]

@pdmthorsrud Abandoned?

[pdmthorsrud]

@pdmthorsrud Abandoned?

Oh, not abandoned, but most definitely forgotten. I will add the doc section!

[fredrikhgrelland]

@tomdgr linter is throwing errors..