Set default value of spark.tispark.telemetry.enable as false (#2621) (#2623)

ti-chi-bot · web-flow · commit acd67ec40f7b · 2023-02-21T10:59:05.000+08:00
diff --git a/core/src/main/scala/org/apache/spark/sql/TiExtensions.scala b/core/src/main/scala/org/apache/spark/sql/TiExtensions.scala
@@ -122,14 +122,14 @@ object TiExtensions {
 
   /**
    * check that telemetry is on
-   * default on
+   * default off
    *
    * @param sparkSession
    * @return
    */
   def telemetryEnable(sparkSession: SparkSession): Boolean = {
     sparkSession.sparkContext.conf
-      .get(TELEMETRY_ENABEL, "true")
+      .get(TELEMETRY_ENABEL, "false")
       .toBoolean
   }
 }
diff --git a/docs/telemetry.md b/docs/telemetry.md
@@ -1,6 +1,6 @@
 # Telemetry
-By default, TiSpark collect usage information and share the information with PingCAP. You can close it by configuring
-`spark.tispark.telemetry.enable = false` in `spark-default.conf`.
+By default, TiSpark does not collect usage information and share the information with PingCAP. You can enable it by configuring
+`spark.tispark.telemetry.enable = true` in `spark-default.conf`.
 
 When the telemetry collection feature is enabled, usage information will be shared, including(but not limited to):
 * A randomly generated telemetry ID.

Original file line number	Diff line number	Diff line change
`@@ -122,14 +122,14 @@ object TiExtensions {`
`122`	`122`
`123`	`123`	`/**`
`124`	`124`	`* check that telemetry is on`
`125`		`- * default on`
	`125`	`+ * default off`
`126`	`126`	`*`
`127`	`127`	`* @param sparkSession`
`128`	`128`	`* @return`
`129`	`129`	`*/`
`130`	`130`	`def telemetryEnable(sparkSession: SparkSession): Boolean = {`
`131`	`131`	`sparkSession.sparkContext.conf`
`132`		`- .get(TELEMETRY_ENABEL, "true")`
	`132`	`+ .get(TELEMETRY_ENABEL, "false")`
`133`	`133`	`.toBoolean`
`134`	`134`	`}`
`135`	`135`	`}`