[CALCITE-3196] In Frameworks, add BasePrepareAction (a functional interface) and deprecate PrepareAction

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.21.0
• Labels: pull-request-available
• Assignee: Danny Chan
• Reporter: Julian Hyde
• Created: 13/Jul/19 01:54
• Updated: 16/Jul/19 01:32
• Resolved: 15/Jul/19 17:55

Description

In Frameworks, add interface BasePrepareAction (a functional interface) and deprecate abstract class PrepareAction. Because PrepareAction has a field (FrameworkConfig config), it cannot be implemented using a lambda. It is simpler and clearer to pass config as an argument to all methods where it is needed.

PrepareAction was introduced in CALCITE-247.

Issue Links

• is related to: CALCITE-247 Add FrameworkContext and FrameworkConfig
• links to: GitHub Pull Request #1318