Merge pull request #59 from beclab/feat/reconstruct-train-rank

fix:  fix findBySourceAndRankTime method wrong for rank

Author: bleachzou3

train-rank/src/knowledgebase_api.cpp

@@ -2345,7 +2345,7 @@ namespace knowledgebase
   {
     http_client *current_client = HttpClientSingleton::get_instance();
     http_client &client = *current_client;
-    std::string current_suffix = std::string(RECOMMEND_TRACE_INFO_API_SUFFIX) + "/findBySourceAndRankTime?source=" + source + "&rank_time" + std::to_string(rank_time);
+    std::string current_suffix = std::string(RECOMMEND_TRACE_INFO_API_SUFFIX) + "/findBySourceAndRankTime?source=" + source + "&rank_time=" + std::to_string(rank_time);
     LOG(DEBUG) << "current_suffix " << current_suffix << std::endl;
 
     std::optional<RecommendTraceInfo> option_info = std::nullopt;

train-rank/test/knowledge_api_test.cpp

@@ -603,8 +603,8 @@ TEST(KnowledgeApiTest, findRecommendTraceInfoByRankTimeAndSource)
   // --gtest_filter=KnowledgeApiTest.findRecommendTraceInfoByRankTimeAndSource
   initDevelop();
   init_log();
-  std::string source = "testsource";
-  int64_t rank_time = 1711080226;
+  std::string source = "r4wechat";
+  int64_t rank_time = 1735832592;
   std::optional<RecommendTraceInfo> recommend_trace_info = knowledgebase::findRecommendTraceInfoByRankTimeAndSource(source, rank_time);
   if (recommend_trace_info != std::nullopt)
   {
@@ -618,11 +618,11 @@ TEST(KnowledgeApiTest, findAllRecomendTraceInfoRankTimesBySource)
   // --gtest_filter=KnowledgeApiTest.findAllRecomendTraceInfoRankTimesBySource
   initDevelop();
   init_log();
-  std::string source = "testsource";
+  std::string source = "r4techbiz";
   std::vector<int> rank_times = knowledgebase::findAllRecomendTraceInfoRankTimesBySource(source);
   for (auto current : rank_times)
   {
-    std::cout << current << " ";
+    std::cout << current << " " << std::endl;
   }
   std::cout << std::endl;
 }