Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Macro definition duplicated #39102

Closed
1 task done
cydrain opened this issue Jan 9, 2025 · 3 comments
Closed
1 task done

[Bug]: Macro definition duplicated #39102

cydrain opened this issue Jan 9, 2025 · 3 comments
Assignees
Labels
kind/bug Issues or changes related a bug needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.

Comments

@cydrain
Copy link
Contributor

cydrain commented Jan 9, 2025

Is there an existing issue for this?

  • I have searched the existing issues

Environment

- Milvus version: master (134952b)
- Deployment mode(standalone or cluster):
- MQ type(rocksmq, pulsar or kafka):    
- SDK version(e.g. pymilvus v2.0.0rc2):
- OS(Ubuntu or CentOS): ubuntu 20.04
- CPU/Memory: 
- GPU: 
- Others: gcc 9.4.0

Current Behavior

see following build warning

In file included from /home/caiyd/work/vec/milvus/internal/core/src/mmap/ChunkedColumn.h:41,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/InsertRecord.h:30,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/FieldIndexing.h:24,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/SegmentInterface.h:22,
                 from /home/caiyd/work/vec/milvus/internal/core/src/exec/QueryContext.h:30,
                 from /home/caiyd/work/vec/milvus/internal/core/src/exec/Driver.h:26,
                 from /home/caiyd/work/vec/milvus/internal/core/src/exec/Driver.cpp:17:
/home/caiyd/work/vec/milvus/internal/core/src/mmap/Utils.h:36: warning: "THROW_FILE_WRITE_ERROR" redefined
   36 | #define THROW_FILE_WRITE_ERROR                                           \
      | 
In file included from /home/caiyd/work/vec/milvus/internal/core/src/common/ChunkTarget.h:17,
                 from /home/caiyd/work/vec/milvus/internal/core/src/common/Chunk.h:23,
                 from /home/caiyd/work/vec/milvus/internal/core/src/mmap/ChunkedColumn.h:32,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/InsertRecord.h:30,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/FieldIndexing.h:24,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/SegmentInterface.h:22,
                 from /home/caiyd/work/vec/milvus/internal/core/src/exec/QueryContext.h:30,
                 from /home/caiyd/work/vec/milvus/internal/core/src/exec/Driver.h:26,
                 from /home/caiyd/work/vec/milvus/internal/core/src/exec/Driver.cpp:17:
/home/caiyd/work/vec/milvus/internal/core/src/common/File.h:25: note: this is the location of the previous definition
   25 | #define THROW_FILE_WRITE_ERROR                                           \
      | 
[ 86%] Building CXX object src/segcore/CMakeFiles/milvus_segcore.dir/reduce/StreamReduce.cpp.o
[ 86%] Building CXX object src/segcore/CMakeFiles/milvus_segcore.dir/reduce_c.cpp.o
[ 86%] Building CXX object src/storage/CMakeFiles/milvus_storage.dir/storage_c.cpp.o
[ 86%] Building CXX object src/exec/CMakeFiles/milvus_exec.dir/expression/BinaryArithOpEvalRangeExpr.cpp.o
In file included from /home/caiyd/work/vec/milvus/internal/core/src/mmap/ChunkedColumn.h:41,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/InsertRecord.h:30,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/FieldIndexing.h:24,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/SegmentInterface.h:22,
                 from /home/caiyd/work/vec/milvus/internal/core/src/segcore/reduce/GroupReduce.cpp:14:
/home/caiyd/work/vec/milvus/internal/core/src/mmap/Utils.h:36: warning: "THROW_FILE_WRITE_ERROR" redefined
   36 | #define THROW_FILE_WRITE_ERROR                                           \
      | 

Expected Behavior

should see no build warning

Steps To Reproduce

No response

Milvus Log

No response

Anything else?

No response

@cydrain cydrain added kind/bug Issues or changes related a bug needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Jan 9, 2025
@cydrain
Copy link
Contributor Author

cydrain commented Jan 9, 2025

/assign

@cydrain
Copy link
Contributor Author

cydrain commented Jan 9, 2025

/close

@sre-ci-robot
Copy link
Contributor

@cydrain: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Issues or changes related a bug needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.
Projects
None yet
Development

No branches or pull requests

3 participants