[ 
https://issues.apache.org/jira/browse/TEZ-4384?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17491906#comment-17491906
 ] 

Laszlo Attila Toth commented on TEZ-4384:
-----------------------------------------

[~abstractdog] It's not related to Java 17, but I found it during working on 
the Java 17 compatibility. The code tried to alter System.getenv directly, 
which doesn't work in Java 17. As I tried to figure out the intention of the 
code I noticed that the class has only one client, only the test class uses it.

Strangely not long after the introduction it's usage is dropped in TEZ-1837, 
but the actual class remained in the code base.

> Remove unused EnvironmentUpdateUtils remaining from TEZ-1837
> ------------------------------------------------------------
>
>                 Key: TEZ-4384
>                 URL: https://issues.apache.org/jira/browse/TEZ-4384
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.10.1
>            Reporter: Laszlo Attila Toth
>            Assignee: Laszlo Attila Toth
>            Priority: Major
>             Fix For: 0.10.2
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> |EnvironmentUpdateUtils's usage was removed in TEZ-1837 in 2015, while its 
> import and the class with the tests were kept.
> This is a dead code as of now.
>  
> It's safe to delete those.|



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to