Task Description

What needs to be done:

We need to follow up separately why there are two methods: validateDataTypeForSecondaryIndex, validateDataTypeForSecondaryOrExpressionIndex. From the naming, they seem to overlap.

In #17599, changes were made to the validateDataTypeForSecondaryOrExpressionIndex method signature, changing the parameter type of Avro.Schema to HoodieSchema, from here, we noticed that there seems to be some method overlap.

Why this task is needed:

Removing code smell and potential bugs.

Task Type

Code improvement/refactoring

Related Issues

Parent feature issue: (if applicable )
Related issues:
NOTE: Use Relationships button to add parent/blocking issues after issue is created.