Neuw84 commented on PR #5463:
URL: https://github.com/apache/hudi/pull/5463#issuecomment-1121545791

   Hi @pratyakshsharma, 
   
   I could try to add a test for initialising the Spark Context. However, the 
change is very simple and looking for the tests I did not see any that test 
that tests the other constructors. 
   
   The only thing is that now, if no Spark master is not defined it will 
inherit or the application will fail as no Spark context is defined. Previously 
it would run with ```local[2]``` as Spark Master.  
   
   What do you have in mind? (looking for some guidance).


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to