watson-developer-cloud / assistant-with-discovery-openwhisk

DEPRECATED: this repo is no longer actively maintained
Apache License 2.0
21 stars 43 forks source link

fix for discovery document upload timeout failure #48

Closed byrne-greg closed 6 years ago

byrne-greg commented 6 years ago

Problem Description

When a user executes npm run train, during the document upload phase, they will see the following error:

manualdocs/manual_0.json { Error: Callback function "uploadDocumentAsync" timed out.
    at Timeout.timeoutCallback [as _onTimeout] (/**/*/WatsonAssistantWithDiscoveryOpenWhisk/node_modules/async/dist/async.js:4916:26)
    at ontimeout (timers.js:458:11)
    at tryOnTimeout (timers.js:296:5)
    at Timer.listOnTimeout (timers.js:259:5) code: 'ETIMEDOUT' }

(see issue #43).

Solution

The default configuration file that was used when creating the Discovery collection was configured to use the older AlchemyLanguage enrichments. As of the migration, when anyone attempted to upload documents, they would receive the above error.

The default configuration file used during the train-discovery.js script was changed to use the NLU enrichment technology.

codecov-io commented 6 years ago

Codecov Report

Merging #48 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #48   +/-   ##
=======================================
  Coverage   96.22%   96.22%           
=======================================
  Files           2        2           
  Lines          53       53           
  Branches        5        5           
=======================================
  Hits           51       51           
  Misses          2        2

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update abc4239...3ac53c0. Read the comment docs.

germanattanasio commented 6 years ago

Thanks @byrne-greg 👍

watson-github-bot commented 6 years ago

:tada: This PR is included in version 1.0.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: