[GitHub] drill issue #1182: DRILL-6287: apache-release profile should be disabled by ...

2018-04-02 Thread parthchandra
Github user parthchandra commented on the issue:

https://github.com/apache/drill/pull/1182
  
Sorry, Maven not being a strong point, I didn't understand initially what I 
was looking at.

+1



---


[GitHub] drill issue #1182: DRILL-6287: apache-release profile should be disabled by ...

2018-04-02 Thread vrozov
Github user vrozov commented on the issue:

https://github.com/apache/drill/pull/1182
  
There are two issues with enabling `apache-release` by default:
-  it triggers creating source `apache-drill-...-src.tar.gz` and 
`apache-drill-...-src.zip` archives.
- maven build for any sub-module fails.

The change disables activation of the `apache-release` profile based on JDK 
version and requires explicit activation during the Apache release process.

JDK 1.7 is not supported. See DRILL-1491 and #1143.


---


[GitHub] drill issue #1182: DRILL-6287: apache-release profile should be disabled by ...

2018-04-02 Thread parthchandra
Github user parthchandra commented on the issue:

https://github.com/apache/drill/pull/1182
  
I don't understand why the apache-release be disabled by default. And I 
don't see how this change achieves that anyway.

Also, moving -Xdoclint:none to all profiles implies we are no longer 
supporting development using JDK7 ? I'm OK with that, but not sure if we 
concluded that at the time of the 1.13 release.

If that's what we want to do, I'm fine with this change.



---


[GitHub] drill issue #1182: DRILL-6287: apache-release profile should be disabled by ...

2018-03-30 Thread vrozov
Github user vrozov commented on the issue:

https://github.com/apache/drill/pull/1182
  
@parthchandra Please review


---