Suyash Patel comments

Repositories
Issues
Comments

Results 3 comments of


                                            Suyash Patel

Move all usage of legacy Apache commons libraries to the new ones

@Jackie-Jiang I would like to take this up. Thanks!

Move all usage of legacy Apache commons libraries to the new ones

@Jackie-Jiang Please review. @shounakmk219 I have made some changes in DataGenerationHelpers.java because I had to upgrade commons-lang -> commons-lang3. It would be helpful if you could take a quick look

Added API to check if 2 tables are co-located

On second thought these is something wrong with the implementation - we shouldn't be getting segment names and comparing them. My implementation works locally because ``` IdealState table2IdealState = _pinotHelixResourceManager.getHelixAdmin()...