BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153] #1170

babupe · 2016-07-12T22:13:57Z

What is this PR for?

Google BigQuery is a popular no-ops datawarehouse. This commit will enable Apache Zeppelin users to perform BI and Analytics on their datasets in BigQuery.

What type of PR is it?

Feature

Todos

Make bigquery interpreter appear in the interpreters section in the UI
Build SQL completion
Authorization of non-gcp

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1153

How should this be tested?

copy conf/zeppelin-site.xml.template to conf/zeppelin-site.xml
Add org.apache.zeppelin.bigquery.bigQueryInterpreter to property zeppelin.interpreters in zeppelin-site.xml
Start Zeppelin
Add BigQuery Interpreter with your project ID
Create new note with %bsql.sql and run your SQL against public datasets in bigquery.

Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

bzz · 2016-07-12T23:20:01Z

bigquery/src/main/java/org/apache/zeppelin/bigquery/bigQueryInterpreter.java

+    String projId = getProperty(PROJECT_ID);
+    long wTime = Long.parseLong(getProperty(WAIT_TIME));
+    Iterator<GetQueryResultsResponse> pages = run(sql, projId, wTime);
+    while (pages.hasNext()) {


Usually we want to handle errors here as well (I.e bad SQL syntax or network issues) and set paragraph error status to Error by returning InterpreterResult(Code.ERROR) in such cases.

I.e by looking at the code below, it seems that pages can be null in case of connectivity issues, resulting in the NPE instead.

bzz · 2016-07-12T23:47:09Z

Great contribution, thank you!

Couple of questions on top of few one already raised above:

will you be willing to add some tests as well? Looking at other interpreters impl like ZEPPELIN-1115: Python - interpreter for SQL over DataFrame #1164 one can see how to test I.e syntax errors and other conditions, etc
would it be hard or do you have plans to implement cancel and getProgress as part of this contribution? They are very usefull in case of big queries and improve user experience a lot on UI
in current code, you use static interpreter registration mechanism, which is now deprecated and all interpreters are in the process of migration. While we are here, in order not to create one more migration sub-tesk for later, could you please switch to interpreter-settings.json one instead? Its a dime change, here is an example of how it is done for another interpreter [ZEPPELIN-1045] Apply new mechanism to PythonInterpreter #1063
every interpreter and it's supported/unsupported features and cobfiguration options are usually documented under /docs/ which becomes part of the website. Would you care to document this one as well?
for every new library introduced by this module, which Jar will end up in the release convenience binary (all, with default scope) we need to have a record in https://github.com/apache/zeppelin/blob/master/zeppelin-distribution/src/bin_license/LICENSE specifying it's license

Feel free to ask here, if something is not clearly or in case you have any further questions!

AhyoungRyu · 2016-07-13T03:23:47Z

bigquery/src/main/java/org/apache/zeppelin/bigquery/bigQueryInterpreter.java

+ * </p>
+ * 
+ */
+


The above lines are better for bigquery.md as @bzz said in #1170 (comment). You can refer to many existed docs in here. It would be helpful to users who want to try BigQuery in Zeppelin.

babupe · 2016-07-13T17:53:44Z

Thanks! I will work on incorporating the feedback.

AhyoungRyu · 2016-07-15T03:20:51Z

bigquery/src/main/resources/interpreter-setting.json

+      }
+    }
+  }
+]


You can add the above property list to bigquery.md as a configuration table like this.

AhyoungRyu · 2016-07-15T03:32:41Z

@babupe Also you need to add

<li><a href="{{BASE_PATH}}/interpreter/bigquery.html">BigQuery</a></li>

this line to here. Then you can find bigquery.md in the dropdown menu on docs site.

babupe · 2016-07-15T04:53:46Z

Hi,

I have added unit tests, but the tests will work with a valid google project id on google compute, else it will fail. This is because the test has to run against the bigquery service which does not have a local build.
I have added a server side cancel operation. However I could not find a viable method to report progress. Will implement this if this becomes available as part of big query API in the future.
Implemented dynamic interpreter
Added BigQuery API license to the license file
Docs updated with BigQuery information.
Exceptions implemented at appropriate places.

Thanks!

AhyoungRyu · 2016-07-15T06:16:20Z

docs/_includes/themes/zeppelin/_navigation.html

                <li><a href="{{BASE_PATH}}/interpreter/scalding.html">Scalding</a></li>
                <li><a href="{{BASE_PATH}}/interpreter/shell.html">Shell</a></li>
                <li><a href="{{BASE_PATH}}/interpreter/spark.html">Spark</a></li>
+                <li><a href="{{BASE_PATH}}/interpreter/bigquery.html">BigQuery</a></li>


As you can see, the lists are in alphabetical order. You need to put bigquery.html between alluxio.html and cassandra.html.

bzz · 2016-07-15T08:22:52Z

@babupe thank you for taking care. @AhyoungRyu thank you for review.

Please let me do another pass on it and get back to you.

bzz · 2016-07-15T08:23:58Z

bigquery/src/main/java/org/apache/zeppelin/bigquery/bigQueryInterpreter.java

+ * Copyright 2016 Google Inc.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.


I would strongly advise to keep original text of the license here and in other files: See the NOTICE file distributed with this work for additional information regarding copyright ownership.

So copyright notice need to be moved to root NOTICE file and refer to ./bigquery/ module.

So, the google copyright should be completely removed here?
Where should I include the Google copyright then? Can you share an example?

bzz · 2016-07-15T09:00:27Z

@babupe great job, I have highlighted few issues inline in code above.

Two last things:

Test with mock instead of live service

I have added unit tests, but the tests will work with a valid google project id on google compute, else it will fail. This is because the test has to run against the bigquery service which does not have a local build.

To make tests part of CI we need them not to depend on external services. Of course everyting would work without such tests, but how do you think, would it be possible mock service in a way it is done in this library ITBigQueryTest.java ?
Readme file with technical documentation

I would really appreciate if, for the sake of maintainability, you could please add ./bigquery/README.md with some technical information on interpreter implementation, like the one we have for python i.e
- point to the sources of current BigQuery library that is being used
```
com.google.apis:google-api-services-bigquery:v2-rev265-1.21.0
```
  I had no luck locating it by searching online :\
- small rationale why that library is used or how it is compared to GoogleCloudPlatform/gcloud-java
- mention authentication mechanism used

babupe · 2016-07-15T21:21:41Z

https://github.com/GoogleCloudPlatform/gcloud-java/blob/master/TESTING.md#testing-code-that-uses-bigquery
There is no emulator for BigQuery today.

babupe · 2016-07-15T21:40:15Z

Please clarify regarding the LICENSE file.
Also, if you have any other feedback, would be happy to take a look.

AhyoungRyu · 2016-07-17T07:05:23Z

+1 for technical docs bigquery/README.md that @bzz suggested and it looks great that you added!
How about adding below one sentence in the end of bigquery.md so that users can know the existence of this file?

## Technical description

For in-depth technical details on current implementation please refer to [bigquery/README.md](https://github.com/apache/zeppelin/blob/master/bigquery/README.md).

bzz · 2016-07-18T02:00:24Z

bigquery/README.md

+The Interpreter opens a connection with the BigQuery Service using the supplied Google project ID and the compute environment variables.
+
+# Google BigQuery API Javadoc
+[API Javadocs](https://developers.google.com/resources/api-libraries/documentation/bigquery/v2/java/latest/)


This is JavaDoc for the artefact

<groupId>com.google.apis</groupId> <artifactId>google-api-services-bigquery</artifactId> <version>v2-rev265-1.21.0</version>

right?

AFAIK it's an open-source library, so would you be so kind to add a link here to it's source code please? This could help future maintainers to keep up with changes, etc.

Got it. These packages are licensed under Apache 2.0. I have asked around to see if the code is publicly available.

Any updates on this one?

babupe · 2016-07-26T19:50:11Z

Made the changes and pushed them. Hope its good now

bzz · 2016-07-28T06:31:13Z

Great stuff! Thank you for taking care.
Few minor updates in babupe#1 please review and merge it in.

Will merge this contribution to master and branch-0.6, if there is no further discussion.

Final updates before merging bigquery interpreter

babupe · 2016-07-28T17:46:14Z

Awesome! Merged this.

bzz · 2016-07-29T05:57:30Z

CI build failed due to networking issues

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-remote-resources-plugin:1.4:process (default) on project zeppelin: Error downloading resources archive. Could not transfer artifact org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT from/to apache-snapshots (https://repository.apache.org/snapshots/): Connect to repository.apache.org:443 [repository.apache.org/207.244.88.143] failed: Connection timed out
[ERROR] org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT

which are not related.

bzz · 2016-07-29T06:13:24Z

While running locally I got:

ERROR [2016-07-29 15:05:09,591] ({pool-2-thread-2} BigQueryInterpreter.java[open]:142) - Cannot open connection
java.io.IOException: The Application Default Credentials are not available. They are available if running in Google Compute Engine. Otherwise, the environment variable GOOGLE_APPLICATION_CREDENTIALS must be defined pointing to a file defining the credentials. See https://developers.google.com/accounts/docs/application-default-credentials for more information.
        at com.google.api.client.googleapis.auth.oauth2.DefaultCredentialProvider.getDefaultCredential(DefaultCredentialProvider.java:95)
        at com.google.api.client.googleapis.auth.oauth2.GoogleCredential.getApplicationDefault(GoogleCredential.java:213)
        at org.apache.zeppelin.bigquery.BigQueryInterpreter.createAuthorizedClient(BigQueryInterpreter.java:155)
        at org.apache.zeppelin.bigquery.BigQueryInterpreter.open(BigQueryInterpreter.java:138)
        at org.apache.zeppelin.interpreter.LazyOpenInterpreter.open(LazyOpenInterpreter.java:69)
        at org.apache.zeppelin.interpreter.LazyOpenInterpreter.interpret(LazyOpenInterpreter.java:93)
        at org.apache.zeppelin.interpreter.remote.RemoteInterpreterServer$InterpretJob.jobRun(RemoteInterpreterServer.java:383)
        at org.apache.zeppelin.scheduler.Job.run(Job.java:176)
        at org.apache.zeppelin.scheduler.FIFOScheduler$1.run(FIFOScheduler.java:139)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
        at java.util.concurrent.FutureTask.run(FutureTask.java:262)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
        at java.lang.Thread.run(Thread.java:744)
 INFO [2016-07-29 15:05:09,593] ({pool-2-thread-2} BigQueryInterpreter.java[close]:281) - Close bqsql connection!

@babupe do you think it would make sense to expose this though interpreter configuration properties, same as project id, so people could override it?

One more thing - in this case the status of the paragraph is ERROR, but there are no output, only logs. It would be great to be able to notify user about the reason of the failure

In any way, we need to document this properly for people to know that there are pre-requests, before using this interpreter.

I will submit the PR with docs soon, but it would be great if you could take a look at error propagation.

bzz · 2016-07-29T06:45:37Z

Added babupe#2

After configuring credentials I got strange behavior: query show tables results is ERROR paragraph status with

no output in GUI
and nothing in the logs

 INFO [2016-07-29 15:39:49,759] ({pool-2-thread-2} SchedulerFactory.java[jobStarted]:131) - Job remoteInterpretJob_1469774389758 started by scheduler org.apache.zeppelin.bigquery.BigQueryInterpreter745463803
 INFO [2016-07-29 15:39:49,759] ({pool-2-thread-2} BigQueryInterpreter.java[interpret]:288) - Run SQL command 'show tables'
 INFO [2016-07-29 15:39:50,531] ({pool-2-thread-2} SchedulerFactory.java[jobFinished]:137) - Job remoteInterpretJob_1469774389758 finished by scheduler org.apache.zeppelin.bigquery.BigQueryInterpreter745463803

Which is a bit frustrating experience, since it is ERROR but user does not know why. @babupe could you take a look as well please?

Other then above, everything 🚀

add docs for BigQuery auth outside of GCE

babupe · 2016-07-30T01:24:23Z

Sorry about the delay. Merged it now. Thank you so much for documenting this!
I am looking at surfacing the errors out to the interpreter. Will have an update on it.
Will also look at exposing creds for non GCE instances through configuration.

bzz · 2016-07-30T02:20:28Z

Thank you! Sounds awesome to me.

CI failed due to networking issue, again..

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-remote-resources-plugin:1.4:process (default) on project zeppelin: Error downloading resources archive. Could not transfer artifact org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT from/to codehaus-snapshots (https://nexus.codehaus.org/snapshots/): nexus.codehaus.org: Name or service not known
[ERROR] org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT

babupe · 2016-07-30T03:43:24Z

I have now pushed the change to capture and show bigquery exceptions with bad statements both in logs and interpreter. Will look into the credentials.

babupe · 2016-07-30T04:14:27Z

Do you think the external auth could be a blocker for the initial merge?
If not, lets table it for now and I will pick this up later?
It may be a little bit more involved than I thought.

bzz · 2016-07-30T14:26:08Z

Got it, makes perfect sense! Thank you for updating error handling and the plan to address external auth cfg later sounds good.

CI failure is still on Travis, not able to reach network resources:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-remote-resources-plugin:1.4:process (default) on project zeppelin: Error downloading resources archive. Could not transfer artifact org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT from/to apache-snapshots (https://repository.apache.org/snapshots/): Connect to repository.apache.org:443 [repository.apache.org/207.244.88.143] failed: Connection timed out
[ERROR] org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT

Looks great to me, merging to master and branch-0.6 if there is not further duscussion!

### What is this PR for? Google BigQuery is a popular no-ops datawarehouse. This commit will enable Apache Zeppelin users to perform BI and Analytics on their datasets in BigQuery. ### What type of PR is it? Feature ### Todos * Make bigquery interpreter appear in the interpreters section in the UI * Build SQL completion * Authorization of non-gcp ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1153 ### How should this be tested? copy conf/zeppelin-site.xml.template to conf/zeppelin-site.xml Add org.apache.zeppelin.bigquery.bigQueryInterpreter to property zeppelin.interpreters in zeppelin-site.xml Start Zeppelin Add BigQuery Interpreter with your project ID Create new note with %bsql.sql and run your SQL against public datasets in bigquery. ### Screenshots (if appropriate) ![screenshot from 2016-07-12 14 27 30](https://cloud.githubusercontent.com/assets/4242273/16785302/31b104e2-4842-11e6-87c0-b79763dd85c0.png) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Babu Prasad Elumalai <babupe@google.com> Author: babupe <babupe@google.com> Author: Alexander Bezzubov <bzz@apache.org> Closes #1170 from babupe/babupe-bigquery and squashes the following commits: ffed801 [Babu Prasad Elumalai] pushing BQ Exception to logs and Interpreter error output d3c2316 [babupe] Merge pull request #2 from bzz/babupe-add-auth-docs 64525b8 [Alexander Bezzubov] Fix typos in docs 03a777f [Alexander Bezzubov] add docs for BigQuery auth outside of GCE fcab6b7 [babupe] Merge pull request #1 from bzz/babupe-final 6a95333 [Alexander Bezzubov] Rename Apach2.0 license for google's code to adhere naming conventions 7d4f40b [Alexander Bezzubov] Add exidentaly removed licenses due to merge conflict 3be1912 [Babu Prasad Elumalai] New changes 41e076e [Babu Prasad Elumalai] Fixed formatting with readme file 97874a4 [Babu Prasad Elumalai] Pushing cropped screenshots 64affbb [babupe] Added cropped interpreter screenshot 4a1d29c [Babu Prasad Elumalai] Removed unnecessary dependencies in pom.xml e520b7b [Babu Prasad Elumalai] Exclude constants.json file for rat plugin since its static config file 69cb724 [Babu Prasad Elumalai] Fixed license header and added manual unit test documentation bbf26cc [Babu Prasad Elumalai] Added path and specific wording 4a3153f [Babu Prasad Elumalai] removed bad package from import d0c8e01 [Babu Prasad Elumalai] Added technical description to bigquery.md b6d181c [Babu Prasad Elumalai] Trying to add screenshot in README 569757f [Babu Prasad Elumalai] Incorporated feedback 764385c [Babu Prasad Elumalai] Interpreter modification, License, doc changes d85abd2 [Babu Prasad Elumalai] Modified code and license 17f6d89 [Babu Prasad Elumalai] ZEPPELIN-1153 comments committed 8fa647b [Babu Prasad Elumalai] BigQuery Interpreter for Apazhe Zeppelin 22e3487 [babupe] Update LICENSE e88b017 [babupe] Created a new license file d90e10f [babupe] Removed BigQuery from notice aa52553 [Babu Prasad Elumalai] Merge branch 'master' of https://github.com/apache/zeppelin ae096d2 [Babu Prasad Elumalai] License changes 20962d2 [Babu Prasad Elumalai] Pushing license changes 3d5f8e7 [Babu Prasad Elumalai] Modified license header 5a2e674 [Babu Prasad Elumalai] Added license info for Jackson library and added BQ API source 4db74c1 [Babu Prasad Elumalai] Adding license stuff 31c373f [Babu Prasad Elumalai] Fixed formatting with readme file 287744c [Babu Prasad Elumalai] Merge branch 'babupe-bigquery' of https://github.com/babupe/zeppelin into babupe-bigquery f318b20 [Babu Prasad Elumalai] Pushing cropped screenshots 17fd4e8 [babupe] Added cropped interpreter screenshot f872aa0 [Babu Prasad Elumalai] Removed unnecessary dependencies in pom.xml 5983e36 [Babu Prasad Elumalai] Exclude constants.json file for rat plugin since its static config file 11e88dc [Babu Prasad Elumalai] Replaced license header with formatting 4b82abd [Babu Prasad Elumalai] Fixed license header and added manual unit test documentation 87f5efe [Babu Prasad Elumalai] Added path and specific wording 6132d78 [Babu Prasad Elumalai] Fixing License and skipping failing tests 2254a49 [Babu Prasad Elumalai] removed bad package from import 73e3f6d [Babu Prasad Elumalai] Added technical description to bigquery.md 089820b [Babu Prasad Elumalai] Trying to add screenshot in README a00b48e [Babu Prasad Elumalai] Incorporated feedback 17846f1 [Babu Prasad Elumalai] Interpreter modification, License, doc changes 50c41fc [Babu Prasad Elumalai] Modified code and license 75d8ee6 [Babu Prasad Elumalai] ZEPPELIN-1153 comments committed 2a2bedc [Babu Prasad Elumalai] BigQuery Interpreter for Apazhe Zeppelin (cherry picked from commit 57c264d) Signed-off-by: Alexander Bezzubov <bzz@apache.org>

### What is this PR for? Google BigQuery is a popular no-ops datawarehouse. This commit will enable Apache Zeppelin users to perform BI and Analytics on their datasets in BigQuery. ### What type of PR is it? Feature ### Todos * Make bigquery interpreter appear in the interpreters section in the UI * Build SQL completion * Authorization of non-gcp ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1153 ### How should this be tested? copy conf/zeppelin-site.xml.template to conf/zeppelin-site.xml Add org.apache.zeppelin.bigquery.bigQueryInterpreter to property zeppelin.interpreters in zeppelin-site.xml Start Zeppelin Add BigQuery Interpreter with your project ID Create new note with %bsql.sql and run your SQL against public datasets in bigquery. ### Screenshots (if appropriate) ![screenshot from 2016-07-12 14 27 30](https://cloud.githubusercontent.com/assets/4242273/16785302/31b104e2-4842-11e6-87c0-b79763dd85c0.png) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Babu Prasad Elumalai <babupe@google.com> Author: babupe <babupe@google.com> Author: Alexander Bezzubov <bzz@apache.org> Closes apache#1170 from babupe/babupe-bigquery and squashes the following commits: ffed801 [Babu Prasad Elumalai] pushing BQ Exception to logs and Interpreter error output d3c2316 [babupe] Merge pull request apache#2 from bzz/babupe-add-auth-docs 64525b8 [Alexander Bezzubov] Fix typos in docs 03a777f [Alexander Bezzubov] add docs for BigQuery auth outside of GCE fcab6b7 [babupe] Merge pull request apache#1 from bzz/babupe-final 6a95333 [Alexander Bezzubov] Rename Apach2.0 license for google's code to adhere naming conventions 7d4f40b [Alexander Bezzubov] Add exidentaly removed licenses due to merge conflict 3be1912 [Babu Prasad Elumalai] New changes 41e076e [Babu Prasad Elumalai] Fixed formatting with readme file 97874a4 [Babu Prasad Elumalai] Pushing cropped screenshots 64affbb [babupe] Added cropped interpreter screenshot 4a1d29c [Babu Prasad Elumalai] Removed unnecessary dependencies in pom.xml e520b7b [Babu Prasad Elumalai] Exclude constants.json file for rat plugin since its static config file 69cb724 [Babu Prasad Elumalai] Fixed license header and added manual unit test documentation bbf26cc [Babu Prasad Elumalai] Added path and specific wording 4a3153f [Babu Prasad Elumalai] removed bad package from import d0c8e01 [Babu Prasad Elumalai] Added technical description to bigquery.md b6d181c [Babu Prasad Elumalai] Trying to add screenshot in README 569757f [Babu Prasad Elumalai] Incorporated feedback 764385c [Babu Prasad Elumalai] Interpreter modification, License, doc changes d85abd2 [Babu Prasad Elumalai] Modified code and license 17f6d89 [Babu Prasad Elumalai] ZEPPELIN-1153 comments committed 8fa647b [Babu Prasad Elumalai] BigQuery Interpreter for Apazhe Zeppelin 22e3487 [babupe] Update LICENSE e88b017 [babupe] Created a new license file d90e10f [babupe] Removed BigQuery from notice aa52553 [Babu Prasad Elumalai] Merge branch 'master' of https://github.com/apache/zeppelin ae096d2 [Babu Prasad Elumalai] License changes 20962d2 [Babu Prasad Elumalai] Pushing license changes 3d5f8e7 [Babu Prasad Elumalai] Modified license header 5a2e674 [Babu Prasad Elumalai] Added license info for Jackson library and added BQ API source 4db74c1 [Babu Prasad Elumalai] Adding license stuff 31c373f [Babu Prasad Elumalai] Fixed formatting with readme file 287744c [Babu Prasad Elumalai] Merge branch 'babupe-bigquery' of https://github.com/babupe/zeppelin into babupe-bigquery f318b20 [Babu Prasad Elumalai] Pushing cropped screenshots 17fd4e8 [babupe] Added cropped interpreter screenshot f872aa0 [Babu Prasad Elumalai] Removed unnecessary dependencies in pom.xml 5983e36 [Babu Prasad Elumalai] Exclude constants.json file for rat plugin since its static config file 11e88dc [Babu Prasad Elumalai] Replaced license header with formatting 4b82abd [Babu Prasad Elumalai] Fixed license header and added manual unit test documentation 87f5efe [Babu Prasad Elumalai] Added path and specific wording 6132d78 [Babu Prasad Elumalai] Fixing License and skipping failing tests 2254a49 [Babu Prasad Elumalai] removed bad package from import 73e3f6d [Babu Prasad Elumalai] Added technical description to bigquery.md 089820b [Babu Prasad Elumalai] Trying to add screenshot in README a00b48e [Babu Prasad Elumalai] Incorporated feedback 17846f1 [Babu Prasad Elumalai] Interpreter modification, License, doc changes 50c41fc [Babu Prasad Elumalai] Modified code and license 75d8ee6 [Babu Prasad Elumalai] ZEPPELIN-1153 comments committed 2a2bedc [Babu Prasad Elumalai] BigQuery Interpreter for Apazhe Zeppelin

BigQuery Interpreter for Apazhe Zeppelin

2a2bedc

bzz reviewed Jul 12, 2016
View reviewed changes

AhyoungRyu reviewed Jul 13, 2016
View reviewed changes

babupe-g added 2 commits July 15, 2016 03:00

ZEPPELIN-1153 comments committed

75d8ee6

Modified code and license

50c41fc

AhyoungRyu reviewed Jul 15, 2016
View reviewed changes

Interpreter modification, License, doc changes

17846f1

AhyoungRyu reviewed Jul 15, 2016
View reviewed changes

bzz reviewed Jul 15, 2016
View reviewed changes

babupe-g added 2 commits July 15, 2016 21:36

Incorporated feedback

a00b48e

Trying to add screenshot in README

089820b

Added technical description to bigquery.md

73e3f6d

bzz reviewed Jul 18, 2016
View reviewed changes

babupe-g added 8 commits July 26, 2016 19:22

Added path and specific wording

bbf26cc

Fixed license header and added manual unit test documentation

69cb724

Exclude constants.json file for rat plugin since its static config file

e520b7b

Removed unnecessary dependencies in pom.xml

4a1d29c

Added cropped interpreter screenshot

64affbb

Pushing cropped screenshots

97874a4

Fixed formatting with readme file

41e076e

New changes

3be1912

bzz added 2 commits July 28, 2016 15:23

Add exidentaly removed licenses due to merge conflict

7d4f40b

Rename Apach2.0 license for google's code to adhere naming conventions

6a95333

Merge pull request #1 from bzz/babupe-final

fcab6b7

Final updates before merging bigquery interpreter

add docs for BigQuery auth outside of GCE

03a777f

bzz and others added 2 commits July 30, 2016 08:59

Fix typos in docs

64525b8

Merge pull request #2 from bzz/babupe-add-auth-docs

d3c2316

add docs for BigQuery auth outside of GCE

pushing BQ Exception to logs and Interpreter error output

ffed801

asfgit closed this in 57c264d Jul 30, 2016

AhyoungRyu mentioned this pull request Feb 26, 2017

Groovy Interpreter for Apazhe Zeppelin [ZEPPELIN-2176] #2068

Closed

BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153] #1170

BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153] #1170

Uh oh!

Conversation

babupe commented Jul 12, 2016

What is this PR for?

What type of PR is it?

Todos

What is the Jira issue?

How should this be tested?

Screenshots (if appropriate)

Questions:

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bzz commented Jul 12, 2016

Uh oh!

AhyoungRyu Jul 13, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

babupe commented Jul 13, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

AhyoungRyu commented Jul 15, 2016

Uh oh!

babupe commented Jul 15, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bzz commented Jul 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bzz Jul 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bzz commented Jul 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

babupe commented Jul 15, 2016

Uh oh!

babupe commented Jul 15, 2016

Uh oh!

AhyoungRyu commented Jul 17, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

babupe commented Jul 26, 2016

Uh oh!

bzz commented Jul 28, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

babupe commented Jul 28, 2016

Uh oh!

bzz commented Jul 29, 2016

Uh oh!

bzz commented Jul 29, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bzz commented Jul 29, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

babupe commented Jul 30, 2016

Uh oh!

bzz commented Jul 30, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

babupe commented Jul 30, 2016

Uh oh!

babupe commented Jul 30, 2016

AhyoungRyu Jul 13, 2016 •

edited

Loading

bzz commented Jul 15, 2016 •

edited

Loading

bzz Jul 15, 2016 •

edited

Loading

bzz commented Jul 15, 2016 •

edited

Loading

bzz commented Jul 28, 2016 •

edited

Loading

bzz commented Jul 29, 2016 •

edited

Loading

bzz commented Jul 29, 2016 •

edited

Loading

bzz commented Jul 30, 2016 •

edited

Loading

bzz commented Jul 30, 2016 •

edited

Loading