Don't log psiLen in VfsEventsMerger, because there is no fast and reliable way to understand if file may have PSI or not

GitOrigin-RevId: f0de712a22a10d2ec2bff34a7d9798f320556bf6
This commit is contained in:
Andrei.Kuznetsov
2024-10-11 12:22:52 +02:00
committed by intellij-monorepo-bot
parent 8a089889af
commit 1247257fe0

View File

@@ -261,7 +261,10 @@ public final class VfsEventsMerger {
if (indexedFile instanceof FileContent fileContent) {
extra += ",contLen(b)=" + fileContent.getContent().length;
FileType fileType = fileContent.getFileType();
extra += ",psiLen=" + (fileType instanceof LanguageFileType ? fileContent.getPsiFile().getTextLength() : -1);
// WARNING: LanguageFileType does not guarantee that there is a PsiFile.
// Example: org.jetbrains.bazel.languages.projectview.base.ProjectViewFileType
// psiLen has never been helpful to me, so don't log it for now.
// extra += ",psiLen=" + (fileType instanceof LanguageFileType ? fileContent.getPsiFile().getTextLength() : -1);
extra += ",bin=" + (fileType.isBinary() ? "t" : "f");
}