aokolnychyi commented on code in PR #10309:
URL: https://github.com/apache/iceberg/pull/10309#discussion_r1597024872
##########
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkConfParser.java:
##########
@@ -70,6 +70,14 @@ public DurationConfParser durationConf() {
return new DurationConfParser();
}
+ private static CaseInsensitiveStringMap
asCaseInsensitiveStringMap(Map<String, String> map) {
Review Comment:
Prior to split of Spark 2 and Spark 3, we had to use `Map<String, String>`
as Spark 3 used one class for options and Spark 2 used another one. It is
always `CaseInsensitiveStringMap` in Spark 3 but I am checking here to be safe.
--
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]
