advancedxy

> We can hold Comet release after the new releases of arrow-rs and DataFusion. Thanks for the clarification. I second this.

> I propose that we create the 0.1.0 source release as soon as we have upated the project to use the upcoming DataFusion 39.0.0 release which should be avalable around...

> We plan to do binary release, although it might not be able to catch up the 0.1.0 source release. I think we are on the same page. > Comet...

> Could you also update docs/source/user-guide/expressions.md to add xxhash64 as a supported expression? Of course, I will update that among other things: such as the review comments and the inspection...

> I'd like to see the tests use some randomly generated inputs. > > As a quick hack, I added the following test to `CometCastSuite` and it shows some differences...

> Let me check why the test is failing first. Found the issue. The `create_hashes_dictionary` doesn't handle input hashes correctly, it affects both murmur3hash and this new xxhash64 method. Let...

> > Our `hash` implementation is also not compatible with Spark. I will file an issue for that. > > I filed #427 Thanks for filing this. I think it's...

> Found the issue. The create_hashes_dictionary doesn't handle input hashes correctly, it affects both murmur3hash and this new xxhash64 method. > Let me try to fix that first. I have...

@andygrove @viirya I have created #433 and mark this as a draft. We should merge that first and then come back to this PR . PLAL when you have tome.

@andygrove @viirya @parthchandra and @sunchao would you mind to take a look at this? I think it's ready for review.