Address minor review comments.

Signed-off-by: currantw <[email protected]>

doctest/test_docs.py

@@ -1,20 +1,21 @@
 # Copyright OpenSearch Contributors
 # SPDX-License-Identifier: Apache-2.0
 
-import click
 import doctest
-import json
 import os
 import os.path
-import random
+import zc.customdoctests
+import json
 import re
+import random
 import subprocess
 import unittest
-import zc.customdoctests
+import click
+
 from functools import partial
-from opensearch_sql_cli.formatter import Formatter
 from opensearch_sql_cli.opensearch_connection import OpenSearchConnection
 from opensearch_sql_cli.utils import OutputSettings
+from opensearch_sql_cli.formatter import Formatter
 from opensearchpy import OpenSearch, helpers
 
 ENDPOINT = "http://localhost:9200"

opensearch/src/main/java/org/opensearch/sql/opensearch/data/value/OpenSearchExprValueFactory.java

@@ -228,9 +228,7 @@ public void extendTypeMapping(Map<String, OpenSearchDataType> typeMapping) {
     for (var field : typeMapping.keySet()) {
       // Prevent overwriting, because aggregation engine may be not aware
       // of all niceties of all types.
-      if (!this.typeMapping.containsKey(field)) {
-        this.typeMapping.put(field, typeMapping.get(field));
-      }
+      this.typeMapping.putIfAbsent(field, typeMapping.get(field));
     }
   }