Transitional flag FIX_UNUSED_PARAMETR_PLS=1 (devtools request)

build/internal/ya.conf

@@ -22,3 +22,5 @@ USE_ICONV = "static"
 USE_IDN = "static"
 APPLE_SDK_LOCAL = "yes"
 CFLAGS = "-fno-omit-frame-pointer"
+# TODO: uncomment after warnings fix
+# FIX_UNUSED_PARAMETR_PLS = "1"  # 31 jul 2024. Transitional flag for protobuf fix, DTCC-2275

ydb/core/blobstorage/nodewarden/node_warden_pdisk.cpp

@@ -485,7 +485,7 @@ namespace NKikimr::NStorage {
             Become(&TThis::StateFunc, TDuration::Seconds(10), new TEvents::TEvWakeup);
         }
 
-        void Handle(TEvents::TEvUndelivered::TPtr ev) {
+        void Handle(TEvents::TEvUndelivered::TPtr /*ev*/) {
             // send this event again, this may be a race with PDisk destruction
             TActivationContext::Send(OriginalEv.release());
             PassAway();

ydb/core/blobstorage/pdisk/blobstorage_pdisk_impl_metadata.cpp

@@ -70,7 +70,7 @@ namespace NKikimr::NPDisk {
                 Release(actorSystem);
             }
 
-            void Release(TActorSystem *actorSystem) override {
+            void Release(TActorSystem * /*actorSystem*/) override {
                 delete this;
             }
         };
@@ -231,7 +231,7 @@ namespace NKikimr::NPDisk {
 
             bool CanHandleResult() const override { return true; }
 
-            void Exec(TActorSystem *actorSystem) override {
+            void Exec(TActorSystem * /*actorSystem*/) override {
                 LOG_DEBUG_S(*PDisk->ActorSystem, NKikimrServices::BS_PDISK, "PDiskId# " << PDisk->PDiskId
                     << " TCompletionWriteUnformattedMetadata::Exec"
                     << " Result# " << Result

ydb/core/config/tools/protobuf_plugin/main.cpp

@@ -44,8 +44,7 @@ class TMessageGenerator
         return name;
     }
 
-    TString ConstructFullFieldPath(const TDeque<const FieldDescriptor*>& fieldPath, const FieldDescriptor* field) const {
-
+    TString ConstructFullFieldPath(const TDeque<const FieldDescriptor*>& fieldPath, const FieldDescriptor* /*field*/) const {
         TVector<TString> path;
         path.push_back("");
         for (size_t i = 1; i < fieldPath.size(); ++i) {

ydb/core/formats/arrow/accessor/composite/accessor.h

@@ -21,7 +21,7 @@ class TCompositeChunkedArray: public NArrow::NAccessor::IChunkedArray {
         return {};
     }
 
-    virtual std::shared_ptr<arrow::Scalar> DoGetScalar(const ui32 index) const override {
+    virtual std::shared_ptr<arrow::Scalar> DoGetScalar(const ui32 /*index*/) const override {
         AFL_VERIFY(false)("problem", "cannot use method");
         return nullptr;
     }

ydb/core/formats/arrow/accessor/composite_serial/accessor.h

@@ -60,7 +60,7 @@ class TDeserializeChunkedArray: public NArrow::NAccessor::IChunkedArray {
         return {};
     }
 
-    virtual std::shared_ptr<arrow::Scalar> DoGetScalar(const ui32 index) const override {
+    virtual std::shared_ptr<arrow::Scalar> DoGetScalar(const ui32 /*index*/) const override {
         AFL_VERIFY(false)("problem", "cannot use method");
         return nullptr;
     }

ydb/core/formats/arrow/accessor/plain/accessor.h

@@ -12,7 +12,7 @@ class TTrivialArray: public IChunkedArray {
 protected:
     virtual std::optional<ui64> DoGetRawSize() const override;
 
-    virtual TLocalDataAddress DoGetLocalData(const std::optional<TCommonChunkAddress>& /*chunkCurrent*/, const ui64 position) const override {
+    virtual TLocalDataAddress DoGetLocalData(const std::optional<TCommonChunkAddress>& /*chunkCurrent*/, const ui64 /*position*/) const override {
         return TLocalDataAddress(Array, 0, 0);
     }
     virtual std::shared_ptr<arrow::ChunkedArray> DoGetChunkedArray() const override {

ydb/core/formats/arrow/accessor/sparsed/accessor.h

@@ -115,7 +115,7 @@ class TSparsedArray: public IChunkedArray {
         return bytes;
     }
 
-    TSparsedArray(std::vector<TSparsedArrayChunk>&& data, const std::shared_ptr<arrow::Scalar>& defaultValue,
+    TSparsedArray(std::vector<TSparsedArrayChunk>&& data, const std::shared_ptr<arrow::Scalar>& /*defaultValue*/,
         const std::shared_ptr<arrow::DataType>& type, const ui32 recordsCount)
         : TBase(recordsCount, EType::SparsedArray, type)
         , Records(std::move(data)) {

ydb/core/formats/arrow/accessor/sparsed/constructor.cpp

@@ -27,7 +27,7 @@ NKikimrArrowAccessorProto::TConstructor TConstructor::DoSerializeToProto() const
     return result;
 }
 
-bool TConstructor::DoDeserializeFromProto(const NKikimrArrowAccessorProto::TConstructor& proto) {
+bool TConstructor::DoDeserializeFromProto(const NKikimrArrowAccessorProto::TConstructor& /*proto*/) {
     return true;
 }
 

ydb/core/formats/arrow/reader/position.h

@@ -97,7 +97,7 @@ class TSortableScanData {
         return PositionAddress[colIdx].GetAddress().GetLocalIndex(pos);
     }
 
-    std::shared_ptr<TSortableScanData> BuildCopy(const ui64 position) const {
+    std::shared_ptr<TSortableScanData> BuildCopy(const ui64 /*position*/) const {
         return std::make_shared<TSortableScanData>(*this);
     }
 

ydb/core/grpc_services/rpc_alter_table.cpp

@@ -228,7 +228,7 @@ class TAlterTableRPC : public TRpcSchemeRequestActor<TAlterTableRPC, TEvAlterTab
         Send(schemeCache, ev);
     }
 
-    void Navigate(const TTableId& pathId, const TActorContext& ctx) {
+    void Navigate(const TTableId& pathId, const TActorContext& /*ctx*/) {
         DatabaseName = Request_->GetDatabaseName()
             .GetOrElse(DatabaseFromDomain(AppData()));
 

ydb/core/persqueue/read_balancer__balancing.cpp

@@ -1827,7 +1827,7 @@ void TBalancer::Handle(TEvPQ::TEvBalanceConsumer::TPtr& ev, const TActorContext&
     }
 }
 
-void TBalancer::Handle(TEvPersQueue::TEvStatusResponse::TPtr& ev, const TActorContext& ctx) {
+void TBalancer::Handle(TEvPersQueue::TEvStatusResponse::TPtr& ev, const TActorContext& /*ctx*/) {
     const auto& record = ev->Get()->Record;
     for (const auto& partResult : record.GetPartResult()) {
         for (const auto& consumerResult : partResult.GetConsumerResult()) {

ydb/core/statistics/aggregator/tx_ack_timeout.cpp

@@ -9,7 +9,7 @@ struct TStatisticsAggregator::TTxAckTimeout : public TTxBase {
 
     TTxType GetTxType() const override { return TXTYPE_ACK_TIMEOUT; }
 
-    bool Execute(TTransactionContext& txc, const TActorContext&) override {
+    bool Execute(TTransactionContext& /*txc*/, const TActorContext&) override {
         SA_LOG_D("[" << Self->TabletID() << "] TTxAckTimeout::Execute");
         return true;
     }

ydb/core/statistics/aggregator/tx_analyze_table.cpp

@@ -50,7 +50,7 @@ struct TStatisticsAggregator::TTxAnalyzeTable : public TTxBase {
         return true;
     }
 
-    void Complete(const TActorContext& ctx) override {
+    void Complete(const TActorContext& /*ctx*/) override {
         SA_LOG_D("[" << Self->TabletID() << "] TTxAnalyzeTable::Complete");
     }
 };

ydb/core/statistics/aggregator/ut/ut_traverse_columnshard.cpp

@@ -55,7 +55,7 @@ Y_UNIT_TEST_SUITE(TraverseColumnShard) {
         auto sender = runtime.AllocateEdgeActor();
         int observerCount = 0;
         auto observer = runtime.AddObserver<TEvTxProxySchemeCache::TEvResolveKeySetResult>(
-            [&](TEvTxProxySchemeCache::TEvResolveKeySetResult::TPtr& ev)
+            [&](TEvTxProxySchemeCache::TEvResolveKeySetResult::TPtr& /*ev*/)
         {
             if (observerCount++ == 2) {
                 RebootTablet(runtime, saTabletId, sender);
@@ -89,7 +89,7 @@ Y_UNIT_TEST_SUITE(TraverseColumnShard) {
         auto sender = runtime.AllocateEdgeActor();
         bool observerFirstExec = true;
         auto observer = runtime.AddObserver<TEvHive::TEvRequestTabletDistribution>(
-            [&](TEvHive::TEvRequestTabletDistribution::TPtr& ev)
+            [&](TEvHive::TEvRequestTabletDistribution::TPtr& /*ev*/)
         {
             if (observerFirstExec) {
                 observerFirstExec = false;
@@ -124,7 +124,7 @@ Y_UNIT_TEST_SUITE(TraverseColumnShard) {
         auto sender = runtime.AllocateEdgeActor();
         bool observerFirstExec = true;
         auto observer = runtime.AddObserver<TEvStatistics::TEvAggregateStatistics>(
-            [&](TEvStatistics::TEvAggregateStatistics::TPtr& ev)
+            [&](TEvStatistics::TEvAggregateStatistics::TPtr& /*ev*/)
         {
             if (observerFirstExec) {
                 observerFirstExec = false;
@@ -159,7 +159,7 @@ Y_UNIT_TEST_SUITE(TraverseColumnShard) {
         auto sender = runtime.AllocateEdgeActor();
         bool observerFirstExec = true;
         auto observer = runtime.AddObserver<TEvStatistics::TEvAggregateStatisticsResponse>(
-            [&](TEvStatistics::TEvAggregateStatisticsResponse::TPtr& ev)
+            [&](TEvStatistics::TEvAggregateStatisticsResponse::TPtr& /*ev*/)
         {
             if (observerFirstExec) {
                 observerFirstExec = false;
@@ -194,7 +194,7 @@ Y_UNIT_TEST_SUITE(TraverseColumnShard) {
         auto sender = runtime.AllocateEdgeActor();
         int observerCount = 0;
         auto observer = runtime.AddObserver<TEvStatistics::TEvStatisticsRequest>(
-            [&](TEvStatistics::TEvStatisticsRequest::TPtr& ev)
+            [&](TEvStatistics::TEvStatisticsRequest::TPtr& /*ev*/)
         {
             if (observerCount++ == 5) {
                 RebootTablet(runtime, saTabletId, sender);

ydb/core/tx/columnshard/engines/changes/indexation.cpp

@@ -53,7 +53,7 @@ class TBatchInfo {
     YDB_READONLY_DEF(std::shared_ptr<NArrow::TGeneralContainer>, Batch);
 
 public:
-    TBatchInfo(const std::shared_ptr<NArrow::TGeneralContainer>& batch, const NEvWrite::EModificationType modificationType)
+    TBatchInfo(const std::shared_ptr<NArrow::TGeneralContainer>& batch, const NEvWrite::EModificationType /*modificationType*/)
         : Batch(batch) {
     }
 };

ydb/core/tx/columnshard/engines/scheme/abstract/index_info.h

@@ -165,7 +165,7 @@ class IIndexInfo {
                fieldId == (ui32)ESpecialColumn::DELETE_FLAG;
     }
 
-    static bool IsNullableVerified(const ui32 fieldId) {
+    static bool IsNullableVerified(const ui32 /*fieldId*/) {
         return false;
     }
 

ydb/core/tx/datashard/build_index.cpp

@@ -283,7 +283,7 @@ class TBuildScanUpload: public TActor<TBuildScanUpload<Activity>>, public NTable
     }
 
     template <typename TAddRow>
-    EScan FeedImpl(TArrayRef<const TCell> key, const TRow& row, TAddRow&& addRow) noexcept {
+    EScan FeedImpl(TArrayRef<const TCell> key, const TRow& /*row*/, TAddRow&& addRow) noexcept {
         LOG_T("Feed key " << DebugPrintPoint(KeyTypes, key, *AppData()->TypeRegistry) << " " << Debug());
 
         addRow();
@@ -440,7 +440,7 @@ class TBuildScanUpload: public TActor<TBuildScanUpload<Activity>>, public NTable
         }
     }
 
-    void HandleWakeup(const NActors::TActorContext& ctx) {
+    void HandleWakeup(const NActors::TActorContext& /*ctx*/) {
         LOG_D("Retry upload " << Debug());
 
         if (!WriteBuf.IsEmpty()) {

ydb/core/tx/schemeshard/schemeshard__operation_apply_build_index.cpp

@@ -28,7 +28,7 @@ ISubOperation::TPtr FinalizeIndexImplTable(TOperationContext& context, const TPa
     return CreateFinalizeBuildIndexImplTable(partId, transaction);
 }
 
-ISubOperation::TPtr DropIndexImplTable(TOperationContext& context, const TPath& index, const TOperationId& nextId, const TOperationId& partId, const TString& name, const TPathId& pathId) {
+ISubOperation::TPtr DropIndexImplTable(TOperationContext& /*context*/, const TPath& index, const TOperationId& nextId, const TOperationId& partId, const TString& name, const TPathId& pathId) {
     TPath implTable = index.Child(name);
     Y_ABORT_UNLESS(implTable->PathId == pathId);
     Y_ABORT_UNLESS(implTable.LeafName() == name);

ydb/core/tx/schemeshard/schemeshard__operation_create_restore_incremental_backup.cpp

@@ -16,7 +16,7 @@ const char* IB_RESTORE_CDC_STREAM_NAME = "__ib_restore_stream";
 
 namespace NKikimr::NSchemeShard {
 
-void DoCreateLock(const TOperationId opId, const TPath& workingDirPath, const TPath& tablePath, bool allowIndexImplLock,
+void DoCreateLock(const TOperationId opId, const TPath& workingDirPath, const TPath& tablePath, bool /*allowIndexImplLock*/,
     TVector<ISubOperation::TPtr>& result)
 {
     auto outTx = TransactionTemplate(workingDirPath.PathString(),

ydb/library/yql/minikql/comp_nodes/mkql_wide_combine.cpp

@@ -132,7 +132,7 @@ struct TCombinerNodes {
         }
     }
 
-    void ConsumeRawData(TComputationContext& ctx, NUdf::TUnboxedValue* keys, NUdf::TUnboxedValue** from, NUdf::TUnboxedValue* to) const {
+    void ConsumeRawData(TComputationContext& /*ctx*/, NUdf::TUnboxedValue* keys, NUdf::TUnboxedValue** from, NUdf::TUnboxedValue* to) const {
         std::fill_n(keys, KeyResultNodes.size(), NUdf::TUnboxedValuePod());
         for (ui32 i = 0U; i < ItemNodes.size(); ++i) {
             if (from[i] && IsInputItemNodeUsed(i)) {

ydb/library/yql/providers/yt/lib/res_pull/res_or_pull.cpp

@@ -185,7 +185,7 @@ void TSkiffExecuteResOrPull::WriteValue(const NUdf::TUnboxedValue& value, TType*
     }
 }
 
-bool TSkiffExecuteResOrPull::WriteNext(TMkqlIOCache& specsCache, const NYT::TNode& rec, ui32 tableIndex) {
+bool TSkiffExecuteResOrPull::WriteNext(TMkqlIOCache& specsCache, const NYT::TNode& rec, ui32 /*tableIndex*/) {
     if (!HasCapacity()) {
         Truncated = true;
         return false;

ydb/library/yql/sql/v1/select.cpp

@@ -287,7 +287,7 @@ class TNodeSource: public ISource {
         return ISource::DoInit(ctx, src);
     }
 
-    TNodePtr Build(TContext& ctx) final  {
+    TNodePtr Build(TContext& /*ctx*/) final  {
         auto nodeAst = AstNode(Node);
         if (WrapToList) {
             nodeAst = Y("ToList", nodeAst);