zhugezifang commented on PR #4591:
URL: https://github.com/apache/zeppelin/pull/4591#issuecomment-1527297261

   > > @Reamer hi,i also want to add this feature into zeppelin,and could I get 
your support?
   > 
   > Of course. I was able to reproduce the problem and understood the problem. 
Your change definitely solves the problem. It's just not a JSON specific 
problem, but a general masking problem.
   > 
   > I would remove the if statement. It is not necessary because replaceAll 
also looks for the quotes.
   > 
   > ```
   > // Mask all quotes
   > stringValue = stringValue.replaceAll('"', '""');
   > ```
   
   ok,i update it later and then add a new pr


-- 
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: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to