Tony Chou

Results 34 comments of Tony Chou

Looking forward to having this PR merged! Would like to use this in the official build 🚀

I tested this in my environment and found out that the data is actually not ingested into neo4j. Good news is after a discussion and pair debugging with @zacr ,...

> Thanks so much @chonyy. I have an update in progress that addresses all three, hope to have it committed in the next day or so. @zacr that's great news!...

Hey @zacr , FYI, I tested your code in neo4j `3.5.26` and apoc `3.5.0.17`, it also doesn't work. The error message is like below. After I changed all the key...

I'm thinking that maybe we don't even have to spend time supporting Neo4j `3.3.0` since it's already EOL. @zacr's version works great with some minor fixes related to case sensitivity....

The `where_clause_suffix` provided in the sample code above is for MySQL. The syntax is not compatible for PostgreSQL. For Postgres it should be replaced to ```python # PostgreSQL where clause...

When working with Airflow pipelines, it would be better to split it into multiple tasks in a dag instead of running multiprocess in a single task. The sample code is...

Based on my experience, the ingestion speed is about 2.5 times faster by applying multiprocessing on the caller side. Since the performance improvement will be bounded by the neo4j instance,...

Hi @zacr , thanks for looking into this! I'm not super familiar with the capabilities of `UNWIND`, but the 5x speedup sounds promising 🚀 One thing that I would like...

@akiratwang @zacr Thanks for all the progress here! I'm so glad to see this issue finally turned into a PR and excited to see it get into production. The speed...