You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We find the newer VNG output is 2 bytes smaller than before.
$ ls -l /tmp/*.vng
-rw-r--r-- 1 phil wheel 7740 Oct 24 10:35 /tmp/88f40ed.vng
-rw-r--r-- 1 phil wheel 7742 Oct 24 10:34 /tmp/c29dd97.vng
Since the changes in #4812 impacted the amount of memory used to store values, I thought this might have been an expected result, but @nwt confirmed it's unexpected. Hence this issue.
The text was updated successfully, but these errors were encountered:
Repro is with Zed commit 88f40ed and the test data file sample.tsv from the Zui repo.
At commit c29dd97 right before the changes in #4812, output the sorted file as VNG.
Repeat the same at commit 88f40ed, which is associated with the changes in #4812.
We find the newer VNG output is 2 bytes smaller than before.
Since the changes in #4812 impacted the amount of memory used to store values, I thought this might have been an expected result, but @nwt confirmed it's unexpected. Hence this issue.
The text was updated successfully, but these errors were encountered: