GitHub user sanjaydasgupta opened a pull request:

    https://github.com/apache/zeppelin/pull/3068

    [ZEPPELIN-3438] Passing Z variables to BigQuery 

    ### What is this PR for?
    This PR enables the interpolation of ZeppelinContext objects into the 
paragraph text of BigQuery cells. It also introduces a new interpreter-level 
configuration parameter named `zeppelin.bigquery.interpolation`. This new 
parameter is `false` by default, and must be set to `true` to enable object 
interpolation. The default value of `false` guarantees backward compatibility 
for users who are not aware of the new feature.
    
    The implementation takes the same approach that was followed in PR 
[2898](https://github.com/apache/zeppelin/pull/2898) and 
[2903](https://github.com/apache/zeppelin/pull/2903).
    
    ### What type of PR is it?
    [ Feature ]
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-3438
    
    ### How should this be tested?
    Two new unit test classes have been added: 
[BigQueryInterpolationDisabled.java](https://github.com/sanjaydasgupta/zeppelin/blob/572c2aaa7a96eca98cdef4c77044dfc7a359f89f/bigquery/src/test/java/org/apache/zeppelin/bigquery/BigQueryInterpolationDisabled.java),
 and 
[BigQueryInterpolationEnabled.java](https://github.com/sanjaydasgupta/zeppelin/blob/572c2aaa7a96eca98cdef4c77044dfc7a359f89f/bigquery/src/test/java/org/apache/zeppelin/bigquery/BigQueryInterpolationEnabled.java).
    
    Also, the code in this PR merely causes the BigQuery interpreter to 
"opt-in" to the implementation of z-variable interpolation already existing in 
the Interpreter base class - described in 
[PR-2898](https://github.com/apache/zeppelin/pull/2898). The unit tests 
necessary for testing z-variable interpolation in detail are already present in 
PR-2898
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update?  No
    * Is there breaking changes for older versions?  No
    * Does this needs documentation?  Yes, documentation changes provided in 
[bigquery.md](https://github.com/sanjaydasgupta/zeppelin/blob/572c2aaa7a96eca98cdef4c77044dfc7a359f89f/docs/interpreter/bigquery.md)


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/sanjaydasgupta/zeppelin 
z3438-zeppelin-variables-for-bigquery

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/3068.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3068
    
----
commit 572c2aaa7a96eca98cdef4c77044dfc7a359f89f
Author: Sanjay Dasgupta <sanjay.dasgupta@...>
Date:   2018-07-13T04:59:36Z

    z3438-zeppelin-variables-for-bigquery

----


---

Reply via email to