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

RAMI-67: Add database build extension to load and evaluate Puse library SQL folder. #126

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ijoel
Copy link

@ijoel ijoel commented Nov 7, 2023

Purpose

Ticket RAMI-67 was created to explore a Triangulum extension that would add database build support to load SQL from a folder on the classpath. Functionality that is required to support loading the required database changes for the common change detection facilities of the Pulse library.

The change adds the --pulse-data long cli flag. When specified as part of a database build, Triangulum will search the classpath for a pulse SQL folder, then load and evaluate its contents.

This is, currently, a work in progress. Supporting changes to the top-sort's "namespace" resolution is needed.

Related Issues

Closes RAMI-67

@ijoel ijoel self-assigned this Nov 7, 2023
@ijoel ijoel changed the title Add build-db feature to load sql from a folder on the classpath RAMI-67: Add database build extension to load and evaluate Puse library SQL folder. Nov 7, 2023
@rjsheperd
Copy link
Member

What do you thinking of making this less specific to pulse and instead allow a CLI argument that allows you to pass in the library to search for? (e.g. --load-from-classpath <regexp>)

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

Successfully merging this pull request may close these issues.

2 participants