Skip to content

Conversation

@pan3793
Copy link
Member

@pan3793 pan3793 commented Apr 9, 2024

🔍 Description

This is the follow-up of #5686, renaming ./pyhive to ./python, and also adding **/python/* to RAT exclusion list temporarily.

"PyHive" may not be a suitable name after being part of Apache Kyuubi, let's use a generic dir name python, and discuss the official name later(we probably keep the code at ./python eventually).

Types of changes 🔖

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Recover RAT checked.


Checklist 📝

Be nice. Be informative.

@pan3793
Copy link
Member Author

pan3793 commented Apr 9, 2024

@pan3793 pan3793 self-assigned this Apr 9, 2024
@pan3793 pan3793 added this to the v1.10.0 milestone Apr 9, 2024
@pan3793 pan3793 closed this in f8c7b93 Apr 9, 2024
Copy link
Contributor

@cxzl25 cxzl25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Later LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants