Skip to content

[Analyze 11] Add pa and cdm config metadata to hana session variables#2840

Open
brandantck wants to merge 3 commits into
developfrom
brandantck/analyze-11_add_pa_and_cdm_config_to_hana_session_variables
Open

[Analyze 11] Add pa and cdm config metadata to hana session variables#2840
brandantck wants to merge 3 commits into
developfrom
brandantck/analyze-11_add_pa_and_cdm_config_to_hana_session_variables

Conversation

@brandantck

Copy link
Copy Markdown
Collaborator
  1. Add api call to get dataset pa and cdm config, and set as hana session variables via credentials.

PA and CDM config metadata now appears in hana session context.
image

Merge Checklist

Please cross check this list if additions / modifications needs to be done on top of your core changes and tick them off. Reviewer can as well glance through and help the developer if something is missed out.

  • Automated Tests (Jasmine integration tests, Unit tests, and/or Performance tests)
  • Updated Manual tests / Demo Config
  • Documentation (Application guide, Admin guide, Markdown, Readme and/or Wiki)
  • Verified that local development environment is working with latest changes (integrated with latest develop branch)
  • following best practices in code review doc

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds dataset-level PA/CDM configuration metadata retrieval and propagates it into SAP HANA session variables, so downstream analytics execution can access these values via SESSION_CONTEXT.

Changes:

  • Extend IMRIRequest with optional PA/CDM config ID/version fields and introduce a typed PABackendConfigResponse.
  • Fetch PA backend config metadata from Portal Server during credential middleware execution and attach it to the request.
  • Inject PA/CDM config metadata into HANA connection session variables when establishing DB connections.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
plugins/functions/analytics-svc/src/types.ts Adds request fields for PA/CDM metadata and defines the PA backend config response type.
plugins/functions/analytics-svc/src/middleware/StudyDbCredential.ts Fetches PA/CDM config metadata from Portal Server and attaches it to the request for later use.
plugins/functions/analytics-svc/src/main.ts Passes PA/CDM metadata into DB connection setup and sets corresponding HANA SESSIONVARIABLE:* values.
plugins/functions/analytics-svc/src/api/PortalServerAPI.ts Adds the Portal API call for PA backend config and refactors some request builders.

Comment thread plugins/functions/analytics-svc/src/middleware/StudyDbCredential.ts
Comment thread plugins/functions/analytics-svc/src/api/PortalServerAPI.ts
Comment thread plugins/functions/analytics-svc/src/main.ts
Comment thread plugins/functions/analytics-svc/src/api/PortalServerAPI.ts
@brandantck brandantck enabled auto-merge July 3, 2026 06:04
@brandantck brandantck added this pull request to the merge queue Jul 3, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 3, 2026
@brandantck brandantck added this pull request to the merge queue Jul 5, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 5, 2026
@brandantck brandantck added this pull request to the merge queue Jul 6, 2026
@brandantck brandantck removed this pull request from the merge queue due to a manual request Jul 6, 2026
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