Commit 60e8b58
refactor: remove support for Greenplum
Greenplum is no longer used by any customer and is being removed from the
backend (F1-2435), so the SDK no longer needs to model it.
Drops the hand-written `CatalogDataSourceGreenplum` and `GreenplumAttributes`
classes, their exports from `gooddata_sdk`, and the Greenplum example from the
data-source docs. The Greenplum test went with them: it was already inert,
sitting inside a `"""` block and referencing a `greenplum.yaml` cassette that
does not exist in the repo.
The generated client was updated the supported way rather than by hand. The
`GREENPLUM` enum entry was removed from the two source schemas it originates
from -- gooddata-metadata-client.json and gooddata-scan-client.json -- and
`gooddata-api-client/` was then regenerated with openapi-generator against the
merged schema. The regeneration produced exactly six deletions and no other
drift, which also confirms the checked-in client was in sync with the schemas.
The merged schemas/gooddata-api-client.json was edited line-wise instead of
being rebuilt through the Makefile's `jq -S -s` step. jq 1.6 does not preserve
number literals and rewrote unrelated values (2.6E+9 -> 2600000000, 1.0 -> 1)
across the file; the checked-in copy was produced with jq 1.7+, which does.
This is a breaking change for anyone importing either class by name, including
code that never connects to Greenplum, since the failure is at import time.
JIRA: F1-2435
risk: low
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>1 parent cbd27a1 commit 60e8b58
13 files changed
Lines changed: 0 additions & 71 deletions
File tree
- docs/content/en/latest/data/data-source
- gooddata-api-client/gooddata_api_client/model
- packages/gooddata-sdk
- src/gooddata_sdk
- catalog/data_source/entity_model
- tests/catalog
- schemas
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
154 | 154 | | |
155 | 155 | | |
156 | 156 | | |
157 | | - | |
158 | | - | |
159 | | - | |
160 | | - | |
161 | | - | |
162 | | - | |
163 | | - | |
164 | | - | |
165 | | - | |
166 | | - | |
167 | | - | |
168 | | - | |
169 | | - | |
170 | | - | |
171 | | - | |
172 | | - | |
173 | | - | |
174 | | - | |
175 | 157 | | |
176 | 158 | | |
177 | 159 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
76 | | - | |
77 | 76 | | |
78 | 77 | | |
79 | 78 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
70 | | - | |
71 | 70 | | |
72 | 71 | | |
73 | 72 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
74 | | - | |
75 | 74 | | |
76 | 75 | | |
77 | 76 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
74 | | - | |
75 | 74 | | |
76 | 75 | | |
77 | 76 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
81 | 81 | | |
82 | 82 | | |
83 | 83 | | |
84 | | - | |
85 | 84 | | |
86 | 85 | | |
87 | 86 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
74 | | - | |
75 | 74 | | |
76 | 75 | | |
77 | 76 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
47 | | - | |
48 | 47 | | |
49 | 48 | | |
50 | 49 | | |
| |||
54 | 53 | | |
55 | 54 | | |
56 | 55 | | |
57 | | - | |
58 | 56 | | |
59 | 57 | | |
60 | 58 | | |
| |||
Lines changed: 0 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
166 | 166 | | |
167 | 167 | | |
168 | 168 | | |
169 | | - | |
170 | | - | |
171 | | - | |
172 | | - | |
173 | | - | |
174 | 169 | | |
175 | 170 | | |
176 | 171 | | |
| |||
230 | 225 | | |
231 | 226 | | |
232 | 227 | | |
233 | | - | |
234 | | - | |
235 | | - | |
236 | | - | |
237 | | - | |
238 | | - | |
239 | 228 | | |
240 | 229 | | |
241 | 230 | | |
| |||
Lines changed: 0 additions & 22 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
769 | 769 | | |
770 | 770 | | |
771 | 771 | | |
772 | | - | |
773 | | - | |
774 | | - | |
775 | | - | |
776 | | - | |
777 | | - | |
778 | | - | |
779 | | - | |
780 | | - | |
781 | | - | |
782 | | - | |
783 | | - | |
784 | | - | |
785 | | - | |
786 | | - | |
787 | | - | |
788 | | - | |
789 | | - | |
790 | | - | |
791 | | - | |
792 | | - | |
793 | | - | |
794 | 772 | | |
795 | 773 | | |
796 | 774 | | |
| |||
0 commit comments