Hey guys, can you please take a look at <Switch th...
# contributei
Iliya R
07/14/2026, 4:50 AMHey guys, can you please take a look at Switch the fingerprint algo to xxh3_128 by Dev-iL · Pull Request #1630 · apache/hamilton when time permits?
🫡 1
s
Stefan Krawczyk
07/14/2026, 5:25 AMlicense addition is the only issue for me
Stefan Krawczyk
07/14/2026, 5:25 AMthat we need to clarify with mentors. CC @Jarek Potiuk plz 🙂
i
Iliya R
07/14/2026, 6:12 AMI'll take a quick look at how this is done in airflow
Iliya R
08/19/2026, 4:09 AM@Stefan Krawczyk think we can merge the PR w/o adding the env var you mentioned? My main concern is YAGNI + it can be added easily once someone asks for it. If it's critical for you (it didn't seem to be the case from the tone of your comment), I'll add it
s
Stefan Krawczyk
08/19/2026, 4:30 AMyeah I think enough time has passed we should just do it.
Stefan Krawczyk
08/19/2026, 4:32 AMmerged
i
Iliya R
08/19/2026, 4:32 AMTy!