LuciferYang commented on code in PR #49434:
URL: https://github.com/apache/spark/pull/49434#discussion_r1909867794
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala:
##########
@@ -338,8 +339,11 @@ class JacksonParser(
UTF8String.fromBytes(writer.toByteArray)
}
}
- // Reset back to the original configuration:
- parser.setFeatureMask(oldFeature)
+ // Reset back to the original configuration using `~0` as the mask,
+ // which is a bitmask with all bits set, effectively allowing all
features
+ // to be reset. This ensures that every feature is restored to its
previous
+ // state as defined by `oldFeature`.
+ parser.overrideStdFeatures(oldFeature, ~0)
Review Comment:
can also use a seat mask of `-1` to restore the state, as long as all the
bits are set to 1.
--
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]
