pjfanning commented on PR #231:
URL: https://github.com/apache/tez/pull/231#issuecomment-1191347179
@ayushtkn I've never used that plugin before. I checked its docs and the
docs don't match what Hadoop has. In
https://github.com/skuzzle/restrict-imports-enforcer-rule - the XML element is
pjfanning commented on PR #231:
URL: https://github.com/apache/tez/pull/231#issuecomment-1191353059
@ayushtkn looks like Hadoop is wrong according to the plugin's release notes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
pjfanning commented on PR #231:
URL: https://github.com/apache/tez/pull/231#issuecomment-1191370908
@ayushtkn I think I have the enforcement rule working now - the
inherited=false flag seems to have stopped the rule being enforced in
sub-modules. The 'restrictImports' XML tag seems to work
tez-yetus commented on PR #231:
URL: https://github.com/apache/tez/pull/231#issuecomment-1191515909
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 6s | Docker mode
tez-yetus commented on PR #231:
URL: https://github.com/apache/tez/pull/231#issuecomment-1191517382
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 17s | Docker mode
tez-yetus commented on PR #231:
URL: https://github.com/apache/tez/pull/231#issuecomment-1191507548
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 17m 33s | Docker mode