npawar commented on issue #4103: Add validations to table config before 
updating a table config
URL: https://github.com/apache/incubator-pinot/pull/4103#issuecomment-482180251
 
 
   > Is the table config modification command covered in an integration test? 
If not, can you add it to one of the existing integration tests? It may also be 
useful to do some basic sanity checks (manual) on table config changes. thanks
   
   It is covered in a few of them. Which is why I had to correct the usage of 
the update in one of the tests.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to