Re: [I] [Apache][Infra] Committer: How to bind GitHub account to Apache account [incubator-graphar]

2024-04-08 Thread via GitHub
sighingnow commented on issue #437: URL: https://github.com/apache/incubator-graphar/issues/437#issuecomment-2044103334 Done. ![image](https://github.com/apache/incubator-graphar/assets/7144772/4e27c488-e50a-40e4-bea6-315ea891e371) -- This is an automated message from the Apac

[PR] Update the doucument with apache guide [incubator-graphar]

2024-04-09 Thread via GitHub
acezen opened a new pull request, #444: URL: https://github.com/apache/incubator-graphar/pull/444 ## Proposed changes Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature reques

Re: [I] [Spark-java-refactor] Split java-info to a single submodule [incubator-graphar]

2024-04-09 Thread via GitHub
acezen commented on issue #445: URL: https://github.com/apache/incubator-graphar/issues/445#issuecomment-2046472215 the discussion note can find in [GraphAr meeting note(2024-04-09)](https://docs.google.com/document/d/1fzMW_6lzTbR0YGa3C0pY3ca_ZYuPSBkmrNtfWXlnYr8/edit#heading=h.k1ts4kvvl8jq)

Re: [PR] [Feat][Doc] Update the repository url and the docs url [incubator-graphar]

2024-04-09 Thread via GitHub
acezen merged PR #444: URL: https://github.com/apache/incubator-graphar/pull/444 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

[PR] chore: fix the header of pyspark source code [incubator-graphar]

2024-04-09 Thread via GitHub
acezen opened a new pull request, #446: URL: https://github.com/apache/incubator-graphar/pull/446 ## Proposed changes as title -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] chore: fix the header of pyspark source code [incubator-graphar]

2024-04-09 Thread via GitHub
acezen merged PR #446: URL: https://github.com/apache/incubator-graphar/pull/446 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

[PR] Add community page [incubator-graphar-website]

2024-04-10 Thread via GitHub
acezen opened a new pull request, #9: URL: https://github.com/apache/incubator-graphar-website/pull/9 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubs

Re: [PR] Add community message [incubator-graphar-website]

2024-04-10 Thread via GitHub
acezen merged PR #9: URL: https://github.com/apache/incubator-graphar-website/pull/9 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...

[PR] chore: Removing the community and release related document (already moved to website) [incubator-graphar]

2024-04-10 Thread via GitHub
acezen opened a new pull request, #447: URL: https://github.com/apache/incubator-graphar/pull/447 ## Proposed changes related #432 In https://github.com/apache/incubator-graphar-website/pull/9, the community and release related document has been add to the website, we should remove

Re: [PR] chore: Removing the community and release related document (already moved to website) [incubator-graphar]

2024-04-10 Thread via GitHub
acezen merged PR #447: URL: https://github.com/apache/incubator-graphar/pull/447 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

[PR] [Docs] Reorg the documents to adapt website building [incubator-graphar]

2024-04-10 Thread via GitHub
acezen opened a new pull request, #448: URL: https://github.com/apache/incubator-graphar/pull/448 ## Proposed changes Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature reques

[PR] chore: run docs CI on ubuntu-latest and separate license check to individual CI [incubator-graphar]

2024-04-10 Thread via GitHub
acezen opened a new pull request, #449: URL: https://github.com/apache/incubator-graphar/pull/449 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscrib

Re: [PR] [Docs] Reorganize the documents to adapt website building [incubator-graphar]

2024-04-11 Thread via GitHub
acezen commented on PR #448: URL: https://github.com/apache/incubator-graphar/pull/448#issuecomment-2049064241 cc/ @lixueclaire @Thespica @SemyonSinchenko This pull request make a lot of change to the document, may need your review. -- This is an automated message from the Apache Git Se

Re: [PR] [Docs] Reorganize the documents to adapt website building [incubator-graphar]

2024-04-11 Thread via GitHub
SemyonSinchenko commented on code in PR #448: URL: https://github.com/apache/incubator-graphar/pull/448#discussion_r1560586502 ## docs/libraries/pyspark/how-to.md: ## @@ -0,0 +1,212 @@ +--- +id: how-to +title: How to use GraphAr PySpark package +sidebar_position: 1 +--- + + +##

Re: [PR] [Docs] Reorganize the documents to adapt website building [incubator-graphar]

2024-04-11 Thread via GitHub
acezen commented on code in PR #448: URL: https://github.com/apache/incubator-graphar/pull/448#discussion_r1560609249 ## docs/libraries/pyspark/how-to.md: ## @@ -0,0 +1,212 @@ +--- +id: how-to +title: How to use GraphAr PySpark package +sidebar_position: 1 +--- + + +## GraphAr P

Re: [PR] [Docs] Reorganize the documents to adapt website building [incubator-graphar]

2024-04-11 Thread via GitHub
acezen commented on code in PR #448: URL: https://github.com/apache/incubator-graphar/pull/448#discussion_r1560609946 ## docs/libraries/pyspark/how-to.md: ## @@ -0,0 +1,212 @@ +--- +id: how-to +title: How to use GraphAr PySpark package +sidebar_position: 1 +--- + + +## GraphAr P

Re: [PR] chore: run docs CI on ubuntu-latest and separate license check to individual CI [incubator-graphar]

2024-04-11 Thread via GitHub
acezen merged PR #449: URL: https://github.com/apache/incubator-graphar/pull/449 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

[PR] Remove the docs content and use content in incubator-graphar repo [incubator-graphar-website]

2024-04-11 Thread via GitHub
acezen opened a new pull request, #10: URL: https://github.com/apache/incubator-graphar-website/pull/10 We put the dev docs in [incubartor-graphar/docs](https://github.com/apache/incubator-graphar/blob/main/docs), remove the docs content of this repo and refine the building process to sync

Re: [PR] [Docs] Reorganize the documents to adapt website building [incubator-graphar]

2024-04-11 Thread via GitHub
acezen commented on PR #448: URL: https://github.com/apache/incubator-graphar/pull/448#issuecomment-2049405354 The path has been replaced with relative path, and I add a TODO for add markdown lint and typo check CI, please review again, thanks~ -- This is an automated message from the Ap

Re: [PR] [Docs] Reorganize the documents to adapt website building [incubator-graphar]

2024-04-11 Thread via GitHub
Thespica commented on PR #448: URL: https://github.com/apache/incubator-graphar/pull/448#issuecomment-2049578642 Hi, @acezen the changes itself is LGTM. But when I try to build locally and take a look, I find it a little bit bothering. Can you write some letters that teach how to bui

Re: [PR] [Docs] Reorganize the documents to adapt website building [incubator-graphar]

2024-04-11 Thread via GitHub
acezen commented on PR #448: URL: https://github.com/apache/incubator-graphar/pull/448#issuecomment-2049653020 > Hi, @acezen the change itself is LGTM. > > But when I try to build locally and take a look, I find it a little bit bothering. Can you write some letters that teach how to

Re: [PR] [Docs] Reorganize the documents to adapt website building [incubator-graphar]

2024-04-11 Thread via GitHub
acezen merged PR #448: URL: https://github.com/apache/incubator-graphar/pull/448 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

Re: [PR] Remove the docs content and use content in incubator-graphar repo instead [incubator-graphar-website]

2024-04-11 Thread via GitHub
acezen merged PR #10: URL: https://github.com/apache/incubator-graphar-website/pull/10 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr.

[PR] Add API reference pages place holder [incubator-graphar-website]

2024-04-11 Thread via GitHub
acezen opened a new pull request, #11: URL: https://github.com/apache/incubator-graphar-website/pull/11 Now the API references are not ready, add place holder to avoid page not found. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Add API reference pages place holder to avoid page not found error [incubator-graphar-website]

2024-04-11 Thread via GitHub
acezen merged PR #11: URL: https://github.com/apache/incubator-graphar-website/pull/11 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr.

[PR] [Minor][Doc] Fix issues in the documentation [incubator-graphar]

2024-04-11 Thread via GitHub
lixueclaire opened a new pull request, #450: URL: https://github.com/apache/incubator-graphar/pull/450 ## Proposed changes as titled. ## Checklist _Put an `x` in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them,

Re: [PR] [Minor][Doc] Fix issues in the documentation [incubator-graphar]

2024-04-11 Thread via GitHub
acezen merged PR #450: URL: https://github.com/apache/incubator-graphar/pull/450 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

[PR] Add devdoc CI to fetch the dev docs and deploy, trigger by manual or … [incubator-graphar-website]

2024-04-12 Thread via GitHub
acezen opened a new pull request, #12: URL: https://github.com/apache/incubator-graphar-website/pull/12 …certain schedule time -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment

Re: [PR] Add devdoc CI to fetch the dev docs and deploy, trigger by manual or … [incubator-graphar-website]

2024-04-12 Thread via GitHub
acezen merged PR #12: URL: https://github.com/apache/incubator-graphar-website/pull/12 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr.

[PR] [Improve][DevInfra] Use korandoru/hawkeye to check license header [incubator-graphar]

2024-04-15 Thread via GitHub
acezen opened a new pull request, #452: URL: https://github.com/apache/incubator-graphar/pull/452 ## Proposed changes related #451 ## Checklist _Put an `x` in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't

Re: [PR] [Improve][DevInfra] Use korandoru/hawkeye to check license header [incubator-graphar]

2024-04-15 Thread via GitHub
acezen merged PR #452: URL: https://github.com/apache/incubator-graphar/pull/452 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

Re: [I] [Feat][DevInfra] Use `korandoru / hawkeye` to check license header [incubator-graphar]

2024-04-15 Thread via GitHub
acezen closed issue #451: [Feat][DevInfra] Use `korandoru / hawkeye` to check license header URL: https://github.com/apache/incubator-graphar/issues/451 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[PR] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-15 Thread via GitHub
acezen opened a new pull request, #453: URL: https://github.com/apache/incubator-graphar/pull/453 ## Proposed changes related issue ## Checklist _Put an `x` in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't

Re: [PR] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-15 Thread via GitHub
acezen commented on PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#issuecomment-2058110635 hi, @SemyonSinchenko. The [CI report](https://github.com/apache/incubator-graphar/actions/runs/8698896273/job/23856591503?pr=453) seems that generate GraphAr PySpark docs need to

[PR] Add api reference generation process [incubator-graphar-website]

2024-04-15 Thread via GitHub
acezen opened a new pull request, #13: URL: https://github.com/apache/incubator-graphar-website/pull/13 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsu

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-15 Thread via GitHub
SemyonSinchenko commented on code in PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#discussion_r1566702548 ## .github/workflows/spark.yaml: ## @@ -52,38 +52,44 @@ jobs: spark-hadoop: "spark-3.3.4-bin-hadoop3" steps: -- uses: actions/ch

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-15 Thread via GitHub
SemyonSinchenko commented on PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#issuecomment-2058226388 > hi, @SemyonSinchenko. The [CI report](https://github.com/apache/incubator-graphar/actions/runs/8698896273/job/23856591503?pr=453) seems that generate GraphAr PySpark do

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-15 Thread via GitHub
acezen commented on code in PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#discussion_r1566728495 ## .github/workflows/spark.yaml: ## @@ -52,38 +52,44 @@ jobs: spark-hadoop: "spark-3.3.4-bin-hadoop3" steps: -- uses: actions/checkout@v3

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-16 Thread via GitHub
acezen commented on PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#issuecomment-2058560221 > > hi, @SemyonSinchenko. The [CI report](https://github.com/apache/incubator-graphar/actions/runs/8698896273/job/23856591503?pr=453) seems that generate GraphAr PySpark docs need

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-16 Thread via GitHub
SemyonSinchenko commented on code in PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#discussion_r1567080373 ## .github/workflows/java.yml: ## @@ -61,18 +61,22 @@ jobs: sudo apt-get install llvm-11 clang-11 lld-11 libclang-11-dev libz-dev -y

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-16 Thread via GitHub
acezen commented on code in PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#discussion_r1567086483 ## .github/workflows/java.yml: ## @@ -61,18 +61,22 @@ jobs: sudo apt-get install llvm-11 clang-11 lld-11 libclang-11-dev libz-dev -y

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-16 Thread via GitHub
acezen commented on PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#issuecomment-2060219702 @lixueclaire @Thespica any comments? I prepare to merge the PR today. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-16 Thread via GitHub
Thespica commented on PR #453: URL: https://github.com/apache/incubator-graphar/pull/453#issuecomment-2060344235 > @lixueclaire @Thespica any comments? I prepare to merge the PR today. LGTM~ -- This is an automated message from the Apache Git Service. To respond to the message, ple

Re: [PR] [Docs] Refine the API reference generation process for libraries [incubator-graphar]

2024-04-16 Thread via GitHub
acezen merged PR #453: URL: https://github.com/apache/incubator-graphar/pull/453 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

Re: [I] [Infra] IP Clearance and Code Cleanup [incubator-graphar]

2024-04-17 Thread via GitHub
CalvinKirs commented on issue #430: URL: https://github.com/apache/incubator-graphar/issues/430#issuecomment-2060571053 Third Party Licensing Policy: https://www.apache.org/legal/resolved.html For binary dependencies, JUnit (EPL)is a perfectly viable option. check here: https://www.ap

Re: [PR] Add api reference generation process [incubator-graphar-website]

2024-04-17 Thread via GitHub
acezen merged PR #13: URL: https://github.com/apache/incubator-graphar-website/pull/13 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr.

Re: [I] [Infra] IP Clearance and Code Cleanup [incubator-graphar]

2024-04-17 Thread via GitHub
acezen commented on issue #430: URL: https://github.com/apache/incubator-graphar/issues/430#issuecomment-2060649077 > Third Party Licensing Policy: https://www.apache.org/legal/resolved.html For binary dependencies, JUnit (EPL)is a perfectly viable option. check here: https://www.apache.or

[PR] Direct cpp api reference to correct location [incubator-graphar-website]

2024-04-17 Thread via GitHub
acezen opened a new pull request, #14: URL: https://github.com/apache/incubator-graphar-website/pull/14 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsu

Re: [PR] Direct cpp api reference to correct location & Add generate api docs to devdoc.yml too [incubator-graphar-website]

2024-04-17 Thread via GitHub
acezen merged PR #14: URL: https://github.com/apache/incubator-graphar-website/pull/14 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr.

[PR] [FEAT][C++] Use `#program once` to replace `#ifndef` as include guards [incubator-graphar]

2024-04-17 Thread via GitHub
acezen opened a new pull request, #455: URL: https://github.com/apache/incubator-graphar/pull/455 ## Proposed changes This change use `#program once` to replace `#ifndef` as include guards ## Further comments Fixs #454 -- This is an automated message from the Apache Git

[PR] [FEAT][C++] Cast to correct schema when get chunk with property reader [incubator-graphar]

2024-04-18 Thread via GitHub
acezen opened a new pull request, #456: URL: https://github.com/apache/incubator-graphar/pull/456 ## Proposed changes as issue #219 describe, the chunk table get from arrow chunk reader may not has the same type with schema. This change help to fix the problem by cast the chunk table t

Re: [PR] [FEAT][C++] Use `#pragma once` to replace `#ifndef` as include guards [incubator-graphar]

2024-04-19 Thread via GitHub
acezen merged PR #455: URL: https://github.com/apache/incubator-graphar/pull/455 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

Re: [PR] [BugFix][C++] Cast to correct schema when get chunk with property reader [incubator-graphar]

2024-04-19 Thread via GitHub
acezen merged PR #456: URL: https://github.com/apache/incubator-graphar/pull/456 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

Re: [I] [Bug]: DataType inconsistent for Int32 in CSV file format [incubator-graphar]

2024-04-19 Thread via GitHub
acezen closed issue #219: [Bug]: DataType inconsistent for Int32 in CSV file format URL: https://github.com/apache/incubator-graphar/issues/219 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the sp

Re: [I] [Feat][C++] Use `#pragma once` to replace include guard [incubator-graphar]

2024-04-19 Thread via GitHub
acezen closed issue #454: [Feat][C++] Use `#pragma once` to replace include guard URL: https://github.com/apache/incubator-graphar/issues/454 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the spec

Re: [I] [Feat][C++] Use `#pragma once` to replace include guard [incubator-graphar]

2024-04-19 Thread via GitHub
acezen commented on issue #454: URL: https://github.com/apache/incubator-graphar/issues/454#issuecomment-2066240085 close as resoved -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user Agwave edited a discussion: fail to run example high_level_writer_example It is ok to Build with examples and then run high_level_writer_example. But when I create a new project and imitate the code of high_level_writer_example. The running result is: Vertices builder vertex_cou

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user Agwave added a comment to the discussion: fail to run example high_level_writer_example ![WXWorkLocal_20240419182000](https://github.com/apache/incubator-graphar/assets/42104894/863299be-6361-46cc-ace4-6ead2bd66577) GitHub link: https://github.com/apache/incubator-graphar/discuss

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user Agwave added a comment to the discussion: fail to run example high_level_writer_example CMakeLists.txt ``` cmake_minimum_required(VERSION 3.0) project(GrapharExample) # 查找Boost库 find_package(Boost REQUIRED COMPONENTS graph) include_directories(${Boost_INCLUDE_DIRS}) # 查找Arrow库 fin

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user Agwave edited a discussion: fail to run example high_level_writer_example It is ok to Build with examples and then run high_level_writer_example. But when I create a new project and imitate the code of high_level_writer_example. The running result is: Vertices builder vertex_cou

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen edited a comment on the discussion: fail to run example high_level_writer_example Thank for posting the question, I will take a look to the code. GitHub link: https://github.com/apache/incubator-graphar/discussions/457#discussioncomment-9165294 This is an automaticall

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen added a comment to the discussion: fail to run example high_level_writer_example Thank for post the question, I will take a look to the code. GitHub link: https://github.com/apache/incubator-graphar/discussions/457#discussioncomment-9165294 This is an automatically se

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen added a comment to the discussion: fail to run example high_level_writer_example how about log out the error message of AddEdge operation: ```c++ auto st = builder.AddEdge(e); if (!st.ok()) { std::cout << "Failing adding edge: " << st.message() << std::endl; } ``` GitHub lin

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen edited a comment on the discussion: fail to run example high_level_writer_example I have reproduce the problem: https://github.com/apache/incubator-graphar/assets/11835645/b8d50052-32be-4709-977c-77339337504c";> It seems that AddEdge need edge info contain both ordered_by_so

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen added a comment to the discussion: fail to run example high_level_writer_example I have reproduce the problem: https://github.com/apache/incubator-graphar/assets/11835645/b8d50052-32be-4709-977c-77339337504c";> It seems that AddEdge need edge info contain both ordered_by_sou

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen edited a comment on the discussion: fail to run example high_level_writer_example I have reproduce the problem: https://github.com/apache/incubator-graphar/assets/11835645/b8d50052-32be-4709-977c-77339337504c";> It seems that you has passed the wrong adj list in **LINE 58**,

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen edited a comment on the discussion: fail to run example high_level_writer_example I have reproduce the problem: https://github.com/apache/incubator-graphar/assets/11835645/b8d50052-32be-4709-977c-77339337504c";> It seems that you has passed the wrong adj list in **LINE 58**,

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen edited a comment on the discussion: fail to run example high_level_writer_example I have reproduce the problem: https://github.com/apache/incubator-graphar/assets/11835645/b8d50052-32be-4709-977c-77339337504c";> It seems that you has passed the wrong adj list in **LINE 58**,

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen edited a comment on the discussion: fail to run example high_level_writer_example I have reproduce the problem: https://github.com/apache/incubator-graphar/assets/11835645/b8d50052-32be-4709-977c-77339337504c";> It seems that you has passed the wrong adj list in **LINE 58**,

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user acezen edited a comment on the discussion: fail to run example high_level_writer_example I have reproduce the problem: https://github.com/apache/incubator-graphar/assets/11835645/b8d50052-32be-4709-977c-77339337504c";> It seems that you has passed the wrong adj list in **LINE 58**,

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-19 Thread via GitHub
GitHub user Agwave added a comment to the discussion: fail to run example high_level_writer_example Thank you, I will give it a try next week GitHub link: https://github.com/apache/incubator-graphar/discussions/457#discussioncomment-9165631 This is an automatically sent email for commit

Re: [D] fail to run example high_level_writer_example [incubator-graphar]

2024-04-21 Thread via GitHub
GitHub user Agwave added a comment to the discussion: fail to run example high_level_writer_example Thank you @acezen for your assistance! Your solution worked perfectly. GitHub link: https://github.com/apache/incubator-graphar/discussions/457#discussioncomment-9182726 This is an automa

[PR] docs: Update CONTRIBUTING.md and READMEs [incubator-graphar]

2024-04-22 Thread via GitHub
acezen opened a new pull request, #458: URL: https://github.com/apache/incubator-graphar/pull/458 ## Proposed changes related issue #432 - Update `CONTRIBUTING` and `README` - Update the related template for pull request and issues -- This is an automated message from the

[PR] feat(c++): Replace with URI parse from arrow::internal::URI with boost::URL [incubator-graphar]

2024-04-22 Thread via GitHub
acezen opened a new pull request, #460: URL: https://github.com/apache/incubator-graphar/pull/460 ## Proposed changes as #459 describe -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to t

Re: [PR] feat(c++): Use simple-uri-parser as uri parser, remove the rely on arrow::internal::URI [incubator-graphar]

2024-04-22 Thread via GitHub
acezen merged PR #460: URL: https://github.com/apache/incubator-graphar/pull/460 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

Re: [PR] docs: Update CONTRIBUTING.md and READMEs [incubator-graphar]

2024-04-22 Thread via GitHub
acezen merged PR #458: URL: https://github.com/apache/incubator-graphar/pull/458 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

Re: [I] [Infra][Doc] Update the documents with Apache branding guide and incubator-graphar-website [incubator-graphar]

2024-04-22 Thread via GitHub
acezen commented on issue #432: URL: https://github.com/apache/incubator-graphar/issues/432#issuecomment-2071474957 The sub-items have done, close the issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [I] [Infra][Doc] Update the documents with Apache branding guide and incubator-graphar-website [incubator-graphar]

2024-04-22 Thread via GitHub
acezen closed issue #432: [Infra][Doc] Update the documents with Apache branding guide and incubator-graphar-website URL: https://github.com/apache/incubator-graphar/issues/432 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[I] feat(c++): move the `result.hpp` third-party dependency from external directory to third-party directory [incubator-graphar]

2024-04-22 Thread via GitHub
acezen opened a new issue, #461: URL: https://github.com/apache/incubator-graphar/issues/461 ### Describe the enhancement requested Currently the single header third-party dependency [result](https://github.com/bitwizeshift/result) is put in the external directory for historical reas

[PR] feat(c++): move the result.hpp third-party dependency from external d… [incubator-graphar]

2024-04-23 Thread via GitHub
acezen opened a new pull request, #462: URL: https://github.com/apache/incubator-graphar/pull/462 …irectory to third-party directory ### Reason for this PR as issue #461 describe ### What changes are included in this PR? ### Are these changes teste

Re: [I] [Infra][Incubator] The tracking issue for Apache incubator code import process [incubator-graphar]

2024-04-23 Thread via GitHub
acezen closed issue #416: [Infra][Incubator] The tracking issue for Apache incubator code import process URL: https://github.com/apache/incubator-graphar/issues/416 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] feat(c++): move the result.hpp third-party dependency from external directory to third-party directory [incubator-graphar]

2024-04-24 Thread via GitHub
acezen merged PR #462: URL: https://github.com/apache/incubator-graphar/pull/462 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

[PR] chore: Update directory paths in README files [incubator-graphar]

2024-04-24 Thread via GitHub
jasinliu opened a new pull request, #464: URL: https://github.com/apache/incubator-graphar/pull/464 ### Reason for this PR ### What changes are included in this PR? ### Are these changes tested? ### Are there any user-facing changes?

Re: [PR] chore: Update directory paths in README files [incubator-graphar]

2024-04-24 Thread via GitHub
liuxiaocs7 commented on code in PR #464: URL: https://github.com/apache/incubator-graphar/pull/464#discussion_r1577724006 ## cpp/README.md: ## @@ -42,7 +42,7 @@ repository and navigated to the ``cpp`` subdirectory: ```bash $ git clone https://github.com/apache/incubator-

Re: [PR] chore: Update directory paths in README files [incubator-graphar]

2024-04-24 Thread via GitHub
acezen commented on code in PR #464: URL: https://github.com/apache/incubator-graphar/pull/464#discussion_r1577725274 ## cpp/README.md: ## @@ -42,7 +42,7 @@ repository and navigated to the ``cpp`` subdirectory: ```bash $ git clone https://github.com/apache/incubator-grap

Re: [PR] chore: Update directory paths in README files [incubator-graphar]

2024-04-24 Thread via GitHub
acezen commented on PR #464: URL: https://github.com/apache/incubator-graphar/pull/464#issuecomment-2074738745 The CI failed is because `sudo apt-get update` got failed in github host, see https://github.com/orgs/community/discussions/120966 it's not related to this PR. -- This is

Re: [PR] chore: Update directory paths in README files [incubator-graphar]

2024-04-24 Thread via GitHub
acezen merged PR #464: URL: https://github.com/apache/incubator-graphar/pull/464 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

[PR] docs: including incubator disclaimer text as footer to api reference [incubator-graphar]

2024-04-24 Thread via GitHub
acezen opened a new pull request, #465: URL: https://github.com/apache/incubator-graphar/pull/465 ### Reason for this PR as https://github.com/apache/incubator-graphar-website/issues/15 describe ### What changes are included in this PR? add incubator disclaimer te

Re: [PR] docs: include incubator disclaimer text as footer to api reference [incubator-graphar]

2024-04-25 Thread via GitHub
acezen merged PR #465: URL: https://github.com/apache/incubator-graphar/pull/465 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

[PR] Add download page [incubator-graphar-website]

2024-04-25 Thread via GitHub
acezen opened a new pull request, #16: URL: https://github.com/apache/incubator-graphar-website/pull/16 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsu

Re: [PR] Add download page [incubator-graphar-website]

2024-04-25 Thread via GitHub
acezen merged PR #16: URL: https://github.com/apache/incubator-graphar-website/pull/16 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr.

Re: [I] [Feat][Format] Support extra metadata in graph info level that provide users to has user defined metadata [incubator-graphar]

2024-04-26 Thread via GitHub
acezen closed issue #354: [Feat][Format] Support extra metadata in graph info level that provide users to has user defined metadata URL: https://github.com/apache/incubator-graphar/issues/354 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [I] Tracking issue of GraphAr first Apache release [incubator-graphar]

2024-04-26 Thread via GitHub
SemyonSinchenko commented on issue #466: URL: https://github.com/apache/incubator-graphar/issues/466#issuecomment-2078850506 Are we going to have Maven and PyPi packages for spark/pyspark? -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [I] Tracking issue of GraphAr first Apache release [incubator-graphar]

2024-04-26 Thread via GitHub
acezen commented on issue #466: URL: https://github.com/apache/incubator-graphar/issues/466#issuecomment-2078921269 > Are we going to have Maven and PyPi packages for spark/pyspark? This is what I want to discuss in the dev list. We can release a source release. But I think it's bett

[I] feat(java): Add aggregator POM for java and spark library to manage the dependency [incubator-graphar]

2024-04-29 Thread via GitHub
acezen opened a new issue, #468: URL: https://github.com/apache/incubator-graphar/issues/468 ### Describe the enhancement requested discussion thread: https://lists.apache.org/thread/hnh2b1pyf6vd0kwok5ggplrso23hrb25 ### Component(s) Java -- This is an automated messag

[PR] feat(java): Add aggregator POM for java and spark library to manage the dependency [incubator-graphar]

2024-04-29 Thread via GitHub
acezen opened a new pull request, #469: URL: https://github.com/apache/incubator-graphar/pull/469 ### Reason for this PR Add aggregator POM for java and spark library to easy manage the dependency ### What changes are included in this PR? add aggregator p

[PR] feat(java): Split java-info to a single module [incubator-graphar]

2024-04-29 Thread via GitHub
Thespica opened a new pull request, #470: URL: https://github.com/apache/incubator-graphar/pull/470 ### Reason for this PR Split the java-info into a single module, which has the same level as spark. Related issue: #445 ### What changes are included in this PR?

Re: [PR] feat(java): Split java-info to a single module [incubator-graphar]

2024-04-29 Thread via GitHub
acezen commented on PR #470: URL: https://github.com/apache/incubator-graphar/pull/470#issuecomment-2083125829 LGTM -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsu

Re: [PR] feat(java): Split java-info to a single module [incubator-graphar]

2024-04-29 Thread via GitHub
acezen merged PR #470: URL: https://github.com/apache/incubator-graphar/pull/470 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gra

Re: [PR] feat(java): Add aggregator POM for java and spark library to manage the dependency [incubator-graphar]

2024-04-29 Thread via GitHub
acezen commented on PR #469: URL: https://github.com/apache/incubator-graphar/pull/469#issuecomment-2084225146 Hi, @SemyonSinchenko @Thespica I would like to get some advice about put the `spark` and `java` libraries to a directory like `maven-projects` next step, that would make the code

  1   2   3   4   5   6   7   8   9   10   >