Add GitHub issues for TODO comments

* removed TODO comments that were no longer applicable (i.e., comments that seemed that should have been removed)

Author: alinaliBQ

cpp/src/arrow/flight/sql/odbc/odbc_impl/accessors/types.h

@@ -102,7 +102,7 @@ class FlightSqlAccessor : public Accessor {
           throw NullWithoutIndicatorException();
         }
       } else {
-        // TODO: Optimize this by creating different versions of MoveSingleCell
+        // GH-47849 TODO: Optimize this by creating different versions of MoveSingleCell
         // depending on if str_len_buffer is null.
         auto row_status = MoveSingleCell(binding, current_arrow_row, i, value_offset,
                                          update_value_offset, diagnostics);

cpp/src/arrow/flight/sql/odbc/odbc_impl/flight_sql_auth_method.cc

@@ -38,9 +38,8 @@ class NoOpAuthMethod : public FlightSqlAuthMethod {
                     FlightCallOptions& call_options) override {
     // Do nothing
 
-    // TODO: implement NoOpAuthMethod to validate server address.
+    // GH-46733 TODO: implement NoOpAuthMethod to validate server address.
     // Can use NoOpClientAuthHandler.
-    // https://github.com/apache/arrow/issues/46733
   }
 };
 

cpp/src/arrow/flight/sql/odbc/odbc_impl/flight_sql_connection.cc

@@ -339,7 +339,7 @@ Location FlightSqlConnection::BuildLocation(
           ThrowIfNotOK(Location::ForGrpcTls(host_name_info, port).Value(&location));
           return location;
         }
-        // TODO: We should log that we could not convert an IP to hostname here.
+        // GH-47852 TODO: We should log that we could not convert an IP to hostname here.
       }
     } catch (...) {
       // This is expected. The Host attribute can be an IP or name, but make_address will

cpp/src/arrow/flight/sql/odbc/odbc_impl/flight_sql_get_tables_reader.cc

@@ -77,10 +77,9 @@ std::shared_ptr<Schema> GetTablesReader::GetSchema() {
   const arrow::Result<std::shared_ptr<Schema>>& result =
       arrow::ipc::ReadSchema(&dataset_schema_reader, &in_memo);
   if (!result.ok()) {
-    // TODO: Test and build the driver against a server that returns
+    // GH-46561 TODO: Test and build the driver against a server that returns
     // complex types columns with the children
     // types and handle the failure properly
-    // https://github.com/apache/arrow/issues/46561
     return nullptr;
   }
 

cpp/src/arrow/flight/sql/odbc/odbc_impl/flight_sql_result_set_metadata.cc

@@ -157,19 +157,27 @@ size_t FlightSqlResultSetMetadata::GetLength(int column_position) {
 }
 
 std::string FlightSqlResultSetMetadata::GetLiteralPrefix(int column_position) {
-  // TODO: Flight SQL column metadata does not have this, should we add to the spec?
+  // GH-47853 TODO: use `ColumnMetadata` to get literal prefix after Flight SQL protocol
+  // adds support for it
+
+  // Flight SQL column metadata does not have literal prefix, empty string is returned
   return "";
 }
 
 std::string FlightSqlResultSetMetadata::GetLiteralSuffix(int column_position) {
-  // TODO: Flight SQL column metadata does not have this, should we add to the spec?
+  // GH-47853 TODO: use `ColumnMetadata` to get literal suffix after Flight SQL protocol
+  // adds support for it
+
+  // Flight SQL column metadata does not have literal suffix, empty string is returned
   return "";
 }
 
 std::string FlightSqlResultSetMetadata::GetLocalTypeName(int column_position) {
   ColumnMetadata metadata = GetMetadata(schema_->field(column_position - 1));
 
-  // TODO: Is local type name the same as type name?
+  // Local type name is for display purpose only.
+  // Return type name as local type name as Flight SQL protocol doesn't have support for
+  // local type name.
   return metadata.GetTypeName().ValueOrElse([] { return ""; });
 }
 
@@ -192,7 +200,7 @@ size_t FlightSqlResultSetMetadata::GetOctetLength(int column_position) {
 
   // Workaround to get the precision for Decimal and Numeric types, since server doesn't
   // return it currently.
-  // TODO: Use the server precision when its fixed.
+  // GH-47854 TODO: Use the server precision when its fixed.
   std::shared_ptr<DataType> arrow_type = field->type();
   if (arrow_type->id() == Type::DECIMAL128) {
     int32_t precision = util::GetDecimalTypePrecision(arrow_type);
@@ -220,7 +228,6 @@ Updatability FlightSqlResultSetMetadata::GetUpdatable(int column_position) {
 bool FlightSqlResultSetMetadata::IsAutoUnique(int column_position) {
   ColumnMetadata metadata = GetMetadata(schema_->field(column_position - 1));
 
-  // TODO: Is AutoUnique equivalent to AutoIncrement?
   return metadata.GetIsAutoIncrement().ValueOrElse([] { return false; });
 }
 

cpp/src/arrow/flight/sql/odbc/odbc_impl/flight_sql_statement_get_columns.cc

@@ -101,10 +101,9 @@ Result<std::shared_ptr<RecordBatch>> TransformInner(
     const auto& table_name = reader.GetTableName();
     const std::shared_ptr<Schema>& schema = reader.GetSchema();
     if (schema == nullptr) {
-      // TODO: Test and build the driver against a server that returns
+      // GH-46561 TODO: Test and build the driver against a server that returns
       // complex types columns with the children
       // types and handle the failure properly.
-      // https://github.com/apache/arrow/issues/46561
       continue;
     }
     for (int i = 0; i < schema->num_fields(); ++i) {

cpp/src/arrow/flight/sql/odbc/odbc_impl/get_info_cache.cc

@@ -316,8 +316,8 @@ bool GetInfoCache::LoadInfoFromServer() {
               std::string server_name(
                   reinterpret_cast<StringScalar*>(scalar->child_value().get())->view());
 
-              // TODO: Consider creating different properties in GetSqlInfo.
-              // TODO: Investigate if SQL_SERVER_NAME should just be the host
+              // GH-47855 TODO: Consider creating different properties in GetSqlInfo.
+              // GH-47856 TODO: Investigate if SQL_SERVER_NAME should just be the host
               // address as well. In JDBC, FLIGHT_SQL_SERVER_NAME is only used for
               // the DatabaseProductName.
               info_[SQL_SERVER_NAME] = server_name;

cpp/src/arrow/flight/sql/odbc/odbc_impl/json_converter.cc

@@ -221,7 +221,7 @@ class ScalarToJson : public ScalarVisitor {
   }
 
   Status Visit(const DurationScalar& scalar) override {
-    // TODO: Append TimeUnit on conversion
+    // GH-47857 TODO: Append TimeUnit on conversion
     return ConvertScalarToStringAndWrite(scalar, writer_);
   }
 

cpp/src/arrow/flight/sql/odbc/odbc_impl/json_converter_test.cc

@@ -164,7 +164,7 @@ TEST(ConvertToJson, MonthInterval) {
 }
 
 TEST(ConvertToJson, Duration) {
-  // TODO: Append TimeUnit on conversion
+  // GH-47857 TODO: Append TimeUnit on conversion
   ASSERT_EQ("\"123\"", ConvertToJson(DurationScalar(123, TimeUnit::SECOND)));
   ASSERT_EQ("\"123\"", ConvertToJson(DurationScalar(123, TimeUnit::MILLI)));
   ASSERT_EQ("\"123\"", ConvertToJson(DurationScalar(123, TimeUnit::MICRO)));

cpp/src/arrow/flight/sql/odbc/odbc_impl/odbc_connection.cc

@@ -233,8 +233,8 @@ SQLRETURN ODBCConnection::GetInfo(SQLUSMALLINT info_type, SQLPOINTER value,
     case SQL_COLUMN_ALIAS:
     case SQL_DBMS_NAME:
     case SQL_DBMS_VER:
-    case SQL_DRIVER_NAME:  // TODO: This should be the driver's filename and shouldn't
-                           // come from the SPI.
+    case SQL_DRIVER_NAME:  // GH-47858 TODO: This should be the driver's filename and
+                           // shouldn't come from the SPI.
     case SQL_DRIVER_VER:
     case SQL_SEARCH_PATTERN_ESCAPE:
     case SQL_SERVER_NAME: