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

Create REQUEST DataModel in CubeJS Schema #29862

Closed
Tracked by #29481 ...
freddyDOTCMS opened this issue Sep 4, 2024 · 5 comments · Fixed by #30039
Closed
Tracked by #29481 ...

Create REQUEST DataModel in CubeJS Schema #29862

freddyDOTCMS opened this issue Sep 4, 2024 · 5 comments · Fixed by #30039

Comments

@freddyDOTCMS
Copy link
Contributor

freddyDOTCMS commented Sep 4, 2024

Parent Issue

User Story

Currently, we have a DataModel that is entirely linked to the Experiment Goal, but for our initial CA version, we will not be supporting the Goal. Therefore, we can leave the existing DataModel as is and create a new one. In this new DataModel, we can group data by request using the new request_id attribute that we need to create and it going to be crated in another issue.

this new DataModel can have a syntax like this one here

Acceptance Criteria

We need to be allow to query using this new DataModel using the new attributes describe in this document .

You can test using the query describe here, but don't limit for just these you can use and document other ones.

Proposed Objective

Core Features

Proposed Priority

Priority 2 - Important

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

@freddyDOTCMS
Copy link
Contributor Author

freddyDOTCMS commented Sep 17, 2024

You need to run this script to make de ClickHouse Database enabled to this changes

ALTER TABLE events ADD COLUMN object_id String;
ALTER TABLE events ADD COLUMN object_title String;
ALTER TABLE events ADD COLUMN object_content_type_id String;
ALTER TABLE events ADD COLUMN object_content_type_name String;
ALTER TABLE events ADD COLUMN object_content_type_var_name String;
ALTER TABLE events ADD COLUMN object_response String;
ALTER TABLE events ADD COLUMN object_forward_to String;
ALTER TABLE events ADD COLUMN object_detail_page_url String;
ALTER TABLE events ADD COLUMN object_url String;
ALTER TABLE events ADD COLUMN comefromvanityurl bool;
ALTER TABLE events ADD COLUMN request_id String;
ALTER TABLE events ADD COLUMN host String;
ALTER TABLE events ADD COLUMN sessionid String;
ALTER TABLE events ADD COLUMN sessionnew bool;

This Script is going to be run automatically after this issue:

To connect with the CH Database you can use the follow parameters:

image (5)

@freddyDOTCMS
Copy link
Contributor Author

PR: #30039

@freddyDOTCMS freddyDOTCMS removed their assignment Sep 18, 2024
@freddyDOTCMS
Copy link
Contributor Author

Note to QA: it should be test with #29712

@freddyDOTCMS
Copy link
Contributor Author

freddyDOTCMS commented Sep 27, 2024

Pass Internal QA: We just have problems retrieving the detail page data, but we already work it on other issues:

@dsilvam dsilvam closed this as completed Oct 2, 2024
@josemejias11
Copy link
Contributor

Approved: Tested on trunk_0028f3c, Docker, macOS 14.5, FF v126.0.1

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

Successfully merging a pull request may close this issue.

4 participants