tdg5 opened a new pull request #34979:
URL: https://github.com/apache/spark/pull/34979


   ### What changes were proposed in this pull request?
   These are a couple of changes I found worthwhile while running docker 
integration tests for https://github.com/apache/spark/pull/34745.
   
   The doc changes are minor fixes to add the missing repository to the 
suggested command to run the docker integration tests.
   
   The library change relates to this comment: 
https://github.com/apache/spark/pull/34745#discussion_r773084417; I don't know 
if my testing was thorough enough, but I found that the referenced dependency 
was not needed and could be removed.
   
   ### Why are the changes needed?
   The doc changes are needed because the current suggested command is missing 
the docker repository so will not work in its current state. 
   
   The library removal is needed because the build fails when the `ojdbc6` 
package is included because the referenced library is no longer available in 
jfrog: https://camunda.jfrog.io/ui/native/public/com/oracle/ojdbc6
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   
   ### How was this patch tested?
   Attempted to run all the docker integration tests locally by running
   ```
   ENABLE_DOCKER_INTEGRATION_TESTS=1 ./build/sbt -Pdocker-integration-tests 
"testOnly *IntegrationSuite"
   ```
   Assuming this didn't miss any tests, all the tests passed.


-- 
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]



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

Reply via email to