Skip to content

adjust binary file location #535

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

Open
nautaa opened this issue Mar 31, 2025 · 4 comments · May be fixed by #541
Open

adjust binary file location #535

nautaa opened this issue Mar 31, 2025 · 4 comments · May be fixed by #541
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Needs help from a contributor

Comments

@nautaa
Copy link
Member

nautaa commented Mar 31, 2025

Enhancement

currently, we put all unittest and binary executable file(such as observer/obclient) together. It's better to put them in different folder.

currently:

cd build_debug/bin/
ls
aggregate_hash_table_test  disk_buffer_pool_test     ob_block_test            ob_table_test
arithmetic_operator_test   disk_log_handler_test     ob_bloomfilter_test      ob_util_test
bitmap_test                double_write_buffer_test  obclient                 parser_test
bplus_tree_log_entry_test  expression_test           ob_compaction_test       pax_storage_test
bplus_tree_log_test        integer_generator_test    ob_lru_cache_test        persist_test
bplus_tree_test            log_buffer_test           oblsm_bench              pidfile_test
bp_manager_test            log_entry_test            oblsm_cli                record_manager_test
buffer_pool_log_test       log_file_test             ob_lsm_manifest_test     ring_buffer_test
catalog_test               log_test                  ob_lsm_test              serializer_test
chunk_test                 lower_bound_test          ob_lsm_transaction_test  simple_queue_test
clog_dump                  md5_test                  ob_lsm_wal_test          thread_pool_executor_test
codec_test                 mem_pool_test             observer
composite_tuple_test       mvcc_trx_log_test         ob_skiplist_basic_test
cpplings                   ob_arena_test             ob_skiplist_test
@nautaa nautaa added enhancement New feature or request good first issue Good for newcomers help wanted Needs help from a contributor labels Mar 31, 2025
@yadunand-kamath
Copy link

Hi, do you want to separate unittest files and binary executables into different folders? If yes, I can work on this.

@nautaa
Copy link
Member Author

nautaa commented Apr 1, 2025

@yadunand-kamath yes! thanks for your interest!

@yadunand-kamath
Copy link

Hi @nautaa, due to multiple cmake build errors I am unable to build the project. I have been trying to fix it but I don't think I can work on this issue if I can't build it

@nautaa
Copy link
Member Author

nautaa commented Apr 3, 2025

@yadunand-kamath how do you build this project? you can take a look at

yadunand-kamath added a commit to yadunand-kamath/miniob that referenced this issue Apr 5, 2025
Signed-off-by: Yadunand Kamath <yadunand.kamath@gmail.com>
yadunand-kamath added a commit to yadunand-kamath/miniob that referenced this issue Apr 20, 2025
Signed-off-by: Yadunand Kamath <yadunand.kamath@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Needs help from a contributor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants