Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pepys database connection issue #5122

Closed
IanMayo opened this issue Jul 13, 2021 · 3 comments · Fixed by #5130
Closed

Pepys database connection issue #5122

IanMayo opened this issue Jul 13, 2021 · 3 comments · Fixed by #5130
Labels
bug CWD Database Backend (Pepys) database connection

Comments

@IanMayo
Copy link
Member

IanMayo commented Jul 13, 2021

Analysts are pulling data into Debrief from Pepys, using the Pepys Import tool. They're in version 3.1.30, dated 2021/05/30

The Test connection function works, but an error is thrown when the Apply button is clicked for filtering.

[exact message to follow]

I've looked at the error log. A "normal" series of database import tracking logging message are shown, including a reference to using the Fast Mode algorithm.

At the top of the error log is a NPE in OSUtils.getInputStreamResource (OSUtils.java#56)

Note: we know that the Pepys database is up & running - since it could be verified in PGAdmin.
Note: there is a current issue with Pepys-Admin, which is showing an Alembic migration error. But, Debrief doesn't use Alembic.

@IanMayo IanMayo added bug CWD Database Backend (Pepys) database connection labels Jul 13, 2021
@IanMayo
Copy link
Member Author

IanMayo commented Aug 10, 2021

Robin has usefully pointed out that one of the missing migrations is the creation of the Stored Procedures that offer simpler connections and faster data transfer.
If Debrief is trying to use the new stored procedures then that would cause an issue. We may be able to reproduce the issue by (temporarily) changing the name of the Stored procedure in the AcuGis instance.

@IanMayo
Copy link
Member Author

IanMayo commented Aug 12, 2021

Ok. There's a strong chance the issue here is down to the stored procedures being missing from Pepys.

Debrief and pepys have got out of sync - so the Stored procedures used by Debrief aren't present.

But, I was able to do some further diagnostics today. I switched off Use stored procedures in Debrief-specific settings. I was able to filter the data, but when I tried to load tracks there was a long delay, then:

The column name reference was not found in the ResultSet

The lowest level Debrief code to this was at StateFastMode.retrieveObject in StateFastMode.java#110

This was where the Speed value was retrieved.

@IanMayo
Copy link
Member Author

IanMayo commented Aug 16, 2021

The stored procedures were missing from the client instance of Pepys.

But, they're present - and we still get the NPE on getInputStreamResource. Need to deploy the Debrief instance with deeper logging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug CWD Database Backend (Pepys) database connection
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant