Github user squito commented on the pull request:
https://github.com/apache/spark/pull/5833#issuecomment-98398306
yeah, I could go either way on the test -- I guess the point was mostly to
make sure it still worked with DBs, and also to have some example code somebody
could try if they did want to mess with oracle integration. it was just a
thought, I don't feel strongly about it.
btw, thanks for adding the comment. I had meant something much more brief,
just like, "this is needed for oracle, see SPARK-7299" but what you have is
good :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
