-
Notifications
You must be signed in to change notification settings - Fork 492
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
add host functions for cluster data (flash sale support) #188
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.
[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 188 Deploy Test Success
Codecov Report
@@ Coverage Diff @@
## main #188 +/- ##
==========================================
- Coverage 81.20% 80.12% -1.08%
==========================================
Files 44 44
Lines 4846 4835 -11
==========================================
- Hits 3935 3874 -61
- Misses 694 740 +46
- Partials 217 221 +4
Continue to review full report at Codecov.
|
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.
[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 188 Deploy Test Success
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.
[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 188 Deploy Test Success
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.
[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 188 Deploy Test Success
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.
[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 188 Deploy Test Success
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.
LGTM
…o#188) * add pipeline name to filter * add host functions for cluster data * add APIs for wasm cluster data * update document * revise host functions to allow zeros in string * improve test coverage * fix typo * revert 1st commit, pass pipeline through meta data * updated according to comments
add host functions for accessing shared cluster data.
add cookbook for flash sale.
other enhancement.
Too many files are modified, propose to review commit by commit.