Skip to content

Conversation

@Leemoonsoo
Copy link
Contributor

issue described here.
ZEPPELIN-121

@Leemoonsoo
Copy link
Contributor Author

fix and unittest included. ready to be merged.

@anthonycorbacho
Copy link
Contributor

I reviewed the code and its look good to me.

For me it is ready to be merged!

@Leemoonsoo
Copy link
Contributor Author

Thanks for the review!

Leemoonsoo added a commit that referenced this pull request Mar 27, 2014
ZEPPELIN-121 Fix handling escpae char in erb evaluator
@Leemoonsoo Leemoonsoo merged commit bfe7ac6 into master Mar 27, 2014
@Leemoonsoo Leemoonsoo deleted the fix_escape_char_parsing branch March 27, 2014 03:27
epahomov pushed a commit to epahomov/zeppelin that referenced this pull request Jul 23, 2016
…from 1.1 to 1.3

I updated the version of spark used by default throughout the dependency listing to be 1.3.0 (to match the version used in the 1.3 profile).

Author: Lee moon soo <moon@apache.org>
Author: Ilya Ganelin <ilya.ganelin@capitalone.com>

This patch had conflicts when merged, resolved by
Committer: Lee moon soo <moon@apache.org>

Closes ZEPL#71 from ilganeli/ZEPPELIN-23 and squashes the following commits:

d1d642e [Ilya Ganelin] Merge pull request ZEPL#1 from Leemoonsoo/ZEPPELIN-23
0108781 [Lee moon soo] update isDataFrameSupported()
ac4e8bc [Lee moon soo] check spark version as a numerical number in tests
405da99 [Lee moon soo] Fix indentation
28dd7bf [Lee moon soo] Make unittest pass with spark-1.3 profile
cf2ed0c [Lee moon soo] Activate spark-1.3 profile by default. Set spark 1.3.1 as a default version of spark-1.3 profile
13bf6c1 [Lee moon soo] remove z.load() error for spark 1.2, spark 1.3
1402df2 [Ilya Ganelin] Updated default spark version to 1.3
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.

3 participants