search

enova / pg_fact_loader

Build fact tables asynchronously with Postgres
MIT License
11 stars 6 forks source link

Test failure with PG12 #11

Closed df7cb closed 4 years ago

df7cb commented 4 years ago

Building against PG12:

**** regression.diffs ****
diff -U3 /tmp/autopkgtest.Ijj9Ul/tree/expected/07_launch_worker.out /tmp/autopkgtest.Ijj9Ul/tree/results/07_launch_worker.out
--- /tmp/autopkgtest.Ijj9Ul/tree/expected/07_launch_worker.out  2018-11-09 22:52:31.131525573 +0100
+++ /tmp/autopkgtest.Ijj9Ul/tree/results/07_launch_worker.out   2020-02-29 22:29:20.221818431 +0100
@@ -35,7 +35,7 @@
            7 | 0001234567 |  35 |               |                   0 | 
            8 | 0001234568 |  35 |               |                   0 | 
            9 | 0001234569 |  35 |               |                   0 | 
-          10 | 0001234577 |  35 |               |                   0 | 
+          10 | 0001234560 |  35 |               |                   0 | 
 (10 rows)

 UPDATE fact_loader.fact_tables SET force_worker_priority = FALSE WHERE fact_table_relid = 'test_fact.customers_fact'::REGCLASS;
jfinzel commented 4 years ago

@df7cb we will be looking at releasing this within the next 2 weeks. Thank you.

jfinzel commented 4 years ago

@df7cb I don't see any failures with PR to-be-merged. Sorry again for the delay on this but I hope to tag a release this week.

jfinzel commented 4 years ago

@df7cb please let me know if the new tag https://github.com/enova/pg_fact_loader/releases/tag/v1.6.0 fixes this issue.

df7cb commented 4 years ago

Looks good, thanks!