Need to add new session settings for Drill-MapRDB plugin

[Smidth Panchamia <sp...@maprtech.com>]

Hello,

I wanted to add new session variables that would be recognized only by the
drill-MapRDB JSON plugin.

Is there a way to add these only through the plugin code(which lies outside
of drill codebase)?

If not, here is the change I have in the drill code:
https://reviews.apache.org/r/44043/
Can someone please review it?

Thanks,
Smidth

Re: Need to add new session settings for Drill-MapRDB plugin

[Jacques Nadeau <ja...@dremio.com>]

We should use the SELECT WITH OPTIONS framework for these options. We're in
the process of deprecating session level options for changing table reading
options. (Otherwise views can fail for one person and not another, results
change and you can combine two different options for different tables in
the same query.)

This also has the nice side effect of not needing to add options for code
outside the core codebase.

--
Jacques Nadeau
CTO and Co-Founder, Dremio

On Thu, Feb 25, 2016 at 3:45 PM, Smidth Panchamia <sp...@maprtech.com>
wrote:

> Hello,
>
> I wanted to add new session variables that would be recognized only by the
> drill-MapRDB JSON plugin.
>
> Is there a way to add these only through the plugin code(which lies outside
> of drill codebase)?
>
> If not, here is the change I have in the drill code:
> https://reviews.apache.org/r/44043/
> Can someone please review it?
>
> Thanks,
> Smidth
>