Love Eklund
09/16/2024, 12:14 PMplugins:
extractors:
- name: tap-oracle
variant: s7clarke10
pip_url:
git+<https://github.com/s7clarke10/pipelinewise-tap-oracle.git@53bb75ed27d7796d2f492e74cec87f10f1bce4d4>
config:
use_ora_rowscn: false
ora_python_driver_type: thick
use_singer_decimal: true
- name: tap-oracle-demo_foo
inherit_from: tap-oracle
config:
user: demo
host: 123
port: 420
service_name: foo
- name: tap-oracle-demo_foo-full_table
inherit_from: tap-oracle-demo_foo
config:
default_replication_method: FULL_TABLE
- name: tap-oracle-demo_foo-full_table-bar_baz
inherit_from: tap-oracle-demo_foo-full_table
select:
- bar-baz.*
- name: tap-oracle-demo_foo-full_table-asd_qwe
inherit_from: tap-oracle-demo_foo-full_table
select:
- asd-qwe.*
when running meltano install
I get the following output.
2024-09-16T09:45:39.392382Z [info ] Installing 8 plugins
2024-09-16T09:45:39.392942Z [info ] Skipped installing extractor 'tap-oracle-demo_foo'
2024-09-16T09:45:39.393006Z [info ] Skipped installing extractor 'tap-oracle-demo_foo-full_table-asd_qwe'
2024-09-16T09:45:39.393708Z [info ] Installing extractor 'tap-testsource'
2024-09-16T09:45:39.408495Z [info ] Installing extractor 'tap-oracle'
2024-09-16T09:45:39.454512Z [info ] Installing extractor 'tap-oracle-demo_foo-full_table'
2024-09-16T09:45:39.587498Z [info ] Installing extractor 'tap-oracle-demo_foo-full_table-bar_baz'
2024-09-16T09:45:39.763693Z [info ] Installing loader 'target-jsonl'
2024-09-16T09:45:39.776647Z [info ] Installing loader 'target-bigquery'
2024-09-16T09:45:43.563063Z [info ] Installed loader 'target-jsonl'
2024-09-16T09:45:46.557149Z [info ] Installed extractor 'tap-oracle'
2024-09-16T09:45:46.708771Z [info ] Installed extractor 'tap-oracle-demo_foo-full_table'
2024-09-16T09:45:46.938197Z [info ] Installed extractor 'tap-oracle-demo_foo-full_table-bar_baz'
2024-09-16T09:45:48.596501Z [info ] Installed extractor 'tap-testsource'
2024-09-16T09:45:55.892412Z [info ] Installed loader 'target-bigquery'
2024-09-16T09:45:55.892868Z [info ] Installed 6/8 plugins
2024-09-16T09:45:55.892947Z [info ] Skipped installing 2/8 plugins
it skips the asd_qwe
but installs the bar_baz
why?
I assume here due to Skipped installing extractor 'tap-oracle-demo_foo'
the that no venv is created for the tap-oracle-demo_foo
but this is the result I get in the .meltano folder ( see attached screenshot).Edgar Ramírez (Arch.dev)
09/16/2024, 5:57 PMit skips theI think both are actually skipped, because I seebut installs theasd_qwe
why?bar_baz
Skipped installing 2/8 plugins
in there. Though that means Installing extractor 'tap-oracle-demo_foo-full_table-bar_baz'
is misleading, so we might need to fix that.
I assume here due toI think those directories actually have emptythe that no venv is created for theSkipped installing extractor 'tap-oracle-demo_foo'
but this is the result I get in the .meltano folder ( see attached screenshot).tap-oracle-demo_foo
venv
subdirs, can you confirm? We might still wanna confirm that no unnecessary directories are created, similar to what we addressed in https://github.com/meltano/meltano/pull/8447.Hayden Ness
09/17/2024, 1:54 AMinstall
installs an environment to the parent.
I have a structure like tap-mssql -> tap-mssql-mydb -> tap-mssql-mydb-installer
. I run meltano install extractor tap-mssql-mydb-installer
which then installs a tap-mssql environment to the tap-mssql-mydb
folder. No folder/files are created for tap-mssql
nor tap-mssql-mydb-installer
. For multiple databases, I ended up with a tap-mssql environment in each tap-mssql-mydbXX
.
If I were to just do meltano install tap-mssql-mydb
then the tap-mssql environment is installed to the tap-mssql
folder, and nothing to the tap-mssql-mydb
folder. Unfortunately, if I do meltano run --no-install tap-mssql-mydb-workerX
, it thinks tap-mssql-mydb
is not installed then, hence the need for the "installer", so it seems multiple levels of inheritance are disregarded here.
It's good that there is some attempt to avoid unnecessary copies of packages, but there's definitely still some duplication, questionable behaviour, and unintuitive design here.Love Eklund
09/17/2024, 6:46 AMEdgar Ramírez (Arch.dev)
09/18/2024, 2:55 PMReuben (Matatika)
09/19/2024, 3:05 AMReuben (Matatika)
09/19/2024, 3:25 AMEdgar Ramírez (Arch.dev)
09/24/2024, 9:47 PMHayden Ness
09/24/2024, 11:18 PM