singer-io / target-csv

Write Singer data to CSV files
GNU Affero General Public License v3.0
37 stars 68 forks source link

tap-marketo not working with target-csv #5

Closed nicholassewitz closed 7 years ago

nicholassewitz commented 7 years ago

When I try and send marketo data to the singer target-csv I am getting this error

artsy-nicholass:marketo-fulcrum nicholassewitz$ tap-marketo -c config.json -s state.json | target-csv -c csv-config.json
Traceback (most recent call last):
  File "/usr/local/bin/tap-marketo", line 7, in <module>
    from tap_marketo import main
  File "/usr/local/lib/python3.6/site-packages/tap_marketo/__init__.py", line 12, in <module>
    from singer import utils
ImportError: cannot import name 'utils'
nicholassewitz commented 7 years ago

fyi when I ran on separate virtualenv's this error went away.

nicholassewitz commented 7 years ago

now getting this error

    sys.exit(main())
  File "/Users/nicholassewitz/marketo-fulcrum/tap-marketo/lib/python3.6/site-packages/tap_marketo/__init__.py", line 320, in main
    do_sync()
  File "/Users/nicholassewitz/marketo-fulcrum/tap-marketo/lib/python3.6/site-packages/tap_marketo/__init__.py", line 267, in do_sync
    singer.write_schema("leads", schema, ["id"])
  File "/Users/nicholassewitz/marketo-fulcrum/tap-marketo/lib/python3.6/site-packages/singer/__init__.py", line 91, in write_schema
    key_properties=key_properties))
  File "/Users/nicholassewitz/marketo-fulcrum/tap-marketo/lib/python3.6/site-packages/singer/__init__.py", line 52, in _write_message
    sys.stdout.write(message.tojson() + '\n')
BrokenPipeError: [Errno 32] Broken pipe
gris commented 7 years ago

Have you tried to install the latest version of singer-python? When I installed target-csv it broke the tap-adwords and that was the issue. I've made a PR #6 that fix this by changing the setup.py to require the latest version.

nicholassewitz commented 7 years ago

I ended up just interacting directly with the api.

On Tue, Aug 8, 2017 at 10:13 AM, João Marcos Gris notifications@github.com wrote:

Have you tried to install the latest version of singer-python? When I installed target-csv it broke the tap-adwords and that was the issue. I've made a PR #6 https://github.com/singer-io/target-csv/pull/6 that fix this by changing the setup.py to require the latest version.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/singer-io/target-csv/issues/5#issuecomment-320968648, or mute the thread https://github.com/notifications/unsubscribe-auth/AKaPgjDWHqI2vl72rGM_3YCKHkku_H7Tks5sWG0VgaJpZM4NKyqv .

gris commented 7 years ago

I don't have a marketo account, but I think this was fixed in target-csv version 0.2.3. Can you check that @cmerrick?

cmerrick commented 7 years ago

this is more of a design issue that's best fixed by using different virtualenvs. That's an issue because it's not very user friendly. Closing for now, but that more general issue is on our radar.