GetIndexedFieldExpr dyn_eq implementation causes stack overflow

[mustafasrepo]

Describe the bug

impl PartialEq<dyn Any> of GetFieldAccessExpr which is a field of GetIndexedFieldExpr has a recursive implementation without a base case. This causes stack overflow.

To Reproduce

Test below can be used to reproduce bug

#[test]
fn get_indexed_field_eq() -> Result<()>{
    let schema = list_schema(&["list", "error"]);
    let expr = col("list", &schema).unwrap();
    let key = col("error", &schema).unwrap();
    let indexed_field = Arc::new(GetIndexedFieldExpr::new_index(expr.clone(), key.clone())) as Arc<dyn PhysicalExpr>;
    let indexed_field_other = Arc::new(GetIndexedFieldExpr::new_index(key, expr)) as Arc<dyn PhysicalExpr>;
    assert!(indexed_field.eq(&indexed_field));
    assert!(!indexed_field.eq(&indexed_field_other));
    Ok(())
}

Expected behavior

No response

Additional context

No response