-
-
Notifications
You must be signed in to change notification settings - Fork 559
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
STANDARD-GENERIC-FUNCTION PGLOADER.CATALOG:FORMAT-TABLE-NAME : MS SQL missing FK table #1016
Comments
OK, I don't see where |
My point was that my first run was without If I include Hope you will do the right thing! |
Yes, but _my_ point was that you said it was in the debug log, and I don't
see it. I think it *should *be, and then both your and my problem would
have been more obvious, but I don't think it's something that should be
done automatically. You've told pgloader to select specific tables, I don't
think it should go beyond that, but it should certainly tell you when it
can't do what you want because of missing dependencies.
…On Wed, Dec 4, 2019 at 9:44 PM Dmitriy Dorofeev ***@***.***> wrote:
My point was that my first run was without dir_PeriodTypes in the config,
and it did not work. The DEBUG is for that case. My expectation was that
pgloader would detect the FK connected tables and include them
automagically.
If I include dir_PeriodTypes into config it worked fine, but not very
convenient ;-)
Hope you will do the right thing!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1016?email_source=notifications&email_token=AADRNO6ILSUORRJX26RMSG3QXAQEDA5CNFSM4IQM2SY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF6TEXY#issuecomment-561853023>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADRNO4IZUP46JW2CUEXGRDQXAQEDANCNFSM4IQM2SYQ>
.
--
derek
|
… out. See #1016 where we try to build the DDL for a foreign key that references tables that are not found in our catalogs. We should probably just ignore those foreign keys, as we might have a partial load to implement.
I have attached an attempt at bug fixing this situation, from my understanding of it reading this issue. Thanks for the detailed output and the solution you implemented. If you can try again and validate that the bug has been addressed, that would be nice. Thanks. |
Thanks dimitri!
Any advise? |
Actually I tried to fix the problem giving answers almost randomly and finally got pgloader compiled. I can confirm that your fix is working fine with my config:
|
did you test a fresh compile from the source tree?
did you search for other similar issues?
SQLite fails to copy all rows, fails with KABOOM! #681
how can I reproduce the bug?
Actually, I think it is the same bug as #681
Once I added missed table
dir_PeriodTypes
mentioned in the debug mess, everything works without any errors.The text was updated successfully, but these errors were encountered: