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

   > @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('"', '""');
   ```
   


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