better name and documentation

Author: XiXiaPdx

newrelic-agent/src/main/java/com/newrelic/agent/tracers/DefaultSqlTracer.java

@@ -270,7 +270,7 @@ protected void recordMetrics(TransactionStats transactionStats) {
             if (sqlObject != null) {
                 slowQuerySql = new PreparedStatementSql(sql, params).toString();
                 if (queryExceedsSlowQueryThreshold(appName)) {
-                    slowQuerySql = obfuscateRawSql(slowQuerySql, appName);
+                    slowQuerySql = getNoRawOrObfuscatedSql(slowQuerySql, appName);
                 }
             }
 
@@ -304,8 +304,12 @@ protected void recordMetrics(TransactionStats transactionStats) {
         super.recordMetrics(transactionStats);
     }
 
-    private String obfuscateRawSql(String rawSql, String appName) {
+    private String getNoRawOrObfuscatedSql(String rawSql, String appName) {
         SqlQueryConverter converter = new SqlQueryConverter(appName, getDatabaseVendor());
+        //This method is named this way because the docs on toObfuscatedQueryString imply obfuscation certainty.
+        //The reality is the SqlQueryConverter implementation of this method has introduced conditional
+        //logic that respects the agent configuration setting `record_sql`.
+        //If OBFUSCATED, sql is obfuscated. If RAW, sql is returned, unchanged. If OFF, null is returned.
         return converter.toObfuscatedQueryString(rawSql);
     }