@@ -455,7 +455,6 @@ public Expression replaceQueryBuilder(QueryBuilder queryBuilder) {
455455 @ Override
456456 public BiConsumer <LogicalPlan , Failures > postAnalysisPlanVerification () {
457457 return (plan , failures ) -> {
458- // TODO: fix this.
459458 super .postAnalysisPlanVerification ().accept (plan , failures );
460459 plan .forEachExpression (Match .class , match -> {
461460 for (Expression field : fields ) {
@@ -513,7 +512,7 @@ protected Query translate(TranslatorHandler handler) {
513512 return new MultiMatchQuery (source (), Objects .toString (queryAsObject ()), fieldsWithBoost , matchQueryOptions ());
514513 }
515514
516- public static String getNameFromFieldAttribute (FieldAttribute fieldAttribute ) {
515+ private static String getNameFromFieldAttribute (FieldAttribute fieldAttribute ) {
517516 String fieldName = fieldAttribute .name ();
518517 if (fieldAttribute .field () instanceof MultiTypeEsField multiTypeEsField ) {
519518 // If we have multiple field types, we allow the query to be done, but getting the underlying field name
@@ -522,7 +521,7 @@ public static String getNameFromFieldAttribute(FieldAttribute fieldAttribute) {
522521 return fieldName ;
523522 }
524523
525- public static FieldAttribute fieldAsFieldAttribute (Expression field ) {
524+ private static FieldAttribute fieldAsFieldAttribute (Expression field ) {
526525 Expression fieldExpression = field ;
527526 // Field may be converted to other data type (field_name :: data_type), so we need to check the original field
528527 if (fieldExpression instanceof AbstractConvertFunction convertFunction ) {
0 commit comments