This is an automated email from the ASF dual-hosted git repository.
yangjie01 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 878de0014a37 [SPARK-48626][CORE] Change the scope of object LogKeys as
private in Spark
878de0014a37 is described below
commit 878de0014a3782187180c40158f0805e51335cb5
Author: Gengliang Wang <[email protected]>
AuthorDate: Fri Jun 14 15:08:41 2024 +0800
[SPARK-48626][CORE] Change the scope of object LogKeys as private in Spark
### What changes were proposed in this pull request?
Change the scope of object LogKeys as private in Spark.
### Why are the changes needed?
LogKeys are internal and developing. Making it private can avoid future
confusion or compiling failures.
This is suggested by pan3793 in
https://github.com/apache/spark/pull/46947#issuecomment-2167164424
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Existing UT
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #46983 from gengliangwang/changeScope.
Authored-by: Gengliang Wang <[email protected]>
Signed-off-by: yangjie01 <[email protected]>
---
common/utils/src/main/scala/org/apache/spark/internal/LogKey.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/common/utils/src/main/scala/org/apache/spark/internal/LogKey.scala
b/common/utils/src/main/scala/org/apache/spark/internal/LogKey.scala
index b8b63382fe4c..ec621c4f84ce 100644
--- a/common/utils/src/main/scala/org/apache/spark/internal/LogKey.scala
+++ b/common/utils/src/main/scala/org/apache/spark/internal/LogKey.scala
@@ -57,7 +57,7 @@ trait LogKey {
* Various keys used for mapped diagnostic contexts(MDC) in logging. All
structured logging keys
* should be defined here for standardization.
*/
-object LogKeys {
+private[spark] object LogKeys {
case object ACCUMULATOR_ID extends LogKey
case object ACL_ENABLED extends LogKey
case object ACTUAL_NUM_FILES extends LogKey
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
