Yifan Zhang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18152 )

Change subject: KUDU-3344: catalog manager clean up metadata for deleted 
tables/tablets
......................................................................


Patch Set 7:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/18152/6/src/kudu/master/catalog_manager.cc
File src/kudu/master/catalog_manager.cc:

http://gerrit.cloudera.org:8080/#/c/18152/6/src/kudu/master/catalog_manager.cc@393
PS6, Line 393: enable_chunked_tablet_writes,
> Ah, and I also made a typo :(
Thanks for the correction!

I also thought about replacing the flag 
'catalog_manager_enable_chunked_tablet_reports' with 
'enable_chuncked_tablet_writes' because they have similar intention. Is it 
necessary to distinguish between them and let users to configure them 
respectively.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idefa2ee2f5108ba913fe0057a4061c3c28351547
Gerrit-Change-Number: 18152
Gerrit-PatchSet: 7
Gerrit-Owner: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: yejiabao <[email protected]>
Gerrit-Comment-Date: Mon, 21 Feb 2022 12:12:05 +0000
Gerrit-HasComments: Yes

Reply via email to