-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[kie-issues-249] Data Index Events changes for allowing historical data #5529
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all new files requires the use of Apache headers and not Red Hat
6a4efec
to
9c0584d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it necessary to have this in Drools? It seems very Kogito specific so why not move it there?
@mariofusco I hear you, and it makes sense. however if you check all events related to process are already here, so I'd argue that we could discuss in a different thread and eventually PR moving the process related content from drools to kogito. |
@mariofusco unfortunately is how the design works. the idea is to strip the jbpm engine in the medium term. so for now we cannot change the dependencies just to acommodate this. it is out of the scope of the jira. |
revisit dependencies is out of the scope of the jira.
@elguardian @porcelli That's fine, for now it's ok for me. If possible please open another ticket to better isolate this stuff in a second step. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
248b893
to
ac8d7fa
Compare
ac8d7fa
to
a5a2f3d
Compare
Issue: apache/incubator-kie-issues#249