[CALCITE-5008] Ignore synthetic and static methods in MetadataDef

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.29.0
• Fix Version/s: 1.30.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Roman Puchkovskiy
• Reporter: Roman Puchkovskiy
• Created: 10/Feb/22 12:03
• Updated: 19/Mar/22 09:15
• Resolved: 21/Feb/22 22:22

Description

According to https://www.jacoco.org/jacoco/trunk/doc/faq.html , Jacoco adds a synthetic static method called $jacocoInit() during instrumentation.

MetadataDef constructor breaks if it encounters such method on the class given to it because it does not expect it. But it looks like it simply should not consider synthetic methods (as well as static methods) at all.

Issue Links

• links to: GitHub Pull Request #2718