[GitHub] [spark] HyukjinKwon commented on pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-05 Thread GitBox
HyukjinKwon commented on pull request #29006: URL: https://github.com/apache/spark/pull/29006#issuecomment-654037436 Nope, I don't think we should remove it now - it should be best to remove it when we completely remove `test-hive1.2`. Having `[test-hive1.2]` is only a dev purpose so shoul

[GitHub] [spark] HyukjinKwon commented on pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-05 Thread GitBox
HyukjinKwon commented on pull request #29006: URL: https://github.com/apache/spark/pull/29006#issuecomment-654001376 I think you can give a shot to rather ban the profile combination with an explicit error message, for example, by using https://maven.apache.org/enforcer/enforcer-rules/bann