[HBASE-18324] [hbase-thirdparty] Tooling to prevent commits that mistakenly make use of unshaded protobuf/guava/etc.

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Assignee: Unassigned
• Reporter: Michael Stack
• Created: 05/Jul/17 22:19
• Updated: 07/Jul/17 01:08

Description

Chatting w/ mdrob, he brought up the question of what if a dev makes use of unshaded protobuf or guava? Afterall, the old jars (pb2.5 and hadoops guava12.0 or whatever) are still on the CLASSPATH because upstream depends on them.

We could add a check as part of prebuild. It would complain if use of com.google instead of org.apache.hadoop.hbase.shaded.com.google. But even then, there are cases where com.google is legit (coprocessor endpoints) so it would have to let these pass.

TODO.