jkevan commented on code in PR #461:
URL: https://github.com/apache/unomi/pull/461#discussion_r918866072
##########
tools/shell-commands/src/main/resources/requestBody/2.0.0/update_profile_interests.json:
##########
@@ -0,0 +1,4 @@
+{
+ "source": "List interests = new ArrayList();
if(ctx._source.properties.interests != null &&
!(ctx._source.properties.interests instanceof List)){ for (entry in
ctx._source.properties.interests.entrySet()){ Map interest = new HashMap();
interest.put('key',entry.getKey()); interest.put('value', entry.getValue());
interests.add(interest);} ctx._source.properties.remove('interests');
ctx._source.properties.put('interests',interests);}",
+ "lang": "painless"
+}
Review Comment:
It could be interesting to help on dev + review painless scripts that the
script it self should be the file content.
like: update_profile_interests.painless.
Even if IDE is not capable of handling those file, may be one day it will.
Also it would help a lot to benefit a least from indentation to be able to
read/debug thoses scripts.
Also the "source" and "lang" could be created on the fly by the reindex
function.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]