alibaba / alibaba/feathub

Remove dummy int64 field in protobuf format

Open
#252 0 comments 0 reactions 0 assignees View on GitHub
type:improvement
Dominant language
Python
Stars
350
Forks
60
PR merge metrics
No merged PRs in 30d

Description

message.proto
```proto
message FileSystemTestMessage {
// TODO: We have to add a dummy field with type int64 at the beginning,
// otherwise exception will be thrown.
// https://issues.apache.org/jira/browse/FLINK-31944
```

```proto
message AllTypesTest {
// TODO: We have to add a dummy field with type int64 at the beginning,
// otherwise exception will be thrown.
// https://issues.apache.org/jira/browse/FLINK-31944
```

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in message.proto by locating the FileSystemTestMessage and AllTypesTest definitions and reviewing why the dummy int64 fields were added. Check FLINK-31944 for the relevant compatibility or exception details, then identify the existing tests for these protobuf messages. Done means the dummy fields can be removed without the exception returning and the affected tests still pass.

Written by the indexing model from the issue text.

Assessment

Domain
data-engineering
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.