common/options: change osd_target_transaction_size from int to uint

Change osd_target_transaction_size from signed int to unsigned int to
match the return type of Transaction::get_num_opts() (ceph_le64).

This change:
- Eliminates compiler warnings when comparing signed/unsigned values
- Enables automatic size conversion (e.g., "4_K" → 4096) via y2c.py
  for improved administrator usability
- Maintains type consistency throughout the codebase

Signed-off-by: Kefu Chai <tchaikov@gmail.com>

diff --git a/src/common/options/global.yaml.in b/src/common/options/global.yaml.in
index f3ebf1b1a8e2e8c6c24fa28af5545eea29ea5cd2..035d81346850934cad851761c263f4c0d131e476 100644 (file)
--- a/src/common/options/global.yaml.in
+++ b/src/common/options/global.yaml.in
@@ -3393,7 +3393,7 @@ options:
   with_legacy: true
 # to adjust various transactions that batch smaller items
 - name: osd_target_transaction_size
-  type: int
+  type: uint
   level: advanced
   default: 30
   with_legacy: true

diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 4de408d146f0438bdcd84b16354a341ed64533e6..d99ea15605b7461789f9234817e3371ad838a677 100644 (file)
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -5061,7 +5061,7 @@ void OSD::clear_temp_objects()
     }
     if (!temps.empty()) {
       ObjectStore::Transaction t;
-      int removed = 0;
+      unsigned removed = 0;
       for (vector<ghobject_t>::iterator q = temps.begin(); q != temps.end(); ++q) {
        dout(20) << "  removing " << *p << " object " << *q << dendl;
        t.remove(*p, *q);