search

lagom / online-auction-java

Other
129 stars 109 forks source link

Add readside processor to user #131

Closed lakhina closed 7 years ago

lakhina commented 7 years ago

Phase one of #120 is complete. Now getUsers() provide you 10 users in nav list in with alphabatically least uuid

TimMoore commented 7 years ago

@lakhina the latest build has a new error:

 Caused by: java.util.concurrent.ExecutionException: java.util.concurrent.ExecutionException: com.datastax.driver.core.exceptions.SyntaxError: line 1:107 mismatched input '(' expecting ')' (..., email text PRIMARY KEY [(]createdAt...)

This indicates a syntax error in your CQL statement. Have a look at the stack trace to find where.

The error "Couldn't register the processor on the testkit" is a generic wrapper exception that could have a variety of different causes. It's important to always read the nested exceptions to find the root cause of the error.

octonato commented 7 years ago

@lakhina I have been trying to trace why the build is failing.

Most of the times it fails with a timeout on the actor system when it tries to create the tables. I could find the cause of that yet. I will keep searching.

However, from time to time it succeeds and when it happens we get other errors on the UserRepositoryTest (see my comments there)

lakhina commented 7 years ago

@rcavalcanti Thanks for the help. I have made the suggested changes but still the same error is persisting.

octonato commented 7 years ago

@lakhina the fix I suggested won't fix the build problem (see previous comment).

What I observed is that when it happens to pass the stage of table creation, it does fail because of errors in the test itself.

Can you push your last changes? Thanks

lakhina commented 7 years ago

@rcavalcanti I have committed the changes. Please have a look.

TimMoore commented 7 years ago

With the latest changes from @rcavalcanti, it looks like the build failure is now a legitimate test failure (should be reproducible locally) rather than a problem with the framework.