[ 
https://issues.apache.org/jira/browse/SPARK-16197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bryan Cutler updated SPARK-16197:
---------------------------------
    Description: 
Cleanup of Status API example to use SparkSession and be more consistent with 
other examples.

Changing this JIRA to just cleanup the example, the other changes are just code 
style which is not really followed in other areas of PySpark
-also noticed that Status defines two empty classes without using 'pass' and 
two methods that do not return 'None' explicitly if requested info can not be 
fetched.  These issues do not cause any errors, but it is good practice to use 
'pass' on an empty class definitions and return 'None' for a function if the 
caller is expecting a return value.-

  was:Cleanup of Status API to use SparkSession and also noticed that Status 
defines two empty classes without using 'pass' and two methods that do not 
return 'None' explicitly if requested info can not be fetched.  These issues do 
not cause any errors, but it is good practice to use 'pass' on an empty class 
definitions and return 'None' for a function if the caller is expecting a 
return value.


> Cleanup PySpark status api and example
> --------------------------------------
>
>                 Key: SPARK-16197
>                 URL: https://issues.apache.org/jira/browse/SPARK-16197
>             Project: Spark
>          Issue Type: Improvement
>          Components: PySpark
>            Reporter: Bryan Cutler
>            Priority: Trivial
>
> Cleanup of Status API example to use SparkSession and be more consistent with 
> other examples.
> Changing this JIRA to just cleanup the example, the other changes are just 
> code style which is not really followed in other areas of PySpark
> -also noticed that Status defines two empty classes without using 'pass' and 
> two methods that do not return 'None' explicitly if requested info can not be 
> fetched.  These issues do not cause any errors, but it is good practice to 
> use 'pass' on an empty class definitions and return 'None' for a function if 
> the caller is expecting a return value.-



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to