Adding <document-api> to basic-search-java causes error when running mvn package about vespa HOT 8 CLOSED

Welcome back Trygve :-)
@gjoranv this is why we should do the "container-dev" -> "container" thing I mentioned.

I guess it won't happen that soon - should we just add vespaclient-container-plugin dep to basic-search to avoid this, or add it to the documentation, or ...?

from vespa.

@bratseth This is the test classpath, not provided/runtime. I'll add vespaclient-container-plugin as a dependency in application for now, and move it to container-test when that is ready.

from vespa.

Has this issue been fixed or not? I encountered the same problem and fixed it by manually adding the dependency. Not sure if other people know how to do or not ...

from vespa.

This is work in progress. New maven artifacts for 3rd party developers will be available soon.

from vespa.

One of the sample applications fails with a similar exception (see SO question at https://stackoverflow.com/q/46772244/1997056). Could it be related to this problem?

from vespa.

@Shastick, your problem is not related to this issue. I'll test the sample app and get back to you on SO.

from vespa.

From version 6.158.42, you can remove all dependencies with groupId com.yahoo.vespa from your project's pom.xml file, and instead insert the following:

        <dependency>
            <groupId>com.yahoo.vespa</groupId>
            <artifactId>container</artifactId>
            <version>${vespa_version}</version>
            <scope>provided</scope>
        </dependency>
        <dependency>
            <groupId>com.yahoo.vespa</groupId>
            <artifactId>container-test</artifactId>
            <version>${vespa_version}</version>
            <scope>test</scope>
        </dependency>

Please make sure that you have the vespa_versionor similar property defined in your pom.xml and update it to 6.158.42 or newer.

@trygve or @huihuang01 , could you please try to apply the proposed changes to your pom.xml?Thanks!

from vespa.

All sample apps have been updated to use 'container' and 'container-test' since vespa-engine/sample-apps#28

from vespa.