Skip to content

Commit d3679a9

Browse files
HeartSaVioRMarcelo Vanzin
authored andcommitted
[SPARK-27748][SS][FOLLOWUP] Correct the order of logging token as debug log
### What changes were proposed in this pull request? This patch fixes the order of elements while logging token. Header columns are printed as ``` "TOKENID", "HMAC", "OWNER", "RENEWERS", "ISSUEDATE", "EXPIRYDATE", "MAXDATE" ``` whereas the code prints out actual information as ``` "HMAC"(redacted), "TOKENID", "OWNER", "RENEWERS", "ISSUEDATE", "EXPIRYDATE", "MAXDATE" ``` This patch fixes this. ### Why are the changes needed? Not critical but it doesn't line up with header columns. ### Does this PR introduce any user-facing change? No ### How was this patch tested? N/A, as it's only logged as debug and it's obvious what/where is the problem and how it can be fixed. Closes #25935 from HeartSaVioR/SPARK-27748-FOLLOWUP. Authored-by: Jungtaek Lim (HeartSaVioR) <kabhwan@gmail.com> Signed-off-by: Marcelo Vanzin <vanzin@cloudera.com>
1 parent 8beb736 commit d3679a9

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

external/kafka-0-10-token-provider/src/main/scala/org/apache/spark/kafka010/KafkaTokenUtil.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -241,8 +241,8 @@ private[spark] object KafkaTokenUtil extends Logging {
241241
"TOKENID", "HMAC", "OWNER", "RENEWERS", "ISSUEDATE", "EXPIRYDATE", "MAXDATE"))
242242
val tokenInfo = token.tokenInfo
243243
logDebug("%-15s %-15s %-15s %-25s %-15s %-15s %-15s".format(
244-
REDACTION_REPLACEMENT_TEXT,
245244
tokenInfo.tokenId,
245+
REDACTION_REPLACEMENT_TEXT,
246246
tokenInfo.owner,
247247
tokenInfo.renewersAsString,
248248
dateFormat.format(tokenInfo.issueTimestamp),

0 commit comments

Comments
 (0)