Skip to content

Commit 5f78be4

Browse files
committed
fix: unipika max size
1 parent 6746821 commit 5f78be4

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

src/containers/Tenant/Diagnostics/TopicData/TopicMessageDetails/TopicMessageDetails.scss

+1
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@
3939
&__string-message {
4040
overflow: hidden;
4141

42+
white-space: pre-wrap;
4243
word-break: break-all;
4344
}
4445
}

src/containers/Tenant/Diagnostics/TopicData/TopicMessageDetails/components/TopicMessage.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import {MESSAGE_SIZE_LIMIT, b} from '../shared';
1414

1515
import {TopicDataSection} from './TopicDataSection';
1616

17-
const UNIPIKA_MAX_SIZE = 100_000;
17+
const UNIPIKA_MAX_SIZE = 1_000_000;
1818

1919
interface TopicMessageProps {
2020
message: string;
@@ -47,6 +47,8 @@ export function TopicMessage({offset, size, message}: TopicMessageProps) {
4747
let convertedMessage;
4848
if (typeof preparedMessage === 'object' && safeParseNumber(size) <= UNIPIKA_MAX_SIZE) {
4949
convertedMessage = unipikaConvert(preparedMessage);
50+
} else if (preparedMessage && typeof preparedMessage === 'object') {
51+
preparedMessage = JSON.stringify(preparedMessage, null, 2);
5052
}
5153

5254
return {preparedMessage, decodedMessage, convertedMessage};
@@ -66,11 +68,7 @@ export function TopicMessage({offset, size, message}: TopicMessageProps) {
6668
) : (
6769
<div className={b('string-message')}>
6870
{/* key is used to reset string's state when toggle fullscreen: otherwise if very long string is expanded, it may be performance issues on open fullscreen mode https://github.com/ydb-platform/ydb-embedded-ui/issues/2265 */}
69-
<ShortyString
70-
key={String(isFullscreen)}
71-
value={JSON.stringify(preparedMessage, null, 2)}
72-
limit={1000}
73-
/>
71+
<ShortyString key={String(isFullscreen)} value={preparedMessage} limit={1000} />
7472
</div>
7573
);
7674

0 commit comments

Comments
 (0)