Investigate why there are validateDataTypeForSecondaryIndex and validateDataTypeForSecondaryOrExpressionIndex #17750
Description
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.