issues
search
flipkart-incubator
/
Krystal
Apache License 2.0
8
stars
14
forks
source link
Krystal 5
#189
Closed
RamAnvesh
closed
10 months ago
RamAnvesh
commented
11 months ago
Features:
.vajram.yaml
s have been replaced with annotated fields in vajram itself
gradle plugin triggered codegen is replaced with java annotation processor
made ID optional in @VajramDef annotation - and made ID Inferrable from className
BugFixes:
Fixed bug introduced in a previous change because of which MultiLeasePool was not pooling threads - leading to millions of threads and OOMs in local benchmark tests where millions of requests are spawned within seconds. (This issue doesn't seem to be happening in production where the number of requests is much smaller.) Change introducing the bug:
https://github.com/flipkart-incubator/Krystal/commit/f8114da09855da125f97c43c303a6fdd5927e5f4#diff-a5df40ecf3aef35683157402a9550989642de5f6cfdb016514a8d1ff9b0930caR61-R65
Other Changes:
Clean up unnecessary code after the annotation processor change
Remove Caramel project module as its been superseded by the vajram programming model for async workflows as well.
Features:
.vajram.yaml
s have been replaced with annotated fields in vajram itselfBugFixes:
Other Changes: