anishshri-db commented on code in PR #51484:
URL: https://github.com/apache/spark/pull/51484#discussion_r2208435421


##########
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/HDFSBackedStateStoreProvider.scala:
##########
@@ -370,6 +373,8 @@ private[sql] class HDFSBackedStateStoreProvider extends 
StateStoreProvider with
 
   override def stateStoreId: StateStoreId = stateStoreId_
 
+  override protected def logName: String = s"${super.logName} 
${stateStoreProviderId}"

Review Comment:
   In the final log, can we make sure to prefix the runId with the field name ?
   
   So, have something like `runId = <queryRunId>` ? It seems like we are only 
printing the value here ?



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

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to