Metatron-discovery: change the connection icon to use static resources

Created on 5 Mar 2019 · 8Comments · Source: metatron-app/metatron-discovery

Is your feature request related to a problem? Please describe.
To extend the data connection later, change the connection icon to use static resources.
Also add an icon for the DB below.

DB
redshift, oracle, tibero

And we need to define icon size for each usage.

Data conenction create
Workbench LNB
Schema Browser
Worspace

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@datasource

Source

ufoscw

All 8 comments

@koeun222
oracle, tibero에 대한 아이콘 전달 부탁드립니다.

@eltriny
추후 추가될 커넥션 정보 api의 response sample 보내드립니다.
scope : 임베드, 익스텐션
name : 커넥션 타입명
implementor : 커넥션 구분자
iconResource1, iconResource2, iconResource3, iconResource4 : icon static resource 경로
-- 우선 4가지 리소스 경로를 정의했는데요. 현재 아이콘은 몇가지 유형을 사용중인가요?

inputSpec : 커넥션 생성시 필수값 여부에 대한 spec
-- 우선 authenticationType, database, sid, catalog 4가지 속성에 대해 노출할지에 대한 코드를 구현해주시면 됩니다.

[
   {
      "scope":"EMBEDDED",
      "name":"Hive",
      "inputSpec":{
         "implementor":"MANDATORY",
         "authenticationType":"MANDATORY",
         "url":"OPTIONAL",
         "options":"OPTIONAL",
         "hostname":"OPTIONAL",
         "port":"OPTIONAL",
         "database":"NONE",
         "sid":"NONE",
         "catalog":"NONE",
         "properties":"OPTIONAL",
         "username":"OPTIONAL",
         "password":"OPTIONAL"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"HIVE",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EMBEDDED",
      "name":"MySQL",
      "inputSpec":{
         "implementor":"MANDATORY",
         "authenticationType":"MANDATORY",
         "url":"OPTIONAL",
         "options":"OPTIONAL",
         "hostname":"OPTIONAL",
         "port":"OPTIONAL",
         "database":"NONE",
         "sid":"NONE",
         "catalog":"NONE",
         "properties":"OPTIONAL",
         "username":"OPTIONAL",
         "password":"OPTIONAL"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"MYSQL",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EMBEDDED",
      "name":"Presto",
      "inputSpec":{
         "implementor":"MANDATORY",
         "authenticationType":"MANDATORY",
         "url":"OPTIONAL",
         "options":"OPTIONAL",
         "hostname":"OPTIONAL",
         "port":"OPTIONAL",
         "database":"NONE",
         "sid":"NONE",
         "catalog":"MANDATORY",
         "properties":"OPTIONAL",
         "username":"OPTIONAL",
         "password":"OPTIONAL"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"PRESTO",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EMBEDDED",
      "name":"PostgreSQL",
      "inputSpec":{
         "implementor":"MANDATORY",
         "authenticationType":"MANDATORY",
         "url":"OPTIONAL",
         "options":"OPTIONAL",
         "hostname":"OPTIONAL",
         "port":"OPTIONAL",
         "database":"MANDATORY",
         "sid":"NONE",
         "catalog":"NONE",
         "properties":"OPTIONAL",
         "username":"OPTIONAL",
         "password":"OPTIONAL"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"POSTGRESQL",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EMBEDDED",
      "name":"Druid",
      "inputSpec":{
         "implementor":"MANDATORY",
         "authenticationType":"NONE",
         "url":"OPTIONAL",
         "options":"OPTIONAL",
         "hostname":"OPTIONAL",
         "port":"OPTIONAL",
         "database":"NONE",
         "sid":"NONE",
         "catalog":"NONE",
         "properties":"OPTIONAL",
         "username":"NONE",
         "password":"NONE"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"DRUID",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EXTENSION",
      "name":"Tibero",
      "inputSpec":{
         "implementor":"MANDATORY",
         "authenticationType":"MANDATORY",
         "url":"OPTIONAL",
         "options":"OPTIONAL",
         "hostname":"OPTIONAL",
         "port":"OPTIONAL",
         "database":"NONE",
         "sid":"MANDATORY",
         "catalog":"NONE",
         "properties":"OPTIONAL",
         "username":"OPTIONAL",
         "password":"OPTIONAL"
      },
      "iconResource4":"/resources/jdbc/icon4.png",
      "iconResource3":"/resources/jdbc/icon3.png",
      "implementor":"TIBERO",
      "iconResource2":"/resources/jdbc/icon1.png",
      "iconResource1":"/resources/jdbc/icon1.png"
   }
]

ufoscw on 15 Mar 2019

oracle, tibero에 대한 아이콘 전달 부탁드립니다.

@ufoscw 위 아이콘 기존에 퍼블로 전달하였었습니다- (이곳으로 전달 필요하시면 알려주세요)
참고로 DB 타입관련 전달된 아이콘 아래 이미지로 첨부합니다

koeun222 on 17 Mar 2019

@eltriny
Oracle과 Tibero 아이콘 저한테 전달 부탁드려요.

ufoscw on 19 Mar 2019

@ufoscw 작업중 문의드립니다.

Q1. inputSpec 프로퍼티들의 value는 아래와 같은 형태만 존재하나요?

MANDATORY = 필수값
OPTIONAL = 옵션 (보통 URL 선택 해제시 사용)
NONE = 사용하지 않음

Q2. PostgreSQL에 대해 문의입니다.

{
      "scope":"EMBEDDED",
      "name":"PostgreSQL",
      "inputSpec":{
         "implementor":"MANDATORY",
         "authenticationType":"MANDATORY",
         "url":"OPTIONAL",
         "options":"OPTIONAL",
         "hostname":"OPTIONAL",
         "port":"OPTIONAL",
         "database":"MANDATORY",
         "sid":"NONE",
         "catalog":"NONE",
         "properties":"OPTIONAL",
         "username":"OPTIONAL",
         "password":"OPTIONAL"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"POSTGRESQL",
      "iconResource2":null,
      "iconResource1":null
   },

위에서 database가 MANDATORY로 되어있는데 현재 화면에서는
아래와 같이 표기되고 있었습니다.
스크린샷 2019-03-20 오후 8 58 34

이러면 database가 OPTIONAL로 표기되어야 하는게 아닌가요?

Q3. DRUID에서 authenticationType가 NONE으로 되어있는데요
그러면 생성시에 authenticationType 값 자체를 넘기지 않아야되나요?

brandon-wonjune on 20 Mar 2019

@brandon-wonjune
Q1 : input 요소들간의 관계를 spec으로 정의하기에는 너무 복잡해질것 같네요.
조금 단순하게 작업하는게 좋을것 같습니다.
inputSpec 적용 대상은 database, sid, catalog, username, password, authenticationType 만으로 정의하도록 하겠습니다.
host, port, url, properties는 항상 노출하도록 하며 host, port <-> url은 양자택일의 기존 로직을 유지해주시면 됩니다.

value는 아래값만 존재하며 각각의 의미는
MANDATORY = 화면 노출 O, 필수값 체크 O
OPTIONAL = 화면 노출 O, 필수값 체크 X
NONE = 화면 노출 X, 필수값 체크 X

Sample

[
   {
      "scope":"EMBEDDED",
      "name":"Hive",
      "inputSpec":{
         "authenticationType":"MANDATORY",
         "database":"NONE",
         "sid":"NONE",
         "catalog":"NONE",
         "username":"MANDATORY",
         "password":"MANDATORY"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"HIVE",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EMBEDDED",
      "name":"MySQL",
      "inputSpec":{
         "authenticationType":"MANDATORY",
         "database":"NONE",
         "sid":"NONE",
         "catalog":"NONE",
         "username":"MANDATORY",
         "password":"MANDATORY"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"MYSQL",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EMBEDDED",
      "name":"Presto",
      "inputSpec":{
         "authenticationType":"MANDATORY",
         "database":"NONE",
         "sid":"NONE",
         "catalog":"MANDATORY",
         "username":"MANDATORY",
         "password":"MANDATORY"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"PRESTO",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EMBEDDED",
      "name":"PostgreSQL",
      "inputSpec":{
         "authenticationType":"MANDATORY",
         "database":"MANDATORY",
         "sid":"NONE",
         "catalog":"NONE",
         "username":"MANDATORY",
         "password":"MANDATORY"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"POSTGRESQL",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EMBEDDED",
      "name":"Druid",
      "inputSpec":{
         "authenticationType":"NONE",
         "database":"NONE",
         "sid":"NONE",
         "catalog":"NONE",
         "username":"NONE",
         "password":"NONE"
      },
      "iconResource4":null,
      "iconResource3":null,
      "implementor":"DRUID",
      "iconResource2":null,
      "iconResource1":null
   },
   {
      "scope":"EXTENSION",
      "name":"Tibero",
      "inputSpec":{
         "authenticationType":"MANDATORY",
         "database":"NONE",
         "sid":"MANDATORY",
         "catalog":"NONE",
         "username":"MANDATORY",
         "password":"MANDATORY"
      },
      "iconResource4":"/resources/jdbc/icon4.png",
      "iconResource3":"/resources/jdbc/icon3.png",
      "implementor":"TIBERO",
      "iconResource2":"/resources/jdbc/icon1.png",
      "iconResource1":"/resources/jdbc/icon1.png"
   }
]

Q2. "URL만" 체크시에는 host, port, sid, catalog, database 는 필수값 체크 로직 예외처리해주세요. 기존 로직 그대로 적용하시면 됩니다.

Q3. 기본값인 MANUAL로 넘기면 됩니다.

ufoscw on 20 Mar 2019

👍1

@brandon-wonjune

이후 작업은 https://github.com/metatron-app/metatron-discovery/tree/f%231295-apply_pf4j_framework 브랜치에서 해주시기 바랍니다.
https://github.com/metatron-app/metatron-discovery/commit/581504577964609fd19e5a4caae8330dc90d07f4 커밋은 위 브랜치로 체리픽 하도록 하겠습니다.

위 브랜치에 extension connection이 포함된 API가 추가되었습니다.

(GET) /api/extensions/connection

워크벤치 테이블 목록 API 요청시 sid 파라미터가 누락되고 있습니다. 확인바랍니다.

ufoscw on 22 Mar 2019

👍1

@brandon-wonjune
워크벤치와 커넥션 생성, 워크스페이스에서 사용할 오라클, 티베로 이미지 파일을 저한테 전달 바랍니다.

ufoscw on 22 Mar 2019

@brandon-wonjune
커밋 메시지의 이슈번호는 #1295 로 통일하도록 하겠습니다.

ufoscw on 22 Mar 2019

👍1

Was this page helpful?

0 / 5 - 0 ratings

Related issues

Out of bar chart x-axis range in combined chart.

Taehui · 3Comments

URI Format Adress Not Seen Consistently in the Data Connection List

Cazen · 3Comments

Users should be able to search for hidden value in filter widget

deidera08 · 9Comments

For line and combine charts, the line thickness will be changeable.

AnnieHwang · 9Comments

Enable to edit column's logical information of datasource and sync with column of metadata

deidera08 · 5Comments