Li Jin

Results 61 comments of Li Jin

Hi Hans, Can you explain a bit what issues did you see?

+1 to this. We internally depend on it currently being projection (isinstance checks) but not a huge deal to update to using the new classes.

stumbled upon this.. I wonder if the main idea to use `Node` as arguments for `Node` (instead of using `Expr`)? This looks like a pretty aggressive change and I wonder...

Thanks @cpcloud. I agree that `analysis.py` is bad and happy to see some effort to replace it. However, I do have some concern and questions about this. > To implement...

@rtpsw Thanks for this - I remember the mailing discussion concluded perhaps a design doc for more detailed discussion, I think what @cpcloud commented here could be good things to...

I think most Arrow folks do not watch ibis issues so maybe mailing list is still the better place for discussion of this

> would it be possible port some of the interval tests to improve upstream's coverage in order to prevent downstream test failures? We have most tests that can be pushed...

@cpcloud Thanks for chiming in.. I think we are in the tricky situation that lot of our code is not just build against the ibis public API (And to be...

I appreciate the thoughts. Sorry for the delayed reply I don't have a very good system to set up to not miss conversations like this :( Re @cpcloud > Considering...