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.

Sign up for GitHub

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

Jump to bottom

Develop #97

Open
wants to merge 140 commits into
base: master
Choose a base branch
from
Open

Develop #97

wants to merge 140 commits into from

Conversation

nareshkumarthota
Copy link
Contributor

This PR adds following functionality:

  1. Flogo-expression based rule conditions.
  2. Invoke flogo-activity / flogo-action when rule gets fired.
  3. Add external store (redis & keydb) support to maintain current rete network state.
  4. Decision table support.
  5. Upgrade flogo-core version to 10.2
  6. Add / Improved examples
  7. Add / Improved gotests

@balamg
expr based condition
fa5167c
@balamg
expr based condition
c4a62bf
@balamg
Merge branch 'master' into expr
ac8f424
@balamg
expr based condition
be0d36d
@balamg
expr based condition
1a28a29
@balamg
expr based condition
98283b6
@balamg
Expression syntax for conditions
cb3df5d 
Merge branch 'expr' into noschema
@balamg
Refactored to use the new expression API of project-flogo/core
b8a741d
@balamg
iterator interface for join table iteration
2b51fea
@balamg
assign integer id to jointablerows
556f7a2
@balamg
handle holds ids of tables and rows
6ec22a2
@balamg
refactor for jointables and rows
59c5dd2
@balamg
refactor for jointables and rows
1d11109
@balamg
refactor for jointables and rows
69c5fc8
@balamg
refactor for jointables and rows
dca233e
@balamg
refactor for jointables and rows
1b9bf49
@balamg
refactor interfaces in order to separate out and organize in-mem impl…
2ec7942 
…ementation
@balamg
refactor interfaces in order to separate out and organize in-mem impl…
e9a9738 
…ementation
@balamg
configuration based impl
61efc06
@balamg
need the condition name to get correponding join tables
448b0b6
@balamg
refactor
204eec4
@balamg
refactor
253adbe
@balamg
refactor
bc213e8
@balamg
refactor
8c970dc
@balamg
refactor
2dce1eb
@balamg
refactor
9f2b44d
@balamg
refactor
4e5097a
@balamg
rename and refactor
35a141d
@balamg
added redis utility methods
fa6c281
@balamg
redis util methods
c686129
pointlander and others added 26 commits December 3, 2019 12:11
@pointlander
Added formal expression parser
60b843a
@pointlander
Improve parser
af493e6
@pointlander
Removed license
4226a51
@pointlander
Refactor
7625da3
@rameshpolishetti
upgrade excelize version to latest
56747f9
@rameshpolishetti
Merge pull request #83 from project-flogo/feature-decision-table-parser
bded2de 
Feature decision table parser
@nareshkumarthota @rameshpolishetti
updated example to use store config file (#81)
9c8f217 
* updated example to use store config file

* fixing excelise v2 upgrade

* update decisiontable example to work with xlsx file and fix go tests (#82)

* update decisiontable example to work with xlsx file and fix go tests

* removing duplicated flogo.json from test folder

* updated decision table example readme

* stabilising mod and sum files
@aashish2001 @nareshkumarthota
Update README.md (#86)
97a95d0
@nareshkumarthota
Merge branch 'develop' into feature-decision-table
077f860
@nareshkumarthota
removed redundant examples
eb1f235
@rameshpolishetti
Merge pull request #87 from project-flogo/feature-decision-table
d57fcc0 
Feature decision table
@LakshmiMekala @rameshpolishetti
Feature travisbuilds (#88)
45b1957 
* Adding travis file with basic configuration

Test travis file currently runs go tests and this will be enhanced further

* travis file enhancements

* updating travis file install latest flogo cli

* restructuring travis file

* resolving build failure issue with gomodules

* fix build failures

* pass correct env values to travis file
@nareshkumarthota
resolving conflicts
4fea442
@nareshkumarthota
fixing replaytplesforrule
c8c8125
@nareshkumarthota
adding replay tuple method in rete
e4d69bd
@nareshkumarthota
tuple null handled in redis
fb4ff64
@nareshkumarthota
upgrading flogo dependency versions
4567024
@nareshkumarthota
go modules updated
2bffc7e
@nareshkumarthota
resolving conflicts
973a7f4
@rameshpolishetti
Merge pull request #90 from project-flogo/develop-master
41ced91 
Merge from master
@nareshkumarthota
added fix for issue 91
05505b2
@rameshpolishetti
Merge pull request #92 from project-flogo/fix-issue-91
2e1fe55 
fixes issue -  #91
@ykalidin
Feature ruleapi examples (#94)
2e3a62f 
* Add examples

* Updating examples

* Updating comments
@nareshkumarthota @ykalidin
Feature Decision Table API model changes (#93)
0a7f842 
* dtable code refactoring to support api model

* adding dtable ruleapi model example

* adding dtable interface

* example modified

* decision table api model refactored and example modified

* Adding Readme

* Update Readme

* go tests added

* code refactoring

Co-authored-by: ykalidin <ykalidin@tibco.com>
@ykalidin
Updating examples (#95)
51126dd 
* Updateing examples

* Update Test
@ykalidin
Chore update readme (#96)
92dca6f 
* Updating Readme

* Update Readme

* Updating Readme
@balamg balamg self-requested a review April 1, 2020 07:44
@nareshkumarthota nareshkumarthota mentioned this pull request Apr 3, 2020
Merged
rameshpolishetti
rameshpolishetti requested changes Apr 29, 2020
View reviewed changes
Copy link
Contributor

@rameshpolishetti rameshpolishetti left a comment

Choose a reason for hiding this comment

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

Since this PR contains several features. It's not a good idea to merge at once, instead break this PR into feature-based PRs and submit one after another.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rameshpolishetti rameshpolishetti rameshpolishetti requested changes

@balamg balamg Awaiting requested review from balamg

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@nareshkumarthota @rameshpolishetti @balamg @LakshmiMekala @pointlander @aashish2001 @ykalidin