-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HBASE-25459 WAL can't be cleaned with phoenix5 #2842
Conversation
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should add logic dealing with the specifics of a 3rd party project (phoenix) into hbase wal handling.
What about changing HBase WALFactory to allow pluggable custom wal providers implementations? Then this phoenix specific logic could be implemented on a Phoenix wal provider, keeping hbase agnostic to phoenix. AbstractFSWALProvider/WALProvider interfaces are all marked as IA Private, though, so it would be challenging maintain compatibility of custom wal providers.
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
I think you are right! I will close this pr! |
|
No description provided.