Skip to content

[ZEPPELIN-4792] Help needs to be updated in Cassandra interpreter#3780

Closed
alexott wants to merge 1 commit intoapache:masterfrom
alexott:ZEPPELIN-4792
Closed

[ZEPPELIN-4792] Help needs to be updated in Cassandra interpreter#3780
alexott wants to merge 1 commit intoapache:masterfrom
alexott:ZEPPELIN-4792

Conversation

@alexott
Copy link
Contributor

@alexott alexott commented May 31, 2020

What is this PR for?

When updating for the new DataStax Java driver, I forgot to update template for a HELP command. This PR fixes this problem, and simplifies maintenance by reading driver version from the resource file generated during compilation, so HELP command will provide information based on the actual driver version used for building interpreter.

What type of PR is it?

Bug Fix

What is the Jira issue?

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

How should this be tested?

@zjffdu
Copy link
Contributor

zjffdu commented Jun 1, 2020

LGTM

@asfgit asfgit closed this in def30ee Jun 1, 2020
asfgit pushed a commit that referenced this pull request Jun 1, 2020
### What is this PR for?

When updating  for the new DataStax Java driver, I forgot to update template for a `HELP` command. This PR fixes this problem, and simplifies maintenance by reading driver version from the resource file generated during compilation, so HELP command will provide information based on the actual driver version used for building interpreter.

### What type of PR is it?
Bug Fix

### What is the Jira issue?

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

### How should this be tested?

* https://travis-ci.org/github/alexott/zeppelin/builds/693123862 ()
* Tested manually

Author: Alex Ott <alexott@gmail.com>

Closes #3780 from alexott/ZEPPELIN-4792 and squashes the following commits:

f62bc5c [Alex Ott] [ZEPPELIN-4792] Help needs to be updated in Cassandra interpreter

(cherry picked from commit def30ee)
Signed-off-by: Alex Ott <alexott@apache.org>
@alexott alexott deleted the ZEPPELIN-4792 branch June 1, 2020 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments