summaryrefslogtreecommitdiffstats
path: root/yql/essentials/sql/pg
Commit message (Collapse)AuthorAgeFilesLines
* YQL-20095: Apply modernize-use-nodiscard at sqlvitya-smirnov2026-05-221-1/+1
| | | | commit_hash:16513c3f8da40e92c22e497694140f1c1584fb3f
* YQL-20095: Enable modernize-use-emplacevitya-smirnov2026-05-132-7/+7
| | | | commit_hash:4d6e2f6fcc1639f09725c0c96de8d2f7f0f70404
* YQL-20095: Enable readability-inconsistent-declaration-parameter-namevitya-smirnov2026-04-281-2/+2
| | | | commit_hash:9589937209af8e742c0c6401bd1c232f073856bf
* YQL-20189: Introduce PRAGMA Warning for pgSQLvitya-smirnov2026-04-152-17/+345
| | | | commit_hash:c2e47cfc5969038aa6a4076a3fc2f5e67d5083ba
* YQL-20095: Enable readability-isolate-declarationvitya-smirnov2026-04-131-1/+2
| | | | commit_hash:c180c2db4897962cb70063ffdc2ddcd21e5e3418
* YQL-20095: Enable readability-container-data-pointervitya-smirnov2026-04-081-1/+1
| | | | commit_hash:6bd18cff912ed7cc4b2bedd531c80cc43bdcd584
* YQL-20095: Enable readability-container-size-emptyvitya-smirnov2026-04-071-6/+6
| | | | commit_hash:464a95fd6de1a395f8ffe20092e837df995e623c
* YQL-20095: Enable readability-duplicate-includevitya-smirnov2026-04-061-1/+0
| | | | commit_hash:ea7cc749f3373f610d5945dc823ee74493744775
* YQL-20095: Enable modernize-use-designated-initializersvitya-smirnov2026-03-175-28/+28
| | | | | https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-designated-initializers.html commit_hash:d73eb463c7bec2bda4c362aab10af49979a7ddd3
* YQL-20095: Enable modernize-use-nullptrvitya-smirnov2026-03-041-3/+3
| | | | commit_hash:a504ad9d540aa714c550320e0ce20f9eb8bd17cf
* YQL-20095: Enable modernize-use-bool-literalsvitya-smirnov2026-03-031-6/+6
| | | | commit_hash:05fdc8e01569c5d74e2840a310ed1f7a34b72374
* YQL-20095: Enable modernize-pass-by-valuevitya-smirnov2026-03-022-3/+4
| | | | commit_hash:4939dfad654f18bb31e40711ee0c39063889f17b
* YQL-20095: Enable modernize-loop-convertvitya-smirnov2026-02-271-3/+5
| | | | | | | | - https://clang.llvm.org/extra/clang-tidy/checks/modernize/loop-convert.html Semi-automatic translation was performed. Manually rewritten some types and names. commit_hash:0ad921f0d512dd4a8ccc8af557197ce6a02582e1
* YQL-20095: Enable modernize-avoid-c-arraysvitya-smirnov2026-02-192-1/+2
| | | | commit_hash:da491ee93c4f4d3c885c7908a22b4d5d66c80388
* YQL-20868: separate typecheck at fastchecklukashevich842026-02-021-2/+9
| | | | commit_hash:4c9fb69c69ca016d87b38a1effc2f3ec3d4a0ecb
* YQL-20095: Enable google-explicit-constructorvitya-smirnov2026-01-233-4/+4
| | | | commit_hash:4d77ad10fd4db303459ec4e45e139967c7fc8196
* YQL-20095: Enable modernize-use-overridevitya-smirnov2026-01-133-4/+4
| | | | | | | | | | | | | | | | | | | The check `modernize-use-override` is useful as it: 1. Removes redundant `override` when it is used with a `final`. 2. Removes redundant `virtual` or replace it with a `override`. So it is more clean for a reader, that overriding happens. It is enabled not out of order, as it is just an alias to `cppcoreguidelines-explicit-virtual-functions`. I also decided to switch a strategy of enabling checks. Now I will enable only a single rule with a single PR and prefer rules with a non-breaking autofix. In the new year with new linter checks! 🎄 commit_hash:e6e233baa90b31e5f65e11837546690c47f71ab5
* YQL-20086: Enable arcadia and bugprone checksvitya-smirnov2025-11-251-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch enables more Clang Tidy checks for the `yql/essentials` project. I chosen the policy to enable all checks by default because - It will keep us updated when new checks are become available in the DevTools whitelist. - You never know what check will be useful, even if it was not fired now. Why some checks from the https://nda.ya.ru/t/8QnKCVIO7NWhBD were disabled? - `arcadia-taxi-` checks are `userver`-specific. - `bugprone-bool-pointer-implicit-conversion` is not applicable to our codebase as we heavily used to it. - `bugprone-branch-clone` triggered on a code that is not obviously rewrittable and not so poor. - `bugprone-dynamic-static-initializers` are heavily used in our codebase especially in the translator. Static variables with dynamic initialization are so common, so refactor them is too hard and requires huge code rewritting to deliver some hardcoded configuration values via ctors, having them in a one place. - `bugprone-exception-escape` is useful check for programs where the care of exception safety is taken, but we a lot of places where exception can be thrown from the destructor or move-operator. I left a TODO in the `.clang-tidy` to fix this. Maybe will create a ticket. - `bugprone-lambda-function-name` appears when we throw an exception from the lambda. I think, that this is okay and do not see any options to fix it. - `bugprone-macro-parentheses` had false positives at `YQL_ENSURE` second argument and also broke some code, by surrounding `A` at `template <class A>`. - `bugprone-misplaced-widening-cast`, `bugprone-narrowing-conversions`, `bugprone-signed-char-misuse` are disabled because checking arithmetics correctness is hard and non-trivial. I am not sure that it worth spending a lot of time on it as everything seems to work well :o) - `bugprone-reserved-identifier` was triggered on some harmless identifier in computation, so I decided just to turn this off, rather than argue with someone. - `bugprone-suspicious-enum-usage` happend on Arcadia HTTP client usage. Do not think that it is worth to argue with Arcadia HTTP client code owners. - `bugprone-suspicious-memory-comparison` happened in some compuration test. I just decided to trust low-level code authors. - `bugprone-too-small-loop-variable` seems not really a problem and lets you safe some symbols on `static_cast`s. - `bugprone-sizeof-expression` had so many false positives at computation. - `bugprone-unchecked-optional-access` a good check, but optionals are dereferenced everywhere in our codebase. Of course, in a perfect world near each of such dereference should be an assertion with comment why it is a valid action, because when static analysis can not infer that the value is non-null, then it is not an obvious thing for a human also. But we a too far from such code quality now. - `bugprone-undefined-memory-manipulation` is interesting. It argued on some `memcpy` at computation. There was some `UnboxedValues` array and this objects have some logic of memory marking and releasing under an `NDEBUG`. I added a TODO to check this. Maybe will create a ticket. - `bugprone-unhandled-self-assignment` has false positives on non-owning objects such as iterators. commit_hash:c0c0de6b157bf095eb681bdea55ada47845de814
* YQL-20086: Add yql/essentials Clang Tidy configvitya-smirnov2025-11-211-3/+3
| | | | | | | | | This patch introduces an extension configuration for the Clang Tidy. It is merged with Arcadia Clang Tidy configuration. To begin with, more checks for identifiers naming are added. Documentation: https://nda.ya.ru/t/AhbDZbiF7MKe3M. commit_hash:3481da4c8df0a4d23a991d4a660ae050d2dc5d33
* YQL-20086 invert flagvvvv2025-10-1010-1641/+1555
| | | | commit_hash:a6b640bc576263b62884a0b4187ba79c893e13d9
* YQL-20086 sqlvvvv2025-06-184-378/+378
| | | | commit_hash:b06f3d2f5fa81e2073966eb172758758bd94fdf2
* refactor SQL translators YQL-19594vvvv2025-02-161-3/+11
| | | | commit_hash:401d21dd23ee9bb7ee52b2fc42e596cb3e4bdda7
* Introduced SQL ITranslatorvvvv2025-02-141-14/+58
| | | | commit_hash:f58b2fb436f14fc4a11ce2e315f10d71e6d1ec77
* Disable pg ext warningsvvvv2024-12-111-1/+1
| | | | commit_hash:999bc4b82f70f18f47724fe965eb636f33e25820
* Support alter sequence restart with in pgshumkovnd2024-11-122-2/+6
| | | | | <https://github.com/ydb-platform/ydb/issues/11475> commit_hash:4db52ba0a2da92d383d7269cdb2e13d5a552b30a
* Moved other yql/essentials libs YQL-19206vvvv2024-11-0712-0/+9135
init commit_hash:7d4c435602078407bbf20dd3c32f9c90d2bbcbc0