voonhous opened a new issue, #17750:
URL: https://github.com/apache/hudi/issues/17750

   ### 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 https://github.com/apache/hudi/pull/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.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to