HyukjinKwon edited a comment on pull request #29172: URL: https://github.com/apache/spark/pull/29172#issuecomment-661637178
I thought it's better to be safe always for `originalMap` to contain the original map as is, and now it's broken. `originalMap` is an exposed property in `CaseInsensitiveMap`. Now once you combine `CaseInsensitiveMap` with another entry, e.g. `caseInsensitiveMap + ("option" -> 1)`, the `originalMap` removes the duplicates such as `Option`, `oPtion`, etc. Maybe I am too much cautious about it. I agree that it's not going to cause an external problem for now. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org