tledkov-gridgain commented on pull request #8947:
URL: https://github.com/apache/ignite/pull/8947#issuecomment-810093466


   > Can you, please, clarify, why you treat the query plan as sensitive info?
   Please take a look on the usage of the `S#includeSensitive`. The class names 
and field names are hidden in log messages.
   SQL queries and plans also contains names of the tables & fields and must be 
hidden in log messages.
   
   I guess the behavior of hiding sensitive info must be consistent.
   
   I don't see any reason to hide SQL constants at the system views because 
user may use parameters for sensitive info and access to system views must 
check security  permission.


-- 
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:
[email protected]


Reply via email to