Skip to content

Fixed pyspark indentation issue - take 2#63

Closed
whisperstream wants to merge 1 commit intoapache:masterfrom
whisperstream:pyspark-indentation-fix-2
Closed

Fixed pyspark indentation issue - take 2#63
whisperstream wants to merge 1 commit intoapache:masterfrom
whisperstream:pyspark-indentation-fix-2

Conversation

@whisperstream
Copy link
Copy Markdown
Contributor

see https://issues.apache.org/jira/browse/ZEPPELIN-34, should allow:

%pyspark
if True:
    print "one"
else:
    print "two"

def test_func(text):
    for i in range(1, 10):
        print text + '-' + str(i)
test_func('fixed')

see https://issues.apache.org/jira/browse/ZEPPELIN-34, should allow:

%pyspark
if True:
    print "one"
else:
    print "two"
    
def test_func(text):
    for i in range(1, 10):
        print text + '-' + str(i)
test_func('fixed')
@Leemoonsoo
Copy link
Copy Markdown
Member

Tested and looks good to me.
Merging it if there's no more discussions.

@whisperstream Thanks for the great fix.

Leemoonsoo pushed a commit to Leemoonsoo/zeppelin that referenced this pull request May 9, 2015
see https://issues.apache.org/jira/browse/ZEPPELIN-34, should allow:

Author: whisperstream <[email protected]>

Closes apache#63 from whisperstream/pyspark-indentation-fix-2 and squashes the following commits:

ec4398d [whisperstream] Fixed pyspark indentation issue
@asfgit asfgit closed this in 956e3f7 May 9, 2015
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