Aliaksandr Valialkin

2.9K posts

Aliaksandr Valialkin

Aliaksandr Valialkin

@valyala

CTO @VictoriaMetrics

Katılım Kasım 2010
1K Takip Edilen4.1K Takipçiler
ns
ns@_naisstep·
@valyala @_devdeep well, not really, all go functions are async and all syscalls are have an await. isn't it like this?
English
1
0
0
31
Devdeep
Devdeep@_devdeep·
Backend Developers Do you know the difference? Promise vs async/await
Devdeep tweet media
English
32
3
116
13.8K
Aliaksandr Valialkin
1. Multiple lists != 100 lists. 2. If the disk read bandwidth it the bottleneck, then store compressed data on disk and unpack it during reading. This is a common technique used in databases optimized for storing and querying big amounts of data, such as VictoriaMetrics / VictoriaLogs.
English
0
0
0
31
avrl ☘
avrl ☘@avrldotdev·
Expedia rejected coz I couldn't answer this question: How would you design a distributed deduplication engine that ingests multiple lists each with 100k+ items, dedupes them in <2s & returns a single combined list?
English
27
1
156
28.3K
Valentin
Valentin@valentin_pollux·
@valyala Does me asking AI to fix it’s functions and all the crap and unnecessarily calling it was making to the API for an exporter counting for minimal human guidance ? 😅
English
1
0
1
215
luna
luna@ImLunaHey·
what IDE is everyone using nowadays?
English
204
0
81
38.9K
Aliaksandr Valialkin
Agreed that the inline comment in the code is the best source of additional context for non-trivial or questionable code. The commit message is still a good source of additional context for the particular change in the code - it lives close enough to the code, and it doesn't need online access to the main repository.
English
0
0
1
15
TWI 🇫🇷 🇪🇺 🇵🇸 🇺🇦 🇱🇧
@valyala In a non-oss org (and even for many oss orgs) there’s lots of text about what to do, how to do it, and why do it that way. Commit messages are one of the places where this information lives, and I’d argue it’s not the best place for it. I’d rather make a comment if need be
English
1
0
1
13
Aliaksandr Valialkin
I feel the same. Histrically software engineers weren't good in writing good commit titles and descriptions. So typical commit titles were "WIP", "fix", "refactor", "chore", etc. Now I see detailed commit titles and very long commit messages with many points, which describe all the changes made by the commit. The problem is that these new titles and messages are mostly useless (and even harmful) because of the reasons outlined by Kenton below. Make an effort and write good concise commit title and description by hand. If you cannot do this, then it is better to use "WIP" title instead of AI-generated misleading slop.
Kenton Varda@KentonVarda

I just declared a moratorium against AI-written change descriptions (e.g. PR and commit messages, also issues/tickets) from my team. AI was writing change descriptions that were worse than useless to me as I tried to review PRs: outlining details of the code that could easily be seen by looking at the code, but omitting the higher-level framing needed to understand broadly what the code is doing. I think people like having AI write these things because the output looks structured and thorough, which makes it feel professional in a way. But this isn't actually valuable. Concise, high-level descriptions are better for everyone. If I need to use my own AI to interpret what your AI wrote then something is wrong. Let AI write code, sure, but for the description, I'd rather see your prompt than your output. We could maybe have extended agents.md with guidelines on writing descriptions, but this seemed a bit pointless since a good, concise change description only takes a few minutes to write -- not a significant time savings to delegate to AI. At least, it doesn't take long if you understand the code -- and if you don't understand the code, then I'm definitely not merging it.

English
8
5
63
8.8K
Aliaksandr Valialkin
If your agents generate commit messages, which describe the contents of the commits, this is not what reviewers want to see. This is useless "capitain obvious" stuff. The commit message must contain higher-level overview of the commit - why it is needed and how does it resolve the given task / issue.
English
1
0
0
37
Jerry Combs
Jerry Combs@PointBlueTech·
@valyala I feel like I’m living in an alternate universe. My agents produce concise, informative, commit messages and PR descriptions. They are much better than those produced by humans.
English
1
0
0
37
max guy 😐
max guy 😐@GolerGkA·
@valyala Are you expecting me to know what every of the up to 500 commits that my agents do per day does?
English
1
0
0
84
Oli Wilkins
Oli Wilkins@okwilkins·
Not sure if it’s comparable to the “real” Grok chat experience on grok.com. Would also say that Grok these past few months got super upgraded. Unlike Perplexity now, it uses ~50-70 sources, searched in parallel, for an answer now. The model capabilities of the model allows it to fully comprehend the relevant ones. It’s largely replaced me manually searching for things when I have questions now.
English
1
0
1
35
Aliaksandr Valialkin
I didn't notice any progress in LLM-based search for the last 3 years. I started using Perplexity 3 years ago, and was impressed by its' web search abilities comparing to Google. As any LLM system at that time, sometimes it could return bullshit. 3 years passed since then, ChatGPT and Google released their versions of LLM-based web search. But they have the same level of response quality and the same issues with hallucinated responses as Perplexity had 3 years ago and has now. Where is the progress in LLM-based web search? I'd expect significant progress according to the marketing claims about significant advances of LLM-based code generation during the last years.
English
4
0
21
2.3K
Aliaksandr Valialkin
Good commit titles and descriptions help investigating commit history. For example, if I need to understand why the particular code is written in that way, I run `git log -p ...` and investigate the related commits (including their titles and descriptions). Bad commit titles and description may slow down the investigation significantly.
English
1
0
0
41
TWI 🇫🇷 🇪🇺 🇵🇸 🇺🇦 🇱🇧
@valyala Well, commit message or PR bodies do not matter that much anyway. What matters is « X made all these changes at that time ». Important things about code should be in the code, not in a foreign metadata
English
1
0
0
50
Aliaksandr Valialkin
@chrisczynski You can freely amend commit messages until the local commits are pushed to the upstream git repository. See stackoverflow.com/a/1186549 . I frequently change the local commit title and description with `git commit --amend` before pushing it to the upstream repo.
English
1
0
1
50
Senior AI Micromanager
Senior AI Micromanager@chrisczynski·
@valyala It always annoyed me that you can't amend git commit messages without breaking the chain. This really needs to be decoupled.
English
1
0
0
65
Aliaksandr Valialkin
@okwilkins Contrary, I noticed a regression in the Grok explanations under tweets: previously it was reasoning much more and was providing interesting insights sometimes. Now it just rephrases the original tweet, and that's it :(
English
1
0
1
42
Aliaksandr Valialkin
@okwilkins I try Grok from time to time for the last year, but don't see significant progress with the quality of answers it provides.
English
1
0
2
87
Harsh
Harsh@theharshpat·
@valyala Loved your unhinged single commit for complete single node implementation of victoria metrics from 2018. Reading that right now
English
1
0
2
341
@jorgelbg@mastodon.social
@[email protected]@jorgelbg·
@valyala If they were good descriptions/commit messages it would be ok~ish. Usually I refer to this as if they were written by "Capt'n Obvious" they don't say why a given change was done, they make a textual equivalent to the diff (best case).
English
2
0
2
243