chore: leverage .openapi-generator-ignore in python client generation

[xerryd]

This PR uses the .openapi-generator-ignore file for the Python client generation, instead of running manual rm commands afterwards. The OpenAPI generator for the Python client requires the ignore file to be in the output directory.

However, the generation metadata .openapi-generator/ is something that can't be skipped in the python client generation, but there are some open PRs to address this across all clients which seem to have been open for a while now. For example: OpenAPITools/openapi-generator#15770.

I see that the repo is also cleaning .openapi-generator-ignore files so feel free to close this if not needed.

Testing

make clean-python && make gen-python

[github-actions]

ACTION NEEDED
Lance follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

For details on the error please inspect the "PR Title Check" action.

[yanghua]

@bxlq, thanks for your contribution, let me verify your changes.

[yanghua]

LGTM

[yanghua]

cc @jackye1995

[jackye1995]

Thanks for the fix! We should probably do the same for the other generated modules as well

[jackye1995]

Oh actually, I noticed one issue, when we do clean we remove the entire directory. That's why I put an external ignore file for the Apache client in Java. Can we do the same here?

[xerryd]

The Python OpenAPI code generation template is stingy and requires the ignore files to be located in the output directory. However, right now the make clean only removes non-hidden files in the output directory so the new .openapi-generator-ignore doesn't actually get removed.

To delete everything, including hidden files, we would need to add rm -rf lance_namespace_urllib3_client/** lance_namespace_urllib3_client/.* to make clean

[jackye1995]

right now the make clean only removes non-hidden files in the output directory so the new .openapi-generator-ignore doesn't actually get removed.

great! In that case I think we can also do the same for the apache client folder, instead of making an external ignore file so it is consistent. Merging this!