+1

tested under win (JDK8) and mac (JDK8)

 -`maven clean verify`

 - examples tests

 - files


Remark pom.xml scm tag <tag>0.6.0</tag> is incorrect. It should be `v0.6.1` - please don't miss it next time.


Paul. I have created additional ticket, https://issues.apache.org/jira/browse/NLPCRAFT-77, so we with your help we have 4 new tickers which can significantly improve and simplify test procedure for next release.

About current release, my opinion that these tests issues (for win) are not critical and can be fixed in next release.

Thank you,


Sergey


09.06.2020 16:50, Paul King пишет:
I added the log for the first failing test to:

https://issues.apache.org/jira/browse/NLPCRAFT-76

The other tests look similar but let me know if you want them also.

Cheers, Paul.


On Tue, Jun 9, 2020 at 9:44 PM Sergey Kamov <skhdlem...@gmail.com> wrote:

Hi Paul
Thank you for your remarks.

There are 3 kinds of problem

1) Some deserialization warnings
  >> Failed to unmarshal object with optimized marshaller
..
I have created ticket - https://issues.apache.org/jira/browse/NLPCRAFT-75
These warnings are not critical for our release I guess, but we have to
fix it of course (warnings are always confused)

2. Problems with test, which ask data from some external weather data
provider (weather service test)
   >> 'ask' request 'tomorrow yesterday' answered
  >> unsuccessfully with error: Unexpected response [code=420,
text=Unable to serve response within time limit, please enhance your calm.]

Sometimes this free external service doesn't work well.
https://issues.apache.org/jira/browse/NLPCRAFT-74 created.

I guess we can't depend on such external service in our suite, so these
tests should be refactored or excluded from suite.
It is not our release bug too, but better to fix it

3. Third error - most unexpected.
  >>!!! Failed 3 from 3 tests. See errors list above. !!!

org.apache.nlpcraft.probe.mgrs.nlp.enrichers.model.NCEnricherNestedModelSpec.test(NCEnricherNestedModelSpec.scala:41)

I have tested under win10, mac and linux - everything passed. I can't
reproduce it.

Could you provide, when you have time, whole log for 'mvn clean verify'
via new Jira issue or email?

Thank you

Sergey

I am +0 (on the basis that you aren't expecting the failures under
Windows)
but I'm happy to change to +1 if I am doing something wrong or if you
believe it is something in my environment. I had no problems under Linux.

I checked:

- "incubating" in name
- DISCLAIMER exists
- LICENSE and NOTICE look good
- no unexpected binary files
- signature/hashes okay
- rat check fails with no profile supplied but passes when given
-Prelease
(I assume that is okay?)
    (I'd suggest removing
<numUnapprovedLicenses>1000000</numUnapprovedLicenses> and adding exclude
for .yaml files)
- "mvn clean package verify" passes on Linux under JDK11
    (No appearance of Stanford or other GPL files with normal default
profile)
- "mvn clean verify" fails under Windows JDK8 and 11 (see below)

Windows Surefire Failure:

[ERROR] Errors:
[ERROR]
NCEnricherNestedModelSpec.test:41->NCEnricherBaseSpec.runBatch:153 »
TestFailed
[ERROR] Tests run: 29, Failures: 0, Errors: 1, Skipped: 1
!!! Failed 3 from 3 tests. See errors list above. !!!
      at

org.apache.nlpcraft.probe.mgrs.nlp.enrichers.model.NCEnricherNestedModelSpec.test(NCEnricherNestedModelSpec.scala:41)
There are errors like below throughout the console logs (possibly
relevant
or perhaps a red herring):

[INFO] Caused by: java.lang.ClassNotFoundException: Optimized stream
class
checksum mismatch (is same version of marshalled class present on all
nodes?) [expected=32321, actual=26723, cls=class
scala.collection.immutable.Vector]
...
[INFO]  at

org.apache.ignite.internal.binary.BinaryFieldAccessor.read(BinaryFieldAccessor.java:188)
~[apache-nlpcraft-incubating-0.6.1-all-deps.jar:0.6.1]
Jun.-09 18:20:58 [INFO ] 'ask' request 'tomorrow yesterday' answered
unsuccessfully with error: Unexpected response [code=420, text=Unable to
serve response within time limit, please enhance your calm.]
...
[INFO] Caused by: java.io.IOException: Failed to deserialize object
[typeName=scala.collection.mutable.HashSet]
...
[INFO] Caused by: org.apache.ignite.IgniteCheckedException: Failed to
deserialize object with given class loader:
[clsLdr=jdk.internal.loader.ClassLoaders$AppClassLoader@30946e09,
err=Failed to deserialize object
[typeName=scala.collection.mutable.HashSet]]
...
[INFO] Caused by: org.apache.ignite.binary.BinaryObjectException: Failed
to unmarshal object with optimized marshaller
...
[INFO] Caused by: org.apache.ignite.binary.BinaryObjectException: Failed
to deserialize object
[typeName=org.apache.nlpcraft.common.nlp.NCNlpSentenceToken]
Let me know if you need more complete stack traces or need a Jira issue
created. While it seems like numerous errors, I expect they are all
related.
Cheers, Paul.


On Tue, Jun 9, 2020 at 5:21 AM Aaron Radzinski <
aradzin...@datalingvo.com>
wrote:

NLPCraft-ers,
This is the vote for release of NLPCraft v0.6.1. This release includes a
number of bug fixes, including the fix for pulling GPL-libraries during
the
build. All 3-rd party dependencies now live in their own non-default
maven
profile.

Quick links:
1. Release location:

https://dist.apache.org/repos/dist/dev/incubator/nlpcraft/nlpcraft/0.6.1/
2. GitHub v0.6.0 tag:
https://github.com/apache/incubator-nlpcraft/tree/v0.6.1
3. JIRA issues closed:
https://issues.apache.org/jira/projects/NLPCRAFT/versions/12347775
4. KEYS file:
https://dist.apache.org/repos/dist/release/incubator/nlpcraft/KEYS

The vote is formal, see voting guidelines here:
https://www.apache.org/foundation/voting.html
+1 - to accept Apache NLPCraft 0.6.1
0 - don't care either way
-1 - DO NOT accept Apache NLPCraft 0.6.1 (explain why)

See notes on how to verify release here:
https://www.apache.org/info/verification.html

The vote will be held for 72 hours and will end on June 11, 2020 at
midnight MSK.

Thank you!
--
Aaron Radzinski

Reply via email to