[GitHub] jmark99 commented on issue #337: ACCUMULO-4732 No APIs to configure iterators or locality groups for new tables

[GitBox <gi...@apache.org>]

jmark99 commented on issue #337: ACCUMULO-4732 No APIs to configure iterators or locality groups for new tables
URL: https://github.com/apache/accumulo/pull/337#issuecomment-352031648
 
 
   I think most comments have been addressed. Let me know if there are more or if I missed some. 
   
   I do have a concern about one of the tests that I hope someone can assist me with. The 'testSetPropertiesAndGroups' test sets two generic properties and a locality group prior to creating the table. After creation I call conn.tableOperations().getProperties to retrieve the properties of the table. I check for the properties of the locality groups and I have tried checking for the generic properties as well. The locality group properties are retrieved without issue. But, the generic properties are not returned. I expected that I would see them as well. Am I missing some step in the setProperties process or misunderstanding the use of the NewTableConfiguration.setProperties method? I have followed through the code and  verified that when the CreateTable() constructor is eventually called the generic properties are still in the supplied options Map. Suggestions or clarification of my expectations would be appreciated.

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


With regards,
Apache Git Services