mboehm7 commented on pull request #1284: URL: https://github.com/apache/systemds/pull/1284#issuecomment-850911151
LGTM - thanks for the patch @OlgaOvcharenko. During the merge, I cleaned up some surrounding redundancy, and fixed two minor things in the tokenization (exception handling, unnecessary symbol table changes at coordinator). Furthermore, note that JSON does not need double quotes around attribute names, so we can avoid the escaping `\"attribute\":value` at DML script level. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
