{"action":"created","issue":{"url":"https://api.github.com/repos/facebook/presto/issues/4317","labels_url":"https://api.github.com/repos/facebook/presto/issues/4317/labels{/name}","comments_url":"https://api.github.com/repos/facebook/presto/issues/4317/comments","events_url":"https://api.github.com/repos/facebook/presto/issues/4317/events","html_url":"https://github.com/facebook/presto/issues/4317","id":126197096,"number":4317,"title":"Connector properties are not propagated to Connectors in JdbcTable","user":{"login":"sopel39","id":5989165,"avatar_url":"https://avatars.githubusercontent.com/u/5989165?v=3","gravatar_id":"","url":"https://api.github.com/users/sopel39","html_url":"https://github.com/sopel39","followers_url":"https://api.github.com/users/sopel39/followers","following_url":"https://api.github.com/users/sopel39/following{/other_user}","gists_url":"https://api.github.com/users/sopel39/gists{/gist_id}","starred_url":"https://api.github.com/users/sopel39/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sopel39/subscriptions","organizations_url":"https://api.github.com/users/sopel39/orgs","repos_url":"https://api.github.com/users/sopel39/repos","events_url":"https://api.github.com/users/sopel39/events{/privacy}","received_events_url":"https://api.github.com/users/sopel39/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-01-12T14:55:48Z","updated_at":"2016-01-12T16:58:34Z","closed_at":null,"body":"I have added support for decimal on `BigInteger` in TPCH connector. To do so, I have added additional schemas. Some of the schemas are `sf*_custom_decimal`. This schema contains TPCH tables with proper decimals that have precision and scale. Precision and scale can be specified by session properties (`custom_decimal_precision` and `custom_decimal_scale`). The problem is that those properties are not propagated to connector when using system tables (e.g. `ColumnJdbcTable`). This is because in `cursor` method `connectorSession` is converted to `Session` object using `FilterUtil.toSession` method. This erases any connector specific properties. I understand that `Session` object is required for further calls to `Metadata`. I tried to think of some elegant way of overcoming or avoiding this issue, but I think that requires some serious refactoring. What would be you suggestion or ideas?\r\n\r\nFYI: @electrum "},"comment":{"url":"https://api.github.com/repos/facebook/presto/issues/comments/170974589","html_url":"https://github.com/facebook/presto/issues/4317#issuecomment-170974589","issue_url":"https://api.github.com/repos/facebook/presto/issues/4317","id":170974589,"user":{"login":"electrum","id":9230,"avatar_url":"https://avatars.githubusercontent.com/u/9230?v=3","gravatar_id":"","url":"https://api.github.com/users/electrum","html_url":"https://github.com/electrum","followers_url":"https://api.github.com/users/electrum/followers","following_url":"https://api.github.com/users/electrum/following{/other_user}","gists_url":"https://api.github.com/users/electrum/gists{/gist_id}","starred_url":"https://api.github.com/users/electrum/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/electrum/subscriptions","organizations_url":"https://api.github.com/users/electrum/orgs","repos_url":"https://api.github.com/users/electrum/repos","events_url":"https://api.github.com/users/electrum/events{/privacy}","received_events_url":"https://api.github.com/users/electrum/received_events","type":"User","site_admin":false},"created_at":"2016-01-12T16:58:34Z","updated_at":"2016-01-12T16:58:34Z","body":"TPCH already has dynamic tables. You can query sf123 for example.\n"}}