Skip to content

Commit 4222e30

Browse files
committed
Address feedback
1 parent 3ed5d15 commit 4222e30

File tree

5 files changed

+5
-26
lines changed

5 files changed

+5
-26
lines changed

x-pack/platform/plugins/shared/streams/common/constants.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,6 @@ export const STREAMS_UI_PRIVILEGES = {
2525
show: 'show',
2626
} as const;
2727

28-
export const FAILURE_STORE_PRIVILEGE = 'read_failure_store';
29-
3028
/**
3129
* Tiered features
3230
*/

x-pack/platform/plugins/shared/streams/server/routes/internal/streams/crud/route.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import type { estypes } from '@elastic/elasticsearch';
1111
import type { ClassicIngestStreamEffectiveLifecycle } from '@kbn/streams-schema';
1212
import { Streams } from '@kbn/streams-schema';
1313
import { processAsyncInChunks } from '../../../../utils/process_async_in_chunks';
14-
import { STREAMS_API_PRIVILEGES, FAILURE_STORE_PRIVILEGE } from '../../../../../common/constants';
14+
import { STREAMS_API_PRIVILEGES } from '../../../../../common/constants';
1515
import { createServerRoute } from '../../../create_server_route';
1616
import { getDataStreamLifecycle } from '../../../../lib/streams/stream_crud';
1717

@@ -39,10 +39,6 @@ export const listStreamsRoute = createServerRoute({
3939

4040
const streamNames = streams.filter(({ exists }) => exists).map(({ stream }) => stream.name);
4141

42-
const failureStorePrivileges = await scopedClusterClient.asCurrentUser.security.hasPrivileges({
43-
index: [{ names: streamNames, privileges: [FAILURE_STORE_PRIVILEGE] }],
44-
});
45-
4642
const dataStreams = await processAsyncInChunks(streamNames, (streamNamesChunk) =>
4743
scopedClusterClient.asCurrentUser.indices.getDataStream({ name: streamNamesChunk })
4844
);
@@ -58,8 +54,6 @@ export const listStreamsRoute = createServerRoute({
5854
stream,
5955
effective_lifecycle: getDataStreamLifecycle(match ?? null),
6056
data_stream: match,
61-
can_read_failure_store:
62-
failureStorePrivileges.index?.[stream.name]?.[FAILURE_STORE_PRIVILEGE],
6357
});
6458
return acc;
6559
}, []);

x-pack/platform/plugins/shared/streams_app/public/components/stream_list_view/data_quality_column.tsx

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,9 @@ import { esqlResultToTimeseries } from '../../util/esql_result_to_timeseries';
1313
import type { StreamHistogramFetch } from '../../hooks/use_streams_histogram_fetch';
1414

1515
export function DataQualityColumn({
16-
indexPattern,
1716
histogramQueryFetch,
18-
considerFailedQuality,
1917
}: {
20-
indexPattern: string;
2118
histogramQueryFetch: StreamHistogramFetch;
22-
considerFailedQuality?: boolean;
2319
}) {
2420
const histogramQueryResult = useAsync(() => histogramQueryFetch.docCount, [histogramQueryFetch]);
2521
const failedDocsResult = useAsync(
@@ -52,8 +48,8 @@ export function DataQualityColumn({
5248
const degradedDocCount = degradedDocsResult?.value
5349
? Number(degradedDocsResult.value?.values?.[0]?.[0])
5450
: 0;
55-
const failedDocCount = degradedDocsResult?.value
56-
? Number(failedDocsResult.value.values?.[0]?.[0])
51+
const failedDocCount = failedDocsResult?.value
52+
? Number(failedDocsResult.value?.values?.[0]?.[0])
5753
: 0;
5854

5955
const degradedPercentage = calculatePercentage({
@@ -66,9 +62,7 @@ export function DataQualityColumn({
6662
count: failedDocCount,
6763
});
6864

69-
const quality = considerFailedQuality
70-
? mapPercentageToQuality([degradedPercentage, failedPercentage])
71-
: mapPercentageToQuality([degradedPercentage]);
65+
const quality = mapPercentageToQuality([degradedPercentage, failedPercentage]);
7266

7367
const isLoading =
7468
histogramQueryResult.loading || failedDocsResult?.loading || degradedDocsResult.loading;

x-pack/platform/plugins/shared/streams_app/public/components/stream_list_view/tree_table.tsx

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -167,13 +167,7 @@ export function StreamsTreeTable({
167167
dataType: 'number',
168168
render: (_: unknown, item: TableRow) =>
169169
item.data_stream ? (
170-
<DataQualityColumn
171-
indexPattern={item.stream.name}
172-
histogramQueryFetch={getStreamDocCounts(item.stream.name)}
173-
considerFailedQuality={
174-
item.can_read_failure_store && item.data_stream?.failure_store?.enabled
175-
}
176-
/>
170+
<DataQualityColumn histogramQueryFetch={getStreamDocCounts(item.stream.name)} />
177171
) : null,
178172
},
179173
{

x-pack/platform/plugins/shared/streams_app/public/components/stream_list_view/utils.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,6 @@ export const enrichStream = (node: StreamTree | ListStreamDetail): EnrichedStrea
128128
stream: node.stream,
129129
effective_lifecycle: node.effective_lifecycle,
130130
data_stream: node.data_stream,
131-
can_read_failure_store: node.can_read_failure_store,
132131
nameSortKey,
133132
documentsCount: 0,
134133
retentionMs,

0 commit comments

Comments
 (0)