Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@dongjoon-hyun @gatorsmile any further thoughts on this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
I've made some changes to the code; the hardcoded `false` value for
`cascade` in `JdbcDialects` is now replaced by a default value of
`isCascadingTruncateTable`. This is also the case for each
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@gatorsmile would be great to hear why you doubt the value of the feature
:). I know that for us it would be extremely valuable (at the moment we have to
do an extra step in our data pipeline b
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@gatorsmile could you explain why you have doubts about the feature? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr.
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20057
Thank you for your contribution! I am doubting the value of this feature.
If you are interested in the JDBC-related work, could you take
https://issues.apache.org/jira/browse/SPARK-22731?
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
I think the default for all dialects should be false, regardless of whether
the cascade feature is even supported. And for those for which it is supported,
it should default to false.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@dongjoon-hyun this is the functionality we discussed in PR for
SPARK-22729, would be great to hear your opinion on this :)
---
---
101 - 108 of 108 matches
Mail list logo
