Yingchun Lai has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/19163 )

Change subject: [KUDU-3412][Tools][Bugfix] Fix core dump of tablet copier
......................................................................

[KUDU-3412][Tools][Bugfix] Fix core dump of tablet copier

The function: CopyTablets() of class TabletCopier creates
a thread to obtain copy process periodically. Every 10s,
it will log the tablet id and tablet status. But fake_replica
does not have a tablet id, here will core dump.

Therefore, this patch uses map<string, TabletReplica*>
copying_replicas_by_tablet_id to store the tablet id and
fake_replica.

Change-Id: Iaa2107d3c04c9f3e850d33dbf420ebb62edc98d8
Reviewed-on: http://gerrit.cloudera.org:8080/19163
Reviewed-by: Yingchun Lai <[email protected]>
Tested-by: Yingchun Lai <[email protected]>
---
M src/kudu/tools/tool_action_local_replica.cc
1 file changed, 9 insertions(+), 7 deletions(-)

Approvals:
  Yingchun Lai: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/19163
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa2107d3c04c9f3e850d33dbf420ebb62edc98d8
Gerrit-Change-Number: 19163
Gerrit-PatchSet: 7
Gerrit-Owner: Wang Xixu <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Wang Xixu <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>

Reply via email to