Fixed timestamp extraction

abhinav04sharma · facebook-github-bot · commit e1cb53ba0f95 · 2018-08-28T12:56:42.000-07:00
Summary: The JSON key was changes from "timestamps" to "ts" to save space in binlogs. We forgot to make the key change in extraction function. Reviewed By: yashtc Differential Revision: D9544555 fbshipit-source-id: 6626b21
diff --git a/sql/log_event.cc b/sql/log_event.cc
@@ -15078,7 +15078,8 @@ inline ulonglong Rows_query_log_event::extract_last_timestamp() const
     return 0;
   }
 
-  auto timestamps= pt.get_child("timestamps", boost::property_tree::ptree());
+  auto timestamps= pt.get_child("ts", boost::property_tree::ptree());
+  DBUG_ASSERT(!timestamps.empty());
   return timestamps.empty() ? 0 :
          timestamps.back().second.get_value<ulonglong>();
 }

Original file line number	Diff line number	Diff line change
`@@ -15078,7 +15078,8 @@ inline ulonglong Rows_query_log_event::extract_last_timestamp() const`
`15078`	`15078`	`return 0;`
`15079`	`15079`	`}`
`15080`	`15080`
`15081`		`- auto timestamps= pt.get_child("timestamps", boost::property_tree::ptree());`
	`15081`	`+ auto timestamps= pt.get_child("ts", boost::property_tree::ptree());`
	`15082`	`+ DBUG_ASSERT(!timestamps.empty());`
`15082`	`15083`	`return timestamps.empty() ? 0 :`
`15083`	`15084`	`timestamps.back().second.get_value<ulonglong>();`
`15084`	`15085`	`}`