You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "huciaa (via GitHub)" <gi...@apache.org> on 2023/10/31 20:45:25 UTC

[PR] Fix the variable name in the docs - testing_pyspark.ipynb [spark]

huciaa opened a new pull request, #43610:
URL: https://github.com/apache/spark/pull/43610

   ### What changes were proposed in this pull request?
   
   I'm changing a variable name in one of the cells in the documentation regarding pyspark testing.
   
   
   ### Why are the changes needed?
   
   Currently the code won't run. This simple fix will make the code in the documenatatnion usable.
   
   
   ### Does this PR introduce _any_ user-facing change?
   
   No
   
   
   ### How was this patch tested?
   I tested this code loccally with pytest and pyspark 3.5.0 
   ![image](https://github.com/apache/spark/assets/51126637/3a29a495-38d3-492c-b192-3e26af6fc72c)
   
   
   
   ### Was this patch authored or co-authored using generative AI tooling?
   No
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Re: [PR] [MINOR][DOCS]Fix the variable name in the docs - testing_pyspark.ipynb [spark]

Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon commented on PR #43610:
URL: https://github.com/apache/spark/pull/43610#issuecomment-1788177955

   Merged to master.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Re: [PR] [MINOR][DOCS]Fix the variable name in the docs - testing_pyspark.ipynb [spark]

Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon closed pull request #43610: [MINOR][DOCS]Fix the variable name in the docs - testing_pyspark.ipynb
URL: https://github.com/apache/spark/pull/43610


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Re: [PR] [MINOR][DOCS]Fix the variable name in the docs - testing_pyspark.ipynb [spark]

Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon commented on PR #43610:
URL: https://github.com/apache/spark/pull/43610#issuecomment-1788126781

   cc @asl3 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Re: [PR] [MINOR][DOCS]Fix the variable name in the docs - testing_pyspark.ipynb [spark]

Posted by "asl3 (via GitHub)" <gi...@apache.org>.
asl3 commented on PR #43610:
URL: https://github.com/apache/spark/pull/43610#issuecomment-1788153546

   thanks for the fix! LGTM 
   @huciaa 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org