[GitHub] [incubator-dolphinscheduler] yangyichao-mango edited a comment on pull request #3300: [BugFix] isssue #3299

[GitBox]

yangyichao-mango edited a comment on pull request #3300:
URL: 
https://github.com/apache/incubator-dolphinscheduler/pull/3300#issuecomment-663387747


   Hi,
   The type of `{'a': "b"}` and `{'a': 'b'}` is not a standard json string, but 
if we add the `ALLOW_SINGLE_QUOTES`, it will be formatted correctly by 
`JSONUtils`. If we just add `ALLOW_SINGLE_QUOTES` for `JSONUtils`, perhaps it 
will cause some unforeseen bugs.
   I think a better way to fix this bug is format the original json not the 
JSONUtils.
   
![image](https://user-images.githubusercontent.com/29545877/88369458-ac857b80-cdc2-11ea-94f0-cda537d61772.png)
   


----------------------------------------------------------------
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