dylanwong250 commented on code in PR #51484:
URL: https://github.com/apache/spark/pull/51484#discussion_r2208585084
##########
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:
I added the override to the toString method on StateStoreProviderId to have
a similar format as the StateStoreId toString method. The updated log prefix is
```StateStoreProviderId[ storeId=$storeId, queryRunId=$queryRunId ]```. I also
updated the example in the PR description.
--
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]
