Correct default formatting of binary fields (#58338)

This was picked up in the backport of #57666 but missed in the original commit
on master, leading to failures such as #58282

Author: romseygeek

server/src/main/java/org/elasticsearch/index/mapper/BinaryFieldMapper.java

@@ -67,6 +67,7 @@ public static class Builder extends FieldMapper.Builder<Builder> {
 
         public Builder(String name) {
             super(name, Defaults.FIELD_TYPE);
+            hasDocValues = false;
             builder = this;
         }
 
@@ -207,6 +208,16 @@ protected void parseCreateField(ParseContext context) throws IOException {
 
     }
 
+    @Override
+    protected boolean indexedByDefault() {
+        return false;
+    }
+
+    @Override
+    protected boolean docValuesByDefault() {
+        return false;
+    }
+
     @Override
     protected void mergeOptions(FieldMapper other, List<String> conflicts) {
 

server/src/test/java/org/elasticsearch/index/mapper/BinaryFieldMapperTests.java

@@ -74,6 +74,8 @@ public void testDefaultMapping() throws Exception {
 
         assertThat(mapper, instanceOf(BinaryFieldMapper.class));
         assertThat(mapper.fieldType.stored(), equalTo(false));
+
+        assertEquals(Strings.toString(mapping), Strings.toString(mapperService.documentMapper()));
     }
 
     public void testStoredValue() throws IOException {