Manuel Garrido

Results 64 comments of Manuel Garrido
trafficstars

Not sure if Travis' failure has to do with my PR (tests passed locally). ```./exporters/export_formatter/avro_export_formatter.py:45:101: E501 line too long (102 > 100 characters) ./tests/test_formatters.py:172:71: E203 whitespace before ',' ./tests/test_formatters.py:197:1: W391...

@eliasdorneles the issue you are facing is because that the schema you provided does not match what RandomReader produces. The items produced by it look like this: ``` {'city': {'name':...

@tsrdatatech I run 2to3 on my code and the only thing I see is the change in `tests/test_formatters.py` ``` def test_create_without_options_raises_errors(self): - with self.assertRaisesRegexp(ConfigurationError, "requires at least one of"): +...

awesome, thanks @tsrdatatech let me know if you need anything else on my side. I'd rather not have to keep my own fork when this PR can benefit the community...

Awesome, thanks @tsrdatatech and @eliasdorneles :)

@Fleid would love to hear how its going :)

@Fleid as requested, im sharing my last comment in https://github.com/dbt-labs/dbt-snowflake/issues/533 to keep track of the bug: --- my original intentions where to setup a macro that select different warehouses depending...

is there any way I can help with getting this feature developed? At my company we are at the verge of needing this feature, and I would rather use a...

@Fleid any updates on this ?