Skip to content

Commit df5e056

Browse files
committed
[MINOR][SQL] Remove signature from Hive thriftserver DEBUG log
### What changes were proposed in this pull request? This PR aims to remove signature from Hive thriftserver DEBUG log. ### Why are the changes needed? This is aligned with the upstream Apache Hive project. - apache/hive#4887 This is also aligned with the previous Spark change. - apache#43402 ### Does this PR introduce _any_ user-facing change? No, this is a debug message removal. ### How was this patch tested? Pass the CIs and manual review (because this is a removal). ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#49752 from dongjoon-hyun/minor_debug_msg. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit 04e0ddf) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent 8229aa2 commit df5e056

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

sql/hive-thriftserver/src/main/java/org/apache/hive/service/CookieSigner.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,6 @@ public String verifyAndExtract(String signedStr) {
7878
String rawValue = signedStr.substring(0, index);
7979
String currentSignature = getSignature(rawValue);
8080

81-
if (LOG.isDebugEnabled()) {
82-
LOG.debug("Signature generated for " + rawValue + " inside verify is " + currentSignature);
83-
}
8481
if (!MessageDigest.isEqual(originalSignature.getBytes(), currentSignature.getBytes())) {
8582
throw new IllegalArgumentException("Invalid sign");
8683
}

0 commit comments

Comments
 (0)