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

Add a new managers layer #528

Open
imedina opened this issue Feb 11, 2020 · 1 comment
Open

Add a new managers layer #528

imedina opened this issue Feb 11, 2020 · 1 comment
Assignees
Labels
in progress work in progress, under active development
Milestone

Comments

@imedina
Copy link
Member

imedina commented Feb 11, 2020

The current design consists of two layers: REST and DBAdapators. It would be great to add a new manager layer for different reasons.

@imedina imedina added this to the v5.0.0 milestone Feb 11, 2020
@julie-sullivan julie-sullivan added the in progress work in progress, under active development label Feb 12, 2020
@julie-sullivan
Copy link
Contributor

  • DBAdaptor - should only be executing queries. Logic should be in manager
  • REST API - should only handle the requests/responses. Logic should be in the manager.

julie-sullivan added a commit that referenced this issue Feb 12, 2020
julie-sullivan added a commit that referenced this issue Feb 13, 2020
julie-sullivan added a commit that referenced this issue Feb 17, 2020
julie-sullivan added a commit that referenced this issue Feb 18, 2020
julie-sullivan added a commit that referenced this issue Feb 24, 2020
… to queryoptions. add a temp toQueryoptions as that's what java commons lib needs. #528
julie-sullivan added a commit that referenced this issue Feb 25, 2020
…yParams because they may not always be the same. #528
julie-sullivan added a commit that referenced this issue Feb 25, 2020
… not recognised. use facet param to hold the groupby,distinct,aggregationstats fields. for /info loop though each ID and create and run each query -- in the REST class. #528
julie-sullivan added a commit that referenced this issue Mar 5, 2020
julie-sullivan added a commit that referenced this issue Mar 5, 2020
…query for now until we decide what to do with queryoptions. #528
julie-sullivan added a commit that referenced this issue Mar 5, 2020
…pile. TEMPORARILY add custom projection method, use the QueryUtils one in core instead. fix expression query. add cases for all query parameters. #528
julie-sullivan added a commit that referenced this issue Mar 6, 2020
julie-sullivan added a commit that referenced this issue Mar 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in progress work in progress, under active development
Projects
None yet
Development

No branches or pull requests

2 participants