[IOTDB-1730] client-cpp, enhance session::insertTablet() etc's performance

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.13.0
• Component/s: Client/Others
• Labels: pull-request-available
• Assignee: Bo Zhang
• Reporter: Bo Zhang
• Created: 24/Sep/21 06:32
• Updated: 25/Sep/21 07:22
• Resolved: 25/Sep/21 07:22

Description

1) session::insertTablet() and session::insertTablets() are main API  used by customer.

2)  custeomer's client oftern inserts 5000~50000 rows per 1 tablet, 

3)  Call stack example: session::insertTablet()  => Session::sortTablet() => Session::sortIndexByTimestamp()

so these functions' efficiency is sensitive.  But current sortIndexByTimestamp() use insert sort Algorithm, which causes 100~3000 times efficiency lossing.

 

On the other hande,  Session::insertTablets() has another useless code that reduces performance.

void Session::insertTablets(map<string, Tablet *> &tablets, bool sorted) {
TSInsertTabletsReq request;
request.__set_sessionId(sessionId);

for (const auto &item: tablets) {
if (sorted) {
if (!checkSorted(*(item.second))) {
throw BatchExecutionException("Times in Tablet are not in ascending order");
}
} else {
sortTablet(*(item.second));    //here, it is wastful to let map search again.
{color:#de350b}}

Issue Links

• links to: GitHub Pull Request #4022