Skip to content

V2 Refactoring: MessageViews + MessageBuilders (fixes #111) - #112

Merged
kratz00 merged 41 commits into
mainfrom
feature/refactoring_ctrl_msg_view
Nov 4, 2026
Merged

V2 Refactoring: MessageViews + MessageBuilders (fixes #111)#112
kratz00 merged 41 commits into
mainfrom
feature/refactoring_ctrl_msg_view

Conversation

@kratz00

@kratz00 kratz00 commented Oct 29, 2026

Copy link
Copy Markdown
Collaborator

Fixes #111

@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval October 29, 2026 08:07 — with GitHub Actions Waiting
@kratz00
kratz00 force-pushed the feature/refactoring_ctrl_msg_view branch from b8a9aff to 7acdb8d Compare October 29, 2026 14:02
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval October 29, 2026 14:02 — with GitHub Actions Waiting
@kratz00 kratz00 changed the title V2 Refactoring: CallMessageView + ControlMessageView (fixes #111) V2 Refactoring: MessageViews + MessageBuilders (fixes #111) Oct 29, 2026
@kratz00
V2 Refactoring (fixes #111)
60462fd 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 force-pushed the feature/refactoring_ctrl_msg_view branch from 7acdb8d to 60462fd Compare October 29, 2026 15:07
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval October 29, 2026 15:07 — with GitHub Actions Waiting
@kratz00
Differentiate CallMessageBuilder
41d32c2 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval October 30, 2026 15:01 — with GitHub Actions Waiting
@kratz00
Fixed clang-tidy findings
2f1c66f 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval October 30, 2026 15:40 — with GitHub Actions Waiting
@kratz00
Renamed method 'create_call_message_builder'
d01348a 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
Improved documentation builders and stream id relationship
21c2991 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
Sonar fixes.
ff1d6d9 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
Increased test coverage
dbaab10 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval October 30, 2026 16:33 — with GitHub Actions Waiting
@kratz00
Simplified error handling
5166eb4 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval October 30, 2026 16:58 — with GitHub Actions Waiting
@kratz00
Fixed Sonar findings
17762c3 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 deployed to v2-fuzzing-pr-approval November 2, 2026 15:38 — with GitHub Actions Active
@kratz00
kratz00 marked this pull request as ready for review November 2, 2026 18:09
@kratz00
kratz00 requested a review from tkilias November 2, 2026 18:09
tkilias
tkilias reviewed Nov 2, 2026
View reviewed changes
Comment thread doc/design/v2/runner/context_implementation.md Resolved
tkilias
tkilias reviewed Nov 2, 2026
View reviewed changes
Comment thread doc/design/v2/runner/context_implementation.md Resolved
@kratz00
Made stream_id 'required'
d6895c9 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval November 3, 2026 12:41 — with GitHub Actions Waiting
@kratz00
Fixed exception usage
6ef2c47 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 deployed to v2-fuzzing-pr-approval November 3, 2026 13:02 — with GitHub Actions Active
@kratz00
kratz00 requested a review from tkilias November 3, 2026 15:50
tkilias
tkilias reviewed Nov 3, 2026
View reviewed changes
Comment thread udf-runner-cpp/v2/BUILD.bazel Resolved
tkilias
tkilias reviewed Nov 3, 2026
View reviewed changes
Comment thread udf-runner-cpp/v2/udf_protocol.fbs Resolved
@kratz00
Rollback stream_id stays a scalar
e2b07aa 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval November 4, 2026 08:39 — with GitHub Actions Waiting
@kratz00
Added tests (decodedFrame = nullptr)
40f6855 
Signed-off-by: Steffen Pankratz <steffen.pankratz@exasol.com>
@kratz00
kratz00 deployed to v2-fuzzing-pr-approval November 4, 2026 09:10 — with GitHub Actions Active
@kratz00
kratz00 requested a review from tkilias November 4, 2026 09:24
tkilias
tkilias previously approved these changes Nov 4, 2026
View reviewed changes
@kratz00
Merge remote-tracking branch 'origin/main' into feature/refactoring_c…
1c3f856 
…trl_msg_view

# Conflicts:
#	doc/design/v2/runner/README.md
@kratz00
kratz00 requested a deployment to v2-fuzzing-pr-approval November 4, 2026 10:51 — with GitHub Actions Waiting
@sonarqubecloud

sonarqubecloud Bot commented Nov 4, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
90.6% Coverage on New Code
1.1% Duplication on New Code

See analysis details on SonarQube Cloud

@kratz00
kratz00 merged commit 9041b1d into main Nov 4, 2026
34 of 35 checks passed
@kratz00
kratz00 deleted the feature/refactoring_ctrl_msg_view branch November 4, 2026 10:59

This branch is waiting to be deployed

1 waiting deployment
v2-fuzzing-pr-approval — 1c3f8562 Waiting Nov 4, 2026 by kratz00 via pr_approval #306
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tkilias tkilias tkilias approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

V2 Refactoring: MessageViews + MessageBuilders

2 participants

@kratz00 @tkilias