CarnationWang23 / hyracks

Automatically exported from code.google.com/p/hyracks
Apache License 2.0
0 stars 0 forks source link

Add policy interface for InlineVariablesRule #99

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Description of the desired feature(s) (incl. use cases, if applicable)
The current InlineVariablesRule works for a specific case. Inlining of 
functions and variables that the top level of the query plan. I suggest 
changing the structure to include an interface that would allow other policies 
to be created for wider usage of this rule. For example inline only reference 
variable for the whole plan. Here is a patch to add the interface and include 
the default policy of the current state.

This issue takes the place of my concerns on #95 and #96.

Original issue reported on code.google.com by ecarm...@ucr.edu on 12 Apr 2013 at 7:57

Attachments:

GoogleCodeExporter commented 8 years ago
Issue 95 has been merged into this issue.

Original comment by ecarm...@ucr.edu on 20 Apr 2013 at 12:05

GoogleCodeExporter commented 8 years ago
Issue 96 has been merged into this issue.

Original comment by ecarm...@ucr.edu on 20 Apr 2013 at 12:05

GoogleCodeExporter commented 8 years ago
Yingyi,

Can you please review this patch and push it into the codebase?

Thanks.

Original comment by vinay...@gmail.com on 25 Apr 2013 at 8:36

GoogleCodeExporter commented 8 years ago
Found an alternate solution.

Original comment by ecarm...@ucr.edu on 5 Jun 2014 at 2:26