November 2014 - teiid-issues - Jboss List Archives

[JBoss JIRA] (TEIID-3193) Teiid not removing/setting the correct TCCL before calling getMetadata on translator

by Steven Hawkins (JIRA)

[ https://issues.jboss.org/browse/TEIID-3193?page=com.atlassian.jira.plugin... ] Steven Hawkins commented on TEIID-3193: --------------------------------------- > Otherwise we would to handle them in each translator. (we could consider this) Don't seem necessary yet as JCA should account for this. > I am drawing blank on Authorization, for JAAS we do not have direct access to the LoginModule, but I could set TCCL to null before SessionManager's call to Login method? I don't think we have to worry about JAAS either, just the interfaces where we load the implementation via modules. So that would be PolicyDecider/AuthorizationValidator. Our other extension points would be UDF/user defined aggregate functions and MetadataRepositories. > Teiid not removing/setting the correct TCCL before calling getMetadata on translator > ------------------------------------------------------------------------------------ > > Key: TEIID-3193 > URL: https://issues.jboss.org/browse/TEIID-3193 > Project: Teiid > Issue Type: Bug > Components: Common > Affects Versions: 8.8 > Reporter: Michael Farwell > Assignee: Ramesh Reddy > Fix For: 8.9 > > > Ramesh helped me debug a class loading issue that I'm experiencing with a custom translator and asked me to open this ticket based on his findings. The forum link is specified below and contains all of the necessary information. > Ramesh's words: "OK, I believe the geoTools is using the TCCL to load the schema, thus the error. However, the error is on the Teiid side not correctly removing/setting the correct TCCL before we call "getMetadata" call on the translator. You can add JIRA for this we will fix it." -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3194) Fix tab/spaces in codebase

by Tom Arnold (JIRA)

[ https://issues.jboss.org/browse/TEIID-3194?page=com.atlassian.jira.plugin... ] Tom Arnold commented on TEIID-3194: ----------------------------------- That would be good. I've noticed the same tab/spaces issues in many places of the code. I've updated the ticket description to broaden the scope. Checkstyle plugin could be added first and then used to track down and fix the issues. > Fix tab/spaces in codebase > -------------------------- > > Key: TEIID-3194 > URL: https://issues.jboss.org/browse/TEIID-3194 > Project: Teiid > Issue Type: Task > Components: Build/Kits > Reporter: Tom Arnold > Assignee: Steven Hawkins > Priority: Minor > > Tabs/spaces are mixed, causing readability issues in the code and some of the build scripts. This is visible on GitHub. For example: > https://github.com/teiid/teiid/blob/master/pom.xml -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3194) Fix tab/spaces in codebase

by Tom Arnold (JIRA)

[ https://issues.jboss.org/browse/TEIID-3194?page=com.atlassian.jira.plugin... ] Tom Arnold updated TEIID-3194: ------------------------------ Component/s: (was: Build/Kits) > Fix tab/spaces in codebase > -------------------------- > > Key: TEIID-3194 > URL: https://issues.jboss.org/browse/TEIID-3194 > Project: Teiid > Issue Type: Task > Reporter: Tom Arnold > Assignee: Steven Hawkins > Priority: Minor > > Tabs/spaces are mixed, causing readability issues in the code and some of the build scripts. This is visible on GitHub. For example: > https://github.com/teiid/teiid/blob/master/pom.xml -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3194) Fix tab/spaces in codebase

by Tom Arnold (JIRA)

[ https://issues.jboss.org/browse/TEIID-3194?page=com.atlassian.jira.plugin... ] Tom Arnold updated TEIID-3194: ------------------------------ Component/s: Build/Kits > Fix tab/spaces in codebase > -------------------------- > > Key: TEIID-3194 > URL: https://issues.jboss.org/browse/TEIID-3194 > Project: Teiid > Issue Type: Task > Components: Build/Kits > Reporter: Tom Arnold > Assignee: Steven Hawkins > Priority: Minor > > Tabs/spaces are mixed, causing readability issues in the code and some of the build scripts. This is visible on GitHub. For example: > https://github.com/teiid/teiid/blob/master/pom.xml -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3194) Fix tab/spaces in codebase

by Tom Arnold (JIRA)

[ https://issues.jboss.org/browse/TEIID-3194?page=com.atlassian.jira.plugin... ] Tom Arnold updated TEIID-3194: ------------------------------ Summary: Fix tab/spaces in codebase (was: Fix indentation of Teiid pom.xml) Description: Tabs/spaces are mixed, causing readability issues in the code and some of the build scripts. This is visible on GitHub. For example: https://github.com/teiid/teiid/blob/master/pom.xml was: Tabs/spaces are mixed causing readability issues. This is visible on GitHub. Should be a quick fix (right-click Format in Netbeans). https://github.com/teiid/teiid/blob/master/pom.xml > Fix tab/spaces in codebase > -------------------------- > > Key: TEIID-3194 > URL: https://issues.jboss.org/browse/TEIID-3194 > Project: Teiid > Issue Type: Task > Components: Build/Kits > Reporter: Tom Arnold > Assignee: Steven Hawkins > Priority: Minor > > Tabs/spaces are mixed, causing readability issues in the code and some of the build scripts. This is visible on GitHub. For example: > https://github.com/teiid/teiid/blob/master/pom.xml -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3193) Teiid not removing/setting the correct TCCL before calling getMetadata on translator

by Ramesh Reddy (JIRA)

[ https://issues.jboss.org/browse/TEIID-3193?page=com.atlassian.jira.plugin... ] Ramesh Reddy commented on TEIID-3193: ------------------------------------- For metadata, handling in the NativeMetadataRepository should take care it. For capabilities, I am thinking of wrapping the capabilities call in ConnectorManager class. Resource Adapter is loaded through JCA, since Teiid does not make any direct calls into this API (all through translator), I am thinking it should be OK, if we handle calls like "getConnection" and "close" in the Connector Manager. At worst it will have Translator's class loader as TCCL. Otherwise we would to handle them in each translator. (we could consider this) I am drawing blank on Authorization, for JAAS we do not have direct access to the LoginModule, but I could set TCCL to null before SessionManager's call to Login method? > Teiid not removing/setting the correct TCCL before calling getMetadata on translator > ------------------------------------------------------------------------------------ > > Key: TEIID-3193 > URL: https://issues.jboss.org/browse/TEIID-3193 > Project: Teiid > Issue Type: Bug > Components: Common > Affects Versions: 8.8 > Reporter: Michael Farwell > Assignee: Ramesh Reddy > Fix For: 8.9 > > > Ramesh helped me debug a class loading issue that I'm experiencing with a custom translator and asked me to open this ticket based on his findings. The forum link is specified below and contains all of the necessary information. > Ramesh's words: "OK, I believe the geoTools is using the TCCL to load the schema, thus the error. However, the error is on the Teiid side not correctly removing/setting the correct TCCL before we call "getMetadata" call on the translator. You can add JIRA for this we will fix it." -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3193) Teiid not removing/setting the correct TCCL before calling getMetadata on translator

by Steven Hawkins (JIRA)

[ https://issues.jboss.org/browse/TEIID-3193?page=com.atlassian.jira.plugin... ] Steven Hawkins commented on TEIID-3193: --------------------------------------- ConnectorWork will cover the runtime operations, but there would still be the metadata and capabilities calls. Resource Adapter class loading should already be handled by JCA correct? > Teiid not removing/setting the correct TCCL before calling getMetadata on translator > ------------------------------------------------------------------------------------ > > Key: TEIID-3193 > URL: https://issues.jboss.org/browse/TEIID-3193 > Project: Teiid > Issue Type: Bug > Components: Common > Affects Versions: 8.8 > Reporter: Michael Farwell > Assignee: Ramesh Reddy > Fix For: 8.9 > > > Ramesh helped me debug a class loading issue that I'm experiencing with a custom translator and asked me to open this ticket based on his findings. The forum link is specified below and contains all of the necessary information. > Ramesh's words: "OK, I believe the geoTools is using the TCCL to load the schema, thus the error. However, the error is on the Teiid side not correctly removing/setting the correct TCCL before we call "getMetadata" call on the translator. You can add JIRA for this we will fix it." -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3193) Teiid not removing/setting the correct TCCL before calling getMetadata on translator

by Ramesh Reddy (JIRA)

[ https://issues.jboss.org/browse/TEIID-3193?page=com.atlassian.jira.plugin... ] Ramesh Reddy commented on TEIID-3193: ------------------------------------- Should I just add a proxy around ConnectorWork instance for translators? That seems to be best. > Teiid not removing/setting the correct TCCL before calling getMetadata on translator > ------------------------------------------------------------------------------------ > > Key: TEIID-3193 > URL: https://issues.jboss.org/browse/TEIID-3193 > Project: Teiid > Issue Type: Bug > Components: Common > Affects Versions: 8.8 > Reporter: Michael Farwell > Assignee: Ramesh Reddy > Fix For: 8.9 > > > Ramesh helped me debug a class loading issue that I'm experiencing with a custom translator and asked me to open this ticket based on his findings. The forum link is specified below and contains all of the necessary information. > Ramesh's words: "OK, I believe the geoTools is using the TCCL to load the schema, thus the error. However, the error is on the Teiid side not correctly removing/setting the correct TCCL before we call "getMetadata" call on the translator. You can add JIRA for this we will fix it." -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3193) Teiid not removing/setting the correct TCCL before calling getMetadata on translator

by Steven Hawkins (JIRA)

[ https://issues.jboss.org/browse/TEIID-3193?page=com.atlassian.jira.plugin... ] Steven Hawkins commented on TEIID-3193: --------------------------------------- For consistency this would need to be on all Translator, Resource Adapter, Authorization, etc. calls where we have extension points. > Teiid not removing/setting the correct TCCL before calling getMetadata on translator > ------------------------------------------------------------------------------------ > > Key: TEIID-3193 > URL: https://issues.jboss.org/browse/TEIID-3193 > Project: Teiid > Issue Type: Bug > Components: Common > Affects Versions: 8.8 > Reporter: Michael Farwell > Assignee: Ramesh Reddy > Fix For: 8.9 > > > Ramesh helped me debug a class loading issue that I'm experiencing with a custom translator and asked me to open this ticket based on his findings. The forum link is specified below and contains all of the necessary information. > Ramesh's words: "OK, I believe the geoTools is using the TCCL to load the schema, thus the error. However, the error is on the Teiid side not correctly removing/setting the correct TCCL before we call "getMetadata" call on the translator. You can add JIRA for this we will fix it." -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

[JBoss JIRA] (TEIID-3193) Teiid not removing/setting the correct TCCL before calling getMetadata on translator

by Ramesh Reddy (JIRA)

[ https://issues.jboss.org/browse/TEIID-3193?page=com.atlassian.jira.plugin... ] Ramesh Reddy updated TEIID-3193: -------------------------------- Fix Version/s: 8.9 (was: 8.10) Assignee: Ramesh Reddy (was: Steven Hawkins) > Teiid not removing/setting the correct TCCL before calling getMetadata on translator > ------------------------------------------------------------------------------------ > > Key: TEIID-3193 > URL: https://issues.jboss.org/browse/TEIID-3193 > Project: Teiid > Issue Type: Bug > Components: Common > Affects Versions: 8.8 > Reporter: Michael Farwell > Assignee: Ramesh Reddy > Fix For: 8.9 > > > Ramesh helped me debug a class loading issue that I'm experiencing with a custom translator and asked me to open this ticket based on his findings. The forum link is specified below and contains all of the necessary information. > Ramesh's words: "OK, I believe the geoTools is using the TCCL to load the schema, thus the error. However, the error is on the Teiid side not correctly removing/setting the correct TCCL before we call "getMetadata" call on the translator. You can add JIRA for this we will fix it." -- This message was sent by Atlassian JIRA (v6.3.1#6329)

10 years, 1 month

1
0
0 / 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

teiid-issues November 2014