hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Billie Rinaldi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-7113) Clean up packaging and dependencies for yarn-native-services
Date Tue, 29 Aug 2017 01:42:00 GMT

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

Billie Rinaldi commented on YARN-7113:
--------------------------------------

Thanks for taking a look, [~jianhe]!

bq. I tried some testing, and looks like the apiserver is still not started, because "src/main/resources/webapps/services-rest-api/app"
is not named properly, and it throws Exception when trying to find the file
I'm not sure what you mean. Patch 01 renames src/main/resources/webapps/services-rest-api/app
to src/main/resources/webapps/api-server/app, and I was able to start apiserver and use it
to launch services. What error did you get when you tried to run it?

bq. some dependency may also be not needed
I ran dependency analysis again and discovered that zookeeper is not needed any longer (since
we are using curator instead). However, we still seem to be using commons compress, configuration2,
and logging:
{noformat}
import org.apache.commons.compress.archivers.tar.TarArchiveEntry;
import org.apache.commons.compress.archivers.tar.TarArchiveOutputStream;
import org.apache.commons.configuration2.SubsetConfiguration;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
{noformat}

It looks like only YarnRegistryViewForProviders and TestYarnNativeServices are using commons-logging
instead of slf4j, so I will switch those two over and remove the commons-logging dependency.

> Clean up packaging and dependencies for yarn-native-services
> ------------------------------------------------------------
>
>                 Key: YARN-7113
>                 URL: https://issues.apache.org/jira/browse/YARN-7113
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: yarn-native-services
>            Reporter: Billie Rinaldi
>            Assignee: Billie Rinaldi
>             Fix For: yarn-native-services
>
>         Attachments: diff.patch, YARN-7113-yarn-native-services.01.patch
>
>
> Since the yarn native services code has been greatly simplified, I think we no longer
need a separate lib directory for services. A dependency cleanup is needed to address unused
declared dependencies and used undeclared dependencies in the new modules. We should also
address NOTICE changes needed for the 3 new dependencies that are being added, jcommander,
snakeyaml, and swagger-annotations.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org


Mime
View raw message