{"comment":{"url":"https://api.github.com/repos/Teradata/presto/comments/18956795","html_url":"https://github.com/Teradata/presto/commit/9eeba7e72715793aa41a1de88a84559ed034b83b#commitcomment-18956795","id":18956795,"user":{"login":"sanjay990","id":7615206,"avatar_url":"https://avatars.githubusercontent.com/u/7615206?v=3","gravatar_id":"","url":"https://api.github.com/users/sanjay990","html_url":"https://github.com/sanjay990","followers_url":"https://api.github.com/users/sanjay990/followers","following_url":"https://api.github.com/users/sanjay990/following{/other_user}","gists_url":"https://api.github.com/users/sanjay990/gists{/gist_id}","starred_url":"https://api.github.com/users/sanjay990/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sanjay990/subscriptions","organizations_url":"https://api.github.com/users/sanjay990/orgs","repos_url":"https://api.github.com/users/sanjay990/repos","events_url":"https://api.github.com/users/sanjay990/events{/privacy}","received_events_url":"https://api.github.com/users/sanjay990/received_events","type":"User","site_admin":false},"position":112,"line":112,"path":"presto-product-tests/src/main/java/com/facebook/presto/tests/jdbc/LdapTests.java","commit_id":"9eeba7e72715793aa41a1de88a84559ed034b83b","created_at":"2016-09-09T02:50:34Z","updated_at":"2016-09-09T02:50:34Z","body":"Shouldn't we remove **LDAP** from these tests and just have **SIMBA_JDBC**\r\nIf I do `-g LDAP`, these tests would be called even though I am not using Simba JDBC drivers and just need to run CLI Ldap tests. The group **LDAP** is also used for CLI tests.\r\n\r\nhttps://github.com/Teradata/presto/blob/50e45a0b3dde5f13c253315792b5e593f4e8602f/presto-product-tests/src/main/java/com/facebook/presto/tests/cli/PrestoLdapCliTests.java#L99"}}
{"actor":{"display_login":"sanjay990"}}