[I] fix(java): adjust children module version of pom to avoid warning [incubator-graphar]

2025-05-23 Thread via GitHub
Thespica opened a new issue, #688: URL: https://github.com/apache/incubator-graphar/issues/688 ### Describe the enhancement requested When running maven-projects, there are warning like: ```bash [WARNING] [WARNING] Some problems were encountered while building the effective mo

[PR] [DO NOT MERGE]Feat/java info unit tests [incubator-graphar]

2025-05-23 Thread via GitHub
Thespica opened a new pull request, #687: URL: https://github.com/apache/incubator-graphar/pull/687 ### Reason for this PR related issue: #676 Let's see the performance of **jules.google** ### What changes are included in this PR? ### Are these cha

[I] feat(ci): allow running CI when PR is draft or forked repo [incubator-graphar]

2025-05-23 Thread via GitHub
Thespica opened a new issue, #686: URL: https://github.com/apache/incubator-graphar/issues/686 ### Describe the enhancement requested reverse prevous issue: https://github.com/apache/incubator-graphar/issues/602 It will be better for developer and user to run CI when PR is draft

Re: [PR] fix(java,info): Reorder parameters in EdgeInfo constructor [incubator-graphar]

2025-05-23 Thread via GitHub
Thespica commented on PR #685: URL: https://github.com/apache/incubator-graphar/pull/685#issuecomment-2904725102 Hi @yangxk1 , I think order of `srcChunkSize` -> `chunkSize` -> `dstChunkSize` will be better. -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] fix(java,info): Reorder parameters in EdgeInfo constructor [incubator-graphar]

2025-05-23 Thread via GitHub
yangxk1 commented on PR #685: URL: https://github.com/apache/incubator-graphar/pull/685#issuecomment-2904255512 Hi, @Thespica. This should solve the problem in #684 #676 , please review this. -- This is an automated message from the Apache Git Service. To respond to the message, please l

[PR] fix(java,info): Reorder parameters in EdgeInfo constructor [incubator-graphar]

2025-05-23 Thread via GitHub
yangxk1 opened a new pull request, #685: URL: https://github.com/apache/incubator-graphar/pull/685 ## Reason for this PR According to the description in #684 , parameter order mismatch between EdgeInfo constructor and EdgeYaml#toEdgeInfo. ## What changes are included in this PR

[I] bug(java, info): Parameter order mismatch between EdgeInfo constructor and EdgeYaml#toEdgeInfo [incubator-graphar]

2025-05-23 Thread via GitHub
yangxk1 opened a new issue, #684: URL: https://github.com/apache/incubator-graphar/issues/684 ### Describe the bug, including details regarding any error messages, version, and platform. The third issue discussed in #676 In the `EdgeInfo` constructor, the parameter `srcChunkSi

Re: [PR] feat(java-info): add java unit test for loading graph (#676) [incubator-graphar]

2025-05-23 Thread via GitHub
yangxk1 commented on PR #683: URL: https://github.com/apache/incubator-graphar/pull/683#issuecomment-2904103288 > Hi @yangxk1 , thanks for your PR! > > Could you new a java-info CI and make the unit test run automatically? Of course! I'll give it a try. -- This is an automat

Re: [I] feat(java, info): Unit test for loading graph [incubator-graphar]

2025-05-23 Thread via GitHub
yangxk1 commented on issue #676: URL: https://github.com/apache/incubator-graphar/issues/676#issuecomment-2904077824 (1) and (2). aren’t urgent. I’ll open an issue to fix (3). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] feat(java-info): add java unit test for loading graph (#676) [incubator-graphar]

2025-05-23 Thread via GitHub
Copilot commented on code in PR #683: URL: https://github.com/apache/incubator-graphar/pull/683#discussion_r2104328049 ## maven-projects/info/src/test/java/org/apache/graphar/info/GraphLoaderTest.java: ## @@ -41,21 +45,139 @@ public static void clean() {} public void testLo

Re: [PR] feat(java-info): add java unit test for loading graph (#676) [incubator-graphar]

2025-05-23 Thread via GitHub
Thespica commented on PR #683: URL: https://github.com/apache/incubator-graphar/pull/683#issuecomment-2904037578 Hi @yangxk1 , thanks for your PR! Could you new a java-info CI and make the unit test run automatically? -- This is an automated message from the Apache Git Service. To

Re: [I] feat(java, info): Unit test for loading graph [incubator-graphar]

2025-05-23 Thread via GitHub
Thespica commented on issue #676: URL: https://github.com/apache/incubator-graphar/issues/676#issuecomment-2904023894 > While writing unit tests, I found three issues: > > 1. Extra `/` in path concatenation(e.g. `"vertex/person//id/"`) (minor). > 2. Missing `getVersion()` in GraphI

Re: [I] [Feat][C++] Support read certain set of properties in the file level, not with `arrow::Table::Select` [incubator-graphar]

2025-05-23 Thread via GitHub
yangxk1 commented on issue #397: URL: https://github.com/apache/incubator-graphar/issues/397#issuecomment-2903953377 Hi, @acezen ! I think we can improve this by setting `ScannerBuilder.Project`. Compared to filtering at the `FileReader` level, although it may have some impact on perfo

Re: [PR] feat(c++): label filtering API, benchmarks, and examples [incubator-graphar]

2025-05-21 Thread via GitHub
yangxk1 commented on code in PR #654: URL: https://github.com/apache/incubator-graphar/pull/654#discussion_r2099524173 ## cpp/src/graphar/high-level/graph_reader.cc: ## @@ -94,6 +100,290 @@ Vertex::Vertex(IdType id, } } +Result VertexIter::hasLabel(const std::string& label

Re: [PR] feat(c++): label filtering API, benchmarks, and examples [incubator-graphar]

2025-05-21 Thread via GitHub
yangxk1 commented on code in PR #654: URL: https://github.com/apache/incubator-graphar/pull/654#discussion_r2099522935 ## cpp/src/graphar/high-level/graph_reader.cc: ## @@ -94,6 +100,290 @@ Vertex::Vertex(IdType id, } } +Result VertexIter::hasLabel(const std::string& label

Re: [I] feat(java, info): Unit test for loading graph [incubator-graphar]

2025-05-20 Thread via GitHub
yangxk1 commented on issue #676: URL: https://github.com/apache/incubator-graphar/issues/676#issuecomment-2893360061 > > I noticed that some interfaces in the Java code are not fully aligned with those in the C++ code. For example, the [Java code](https://github.com/apache/incubator-grapha

Re: [I] feat(java, info): Unit test for loading graph [incubator-graphar]

2025-05-20 Thread via GitHub
yangxk1 commented on issue #676: URL: https://github.com/apache/incubator-graphar/issues/676#issuecomment-2893375821 While writing unit tests, I found three issues: 1. Extra `/` in path concatenation(e.g. `"vertex/person//id/"`) (minor). 2. Missing `getVersion()` in GraphInfo, Vert

[PR] feat(java-info): add java unit test for loading graph (#676) [incubator-graphar]

2025-05-20 Thread via GitHub
yangxk1 opened a new pull request, #683: URL: https://github.com/apache/incubator-graphar/pull/683 ### Reason for this PR As described in issue #676 , there is currently no unit test to verify loading graph in java-info module. This PR adds unit tests for the Java info module to ensu

Re: [PR] chore(NOTICE): Update NOTICE year to 2025 [incubator-graphar]

2025-05-06 Thread via GitHub
lixueclaire merged PR #682: URL: https://github.com/apache/incubator-graphar/pull/682 -- 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: [PR] chore(github): Fix the incorrect configuration in .asf.yaml [incubator-graphar]

2025-05-06 Thread via GitHub
lixueclaire merged PR #681: URL: https://github.com/apache/incubator-graphar/pull/681 -- 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(NOTICE): Update NOTICE year to 2025 [incubator-graphar]

2025-05-06 Thread via GitHub
CalvinKirs opened a new pull request, #682: URL: https://github.com/apache/incubator-graphar/pull/682 ### Reason for this PR I couldn't find a detailed definition from ASF on this, but it's okay. In short, it’s related to the validity of intellectual property rights.

[PR] chore(github): Fix the incorrect configuration in .asf.yaml [incubator-graphar]

2025-05-06 Thread via GitHub
CalvinKirs opened a new pull request, #681: URL: https://github.com/apache/incubator-graphar/pull/681 ### Reason for this PR **required_linear_history** and **required_pull_request_reviews** were mistakenly nested when they should have been at the same level. Since

[I] [OSPP] LDBC Data Generator Integration for GraphAr [incubator-graphar]

2025-05-05 Thread via GitHub
Thespica opened a new issue, #680: URL: https://github.com/apache/incubator-graphar/issues/680 ### Describe the enhancement requested # Description This project aims to enhance Apache GraphAr (incubating)'s data loading capabilities within the Spark environment, enabling it to

[I] [OSPP] GraphAr Extension for Kuzu [incubator-graphar]

2025-05-05 Thread via GitHub
Thespica opened a new issue, #679: URL: https://github.com/apache/incubator-graphar/issues/679 ### Describe the enhancement requested # Description Apache GraphAr (incubating) is a standard graph storage file format framework, with underlying storage based on CSV format, as wel

Re: [PR] chore(doc): refine readme [incubator-graphar]

2025-04-27 Thread via GitHub
Thespica merged PR #678: URL: https://github.com/apache/incubator-graphar/pull/678 -- 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...@g

Re: [PR] chore(doc): refine empty line [incubator-graphar]

2025-04-27 Thread via GitHub
Thespica merged PR #677: URL: https://github.com/apache/incubator-graphar/pull/677 -- 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...@g

[PR] chore(doc): refine readme [incubator-graphar]

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

[PR] chore(doc): refine empty line [incubator-graphar]

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

Re: [I] feat(java, info): Unit test for loading graph [incubator-graphar]

2025-04-24 Thread via GitHub
Thespica commented on issue #676: URL: https://github.com/apache/incubator-graphar/issues/676#issuecomment-2829232376 > I noticed that some interfaces in the Java code are not fully aligned with those in the C++ code. For example, the [Java code](https://github.com/apache/incubator-graphar

Re: [I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-24 Thread via GitHub
unical1988 commented on issue #672: URL: https://github.com/apache/incubator-graphar/issues/672#issuecomment-2828823700 Shall we first tackle PropertyGraphFrame to make it compatible with GAR? as a first step? -- This is an automated message from the Apache Git Service. To respond to the

Re: [I] feat(java, info): Unit test for loading graph [incubator-graphar]

2025-04-24 Thread via GitHub
yangxk1 commented on issue #676: URL: https://github.com/apache/incubator-graphar/issues/676#issuecomment-2826645975 I noticed that some interfaces in the Java code are not fully aligned with those in the C++ code. For example, the [Java code](https://github.com/apache/incubator-graphar/bl

Re: [I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-23 Thread via GitHub
SemyonSinchenko commented on issue #672: URL: https://github.com/apache/incubator-graphar/issues/672#issuecomment-2823650001 Based on [this comment](https://github.com/kuzudb/kuzu/issues/3353#issuecomment-2072206839) I can assume, that we can create an extension and maintain it here, in Gr

Re: [I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-22 Thread via GitHub
lixueclaire commented on issue #672: URL: https://github.com/apache/incubator-graphar/issues/672#issuecomment-2823166990 > There is a Cpp arrow-backed project Kuzu, that is an emebeddable graph database and in-memory graph processing tool. In my understanding, it is like DuckDB, but for Gr

Re: [I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-21 Thread via GitHub
SemyonSinchenko commented on issue #672: URL: https://github.com/apache/incubator-graphar/issues/672#issuecomment-2818021871 There is a Cpp arrow-backed project Kuzu, that is an emebeddable graph database and in-memory graph processing tool. In my understanding, it is like DuckDB, but for

[I] [java-info] Unit test for loading graph [incubator-graphar]

2025-04-10 Thread via GitHub
Thespica opened a new issue, #676: URL: https://github.com/apache/incubator-graphar/issues/676 ### Describe the enhancement requested Write unit test for java-info module, loading graph from [test set](https://github.com/apache/incubator-graphar-testing/tree/955596c325ceba7b607e285738

[I] infra: it looks like our .asf.yaml is broken [incubator-graphar]

2025-04-10 Thread via GitHub
SemyonSinchenko opened a new issue, #675: URL: https://github.com/apache/incubator-graphar/issues/675 ### Describe the bug, including details regarding any error messages, version, and platform. ``` An error occurred while processing the github feature in .asf.yaml: while p

[I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-09 Thread via GitHub
Thespica opened a new issue, #672: URL: https://github.com/apache/incubator-graphar/issues/672 ### Describe the enhancement requested Dear developers, We are calling on the entire open source community to submit project ideas for the OSPP (Open Source Project Plan). Our goal is

Re: [I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-08 Thread via GitHub
Thespica commented on issue #672: URL: https://github.com/apache/incubator-graphar/issues/672#issuecomment-2785421107 cc @SemyonSinchenko @lixueclaire @acezen -- 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: [I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-08 Thread via GitHub
lixueclaire commented on issue #672: URL: https://github.com/apache/incubator-graphar/issues/672#issuecomment-2785438566 [Spark] LDBC Data Generator Integration for GraphAr Description: Utilize GraphAr's Spark API to integrate with the [LDBC Data Generator](https://github.com/ldbc/ldb

Re: [PR] fix(doc): remove meeting statements in README.md [incubator-graphar]

2025-04-08 Thread via GitHub
Thespica merged PR #674: URL: https://github.com/apache/incubator-graphar/pull/674 -- 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...@g

Re: [I] fix(doc): remove meeting in README.md [incubator-graphar]

2025-04-08 Thread via GitHub
Thespica closed issue #673: fix(doc): remove meeting in README.md URL: https://github.com/apache/incubator-graphar/issues/673 -- 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

Re: [I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-08 Thread via GitHub
SemyonSinchenko commented on issue #672: URL: https://github.com/apache/incubator-graphar/issues/672#issuecomment-2785539297 [Spark] At the moment I'm hardly working on bringing GraphFrames project back to live and resume publishing it to Maven Central (https://github.com/graphframes/graph

[PR] fix(doc): remove meeting in README.md [incubator-graphar]

2025-04-08 Thread via GitHub
Thespica opened a new pull request, #674: URL: https://github.com/apache/incubator-graphar/pull/674 ### Reason for this PR fix #673 ### What changes are included in this PR? modify README, remove meeting statements ### Are these changes tested? no n

Re: [I] OSPP 2025 Idea Collection [incubator-graphar]

2025-04-08 Thread via GitHub
lixueclaire commented on issue #672: URL: https://github.com/apache/incubator-graphar/issues/672#issuecomment-2785434971 [C++] Out-of-Core Graph Computing Engine Description: Leverage the foundational APIs provided by GraphAr to implement a simple out-of-core graph computing engine. T

[I] fix(doc): remove meeting in README.md [incubator-graphar]

2025-04-07 Thread via GitHub
Thespica opened a new issue, #673: URL: https://github.com/apache/incubator-graphar/issues/673 ### Describe the enhancement requested Cause we no longer keep biweekly meetings, it's better to remove it to prevent misunderstanding. ### Component(s) Documentation -- Thi

Re: [I] A small error about C++ compilation paths on cpp/README.md [incubator-graphar]

2025-03-09 Thread via GitHub
lixueclaire closed issue #670: A small error about C++ compilation paths on cpp/README.md URL: https://github.com/apache/incubator-graphar/issues/670 -- 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

Re: [PR] docs: update README to correct wrong directory path for building cpp [incubator-graphar]

2025-03-09 Thread via GitHub
lixueclaire merged PR #671: URL: https://github.com/apache/incubator-graphar/pull/671 -- 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..

[I] A small error about C++ compilation paths on cpp/README.md [incubator-graphar]

2025-03-09 Thread via GitHub
mocusez opened a new issue, #670: URL: https://github.com/apache/incubator-graphar/issues/670 ### Describe the bug, including details regarding any error messages, version, and platform. GraphAr is great, but when I looked at the C++ compilation documentation I found the entry path w

[PR] docs: update README to correct wrong directory path for building cpp [incubator-graphar]

2025-03-09 Thread via GitHub
mocusez opened a new pull request, #671: URL: https://github.com/apache/incubator-graphar/pull/671 ### Reason for this PR #670 ### What changes are included in this PR? Modify line 69 of cpp/README.md from `cd graphar/cpp ` to `cd incubator-graphar/cpp` ###

Re: [PR] docs: update information for publications [incubator-graphar]

2025-03-07 Thread via GitHub
lixueclaire merged PR #668: URL: https://github.com/apache/incubator-graphar/pull/668 -- 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] update citation information [incubator-graphar]

2025-03-05 Thread via GitHub
lixueclaire opened a new pull request, #669: URL: https://github.com/apache/incubator-graphar/pull/669 ### Reason for this PR ### What changes are included in this PR? ### Are these changes tested? ### Are there any user-facing changes?

Re: [PR] update citation information [incubator-graphar]

2025-03-05 Thread via GitHub
lixueclaire merged PR #669: URL: https://github.com/apache/incubator-graphar/pull/669 -- 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] docs: update information for publications [incubator-graphar]

2025-03-05 Thread via GitHub
lixueclaire opened a new pull request, #668: URL: https://github.com/apache/incubator-graphar/pull/668 ### Reason for this PR update information for publications ### What changes are included in this PR? readme and its Chinese version ### Are these changes test

Re: [I] feat(java): provide additional methods in java info [incubator-graphar]

2025-03-05 Thread via GitHub
Thespica commented on issue #667: URL: https://github.com/apache/incubator-graphar/issues/667#issuecomment-2700839058 > I will open a PR with `equals`, `toString` and `hashCode` only. Beyond this, you can add the missed method `getPropertyGroup` to VertexInfo. Maybe rename it to `get

Re: [I] feat(java): provide additional methods in java info [incubator-graphar]

2025-03-04 Thread via GitHub
SemyonSinchenko commented on issue #667: URL: https://github.com/apache/incubator-graphar/issues/667#issuecomment-2697393982 I will open a PR with `equals`, `toString` and `hashCode` only. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [I] feat(java): provide additional methods in java info [incubator-graphar]

2025-03-04 Thread via GitHub
SemyonSinchenko commented on issue #667: URL: https://github.com/apache/incubator-graphar/issues/667#issuecomment-2697329874 I was trying to map existing spark infos to the new java info and realized that some methods are missing, but it looks like they are just named differently. Thanks!

Re: [I] feat(java): provide additional methods in java info [incubator-graphar]

2025-03-04 Thread via GitHub
Thespica commented on issue #667: URL: https://github.com/apache/incubator-graphar/issues/667#issuecomment-2697321974 VertexInfo and EdgeInfo can check if a PropertyGroup is in the info: https://github.com/apache/incubator-graphar/blob/0bf48c1f4a254a06ced3dff1f5c116272bb4b11a/maven-proje

[I] feat(java): follow the same style for proto ser-de [incubator-graphar]

2025-03-04 Thread via GitHub
SemyonSinchenko opened a new issue, #666: URL: https://github.com/apache/incubator-graphar/issues/666 ### Describe the enhancement requested At the moment, for example, to create `EdgeInfo` from the `proto` I should call the constructor, but for `PropertyGroup` I should call `ofProto`

[I] feat(java): provide additional methods in java info [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko opened a new issue, #667: URL: https://github.com/apache/incubator-graphar/issues/667 ### Describe the enhancement requested At least we should override `equals()`, `toString()` and `hashCode`. It is nice to have also methods that checks is a property group in the info

Re: [I] feat(java): remove the hadoop dependency from java-info [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko closed issue #664: feat(java): remove the hadoop dependency from java-info URL: https://github.com/apache/incubator-graphar/issues/664 -- 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

Re: [PR] feat(java): remove Hadoop dependency and update tests / Local [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko merged PR #665: URL: https://github.com/apache/incubator-graphar/pull/665 -- 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-unsubs

Re: [PR] [WIP] add benchmark in README.md [incubator-graphar]

2025-03-03 Thread via GitHub
Thespica commented on PR #656: URL: https://github.com/apache/incubator-graphar/pull/656#issuecomment-2696039846 Cool! @Elssky Could you update the zh-ch README too? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] [WIP] add benchmark in README.md [incubator-graphar]

2025-03-03 Thread via GitHub
lixueclaire commented on PR #656: URL: https://github.com/apache/incubator-graphar/pull/656#issuecomment-2695961624 > @Elssky Hi! I checked the rendered version and it looks cool. What do you think about merging it? You can finish what is not done later. I agree! @Elssky,this PR seem

Re: [PR] [WIP] add benchmark in README.md [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko commented on PR #656: URL: https://github.com/apache/incubator-graphar/pull/656#issuecomment-2694710802 @Elssky Hi! I checked the rendered version and it looks cool. What do you think about merging it? You can finish what is not done later. -- This is an automated message

[PR] feat(java): remove Hadoop dependency and update tests / Local [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko opened a new pull request, #665: URL: https://github.com/apache/incubator-graphar/pull/665 ### Reason for this PR Close #664 ### What changes are included in this PR? - Removing of Hadoop dependencies - Corresponding updates of LocalGrapheS

Re: [PR] feat(java): remove Hadoop dependency and update tests / Local [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko commented on PR #665: URL: https://github.com/apache/incubator-graphar/pull/665#issuecomment-2694623519 cc: @Thespica -- 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 specifi

Re: [I] feat(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko closed issue #662: feat(java): shade protobuf dependency in Java info URL: https://github.com/apache/incubator-graphar/issues/662 -- 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

Re: [PR] feat: shade proto [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko merged PR #663: URL: https://github.com/apache/incubator-graphar/pull/663 -- 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-unsubs

Re: [I] feat(java): remove the hadoop dependency from java-info [incubator-graphar]

2025-03-03 Thread via GitHub
SemyonSinchenko commented on issue #664: URL: https://github.com/apache/incubator-graphar/issues/664#issuecomment-2693556480 I will work on removing Hadoop dependency in a way when its integration back will be as smooth as possible! -- This is an automated message from the Apache Git Ser

Re: [PR] feat: shade proto [incubator-graphar]

2025-03-02 Thread via GitHub
SemyonSinchenko commented on PR #663: URL: https://github.com/apache/incubator-graphar/pull/663#issuecomment-2693536931 @Thespica Yes! From now it is ready for the review. **Changes:** - Shading snakeyaml too (it is a hadoop dependency and may cause dependency hell) - Updating

Re: [PR] feat: shade proto [incubator-graphar]

2025-03-02 Thread via GitHub
Thespica commented on PR #663: URL: https://github.com/apache/incubator-graphar/pull/663#issuecomment-2693470176 Is this PR ready? It seems that the dependency-reduce-pom.xml file is temporary, and we do not need to add it to git(from [stackoverflow.com/a/35155294](https://stackoverflow

Re: [I] feat(java): remove the hadoop dependency from java-info [incubator-graphar]

2025-03-02 Thread via GitHub
Thespica commented on issue #664: URL: https://github.com/apache/incubator-graphar/issues/664#issuecomment-2693467741 > [@Thespica](https://github.com/Thespica) cc I remembered that I imported the hadoop dependency so that we can read and wire file from in different places(e.g. file,

Re: [I] feat(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-02 Thread via GitHub
Thespica commented on issue #662: URL: https://github.com/apache/incubator-graphar/issues/662#issuecomment-2693456575 > [@Thespica](https://github.com/Thespica) cc Is this PR ready? It seems that the `dependency-reduce-pom.xml` file is temporary, and we do not need to add it to g

Re: [I] feat(java): remove the hadoop dependency from java-info [incubator-graphar]

2025-03-02 Thread via GitHub
SemyonSinchenko commented on issue #664: URL: https://github.com/apache/incubator-graphar/issues/664#issuecomment-2692634617 @Thespica cc -- 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(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-02 Thread via GitHub
SemyonSinchenko commented on issue #662: URL: https://github.com/apache/incubator-graphar/issues/662#issuecomment-2692626793 Exactly! I slightly changed the pom (will describe in the future commit). And I have now two generated JARs: - info - original info ![Image](https://git

Re: [PR] feat: shade proto [incubator-graphar]

2025-03-02 Thread via GitHub
SemyonSinchenko commented on PR #663: URL: https://github.com/apache/incubator-graphar/pull/663#issuecomment-2692631944 > Looks cool! By shading like this, we can get a jar with info package, right? I answered in the issue -- This is an automated message from the Apache Git Servic

[I] feat(java): remove the hadoop dependency from java-info [incubator-graphar]

2025-03-02 Thread via GitHub
SemyonSinchenko opened a new issue, #664: URL: https://github.com/apache/incubator-graphar/issues/664 ### Describe the enhancement requested At the moment Hadoop is a dependency of the java-info package. But by the design this package was supposed to be a lightweight and (almost) depe

Re: [I] feat(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-02 Thread via GitHub
SemyonSinchenko commented on issue #662: URL: https://github.com/apache/incubator-graphar/issues/662#issuecomment-2692627092 @Thespica cc -- 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: [PR] feat: shade proto [incubator-graphar]

2025-03-01 Thread via GitHub
Thespica commented on PR #663: URL: https://github.com/apache/incubator-graphar/pull/663#issuecomment-2692583038 Looks cool! By shading like this, we can get a jar with info package, right? -- This is an automated message from the Apache Git Service. To respond to the message, please l

[PR] feat: shade proto [incubator-graphar]

2025-03-01 Thread via GitHub
SemyonSinchenko opened a new pull request, #663: URL: https://github.com/apache/incubator-graphar/pull/663 ### Reason for this PR As described in #662 ### What changes are included in this PR? Only shading rules ### Are these changes tested? Changes

Re: [I] feat(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-01 Thread via GitHub
SemyonSinchenko commented on issue #662: URL: https://github.com/apache/incubator-graphar/issues/662#issuecomment-2692403189 @Thespica You can check in #663 -- 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] feat(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-01 Thread via GitHub
SemyonSinchenko commented on issue #662: URL: https://github.com/apache/incubator-graphar/issues/662#issuecomment-2692255500 > Is it means that we generate the proto-code once and save them, so we no longer need protobuf related dependencies when releasing? Yes. All the needed classe

Re: [I] feat(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-01 Thread via GitHub
SemyonSinchenko commented on issue #662: URL: https://github.com/apache/incubator-graphar/issues/662#issuecomment-2692255654 I can actually open a draft PR to explain it by the code -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [I] feat(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-01 Thread via GitHub
Thespica commented on issue #662: URL: https://github.com/apache/incubator-graphar/issues/662#issuecomment-2692254211 > [@Thespica](https://github.com/Thespica) [@acezen](https://github.com/acezen) what do you think? Is it means that we generate the proto-code once and save them, so

Re: [I] feat(java): shade protobuf dependency in Java info [incubator-graphar]

2025-03-01 Thread via GitHub
SemyonSinchenko commented on issue #662: URL: https://github.com/apache/incubator-graphar/issues/662#issuecomment-2692130833 @Thespica @acezen what do you think? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[I] feat: shade protobuf dependency in Java info [incubator-graphar]

2025-03-01 Thread via GitHub
SemyonSinchenko opened a new issue, #662: URL: https://github.com/apache/incubator-graphar/issues/662 ### Describe the enhancement requested Protobuf is very popular but version-sensitive dependency that is used literally everywhere. We should shade it in Java info to simplify the in

Re: [D] Use delta lake data [incubator-graphar]

2025-02-04 Thread via GitHub
GitHub user SemyonSinchenko added a comment to the discussion: Use delta lake data If you have a Databricks you can use GraphX / GraphFrames to read data from delta tables, and after that you can use GAR Spark to save your graphs to your cloud delta lake. I think you can register GAR graphs a

Re: [D] Use delta lake data [incubator-graphar]

2025-02-04 Thread via GitHub
GitHub user MammadTavakoli added a comment to the discussion: Use delta lake data hi @SemyonSinchenko . I have ny data in delta lake and i want have one source for my data GitHub link: https://github.com/apache/incubator-graphar/discussions/661#discussioncomment-12051983 This is an auto

Re: [PR] chore(c++): update fwd.h to add label parameter [incubator-graphar]

2025-02-03 Thread via GitHub
lixueclaire merged PR #652: URL: https://github.com/apache/incubator-graphar/pull/652 -- 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: [D] Use delta lake data [incubator-graphar]

2025-02-03 Thread via GitHub
GitHub user SemyonSinchenko added a comment to the discussion: Use delta lake data Delta tables are parquet files + metadata layer on top of them. GraphAr share the same idea: parquet files and metadata on top of them. But these metadata layers are not compatible between each other. Could you

Re: [I] Looking for anything on my paystring from 2020 ,2021 and where the funds are and how to recover them [incubator-graphar-website]

2025-01-02 Thread via GitHub
lixueclaire closed issue #39: Looking for anything on my paystring from 2020 ,2021 and where the funds are and how to recover them URL: https://github.com/apache/incubator-graphar-website/issues/39 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [I] Looking for what happened to this paystring back in 2021 during the audit that had happened and why it is not recognized anymore nineteen69$paystringcrypto.com [incubator-graphar-website]

2025-01-02 Thread via GitHub
lixueclaire closed issue #38: Looking for what happened to this paystring back in 2021 during the audit that had happened and why it is not recognized anymore nineteen69$paystringcrypto.com URL: https://github.com/apache/incubator-graphar-website/issues/38 -- This is an automated message fr

Re: [I] Find what happened to my audit, and my merkey. The tree root.Nineteen69$paystring.crypto.com [incubator-graphar-website]

2025-01-02 Thread via GitHub
lixueclaire closed issue #36: Find what happened to my audit, and my merkey. The tree root.Nineteen69$paystring.crypto.com URL: https://github.com/apache/incubator-graphar-website/issues/36 -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] chore(ci): Upgrade artifact actions from v3 to v4 [incubator-graphar]

2024-12-23 Thread via GitHub
lixueclaire merged PR #659: URL: https://github.com/apache/incubator-graphar/pull/659 -- 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..

[I] Looking for what happened to this paystring back in 2021 during the audit that had happened and why it is not recognized anymore nineteen69$paystringcrypto.com [incubator-graphar-website]

2024-12-23 Thread via GitHub
Killinskillz69 opened a new issue, #38: URL: https://github.com/apache/incubator-graphar-website/issues/38 Looking for what happened to this paystring back in 2021 during the audit that had happened and why it is not recognized anymore _Originally posted by @Killinskillz69 in https:

Re: [I] Find what happened to my audit, and my merkey. The tree root.Nineteen69$paystring.crypto.com [incubator-graphar-website]

2024-12-23 Thread via GitHub
Killinskillz69 commented on issue #36: URL: https://github.com/apache/incubator-graphar-website/issues/36#issuecomment-2560359675 Looking for what happened to this paystring back in 2021 during the audit that had happened and why it is not recognized anymore -- This is an automated mess

Re: [PR] fix(c++): fix the 'multiple definition' bug when linking libarrow.a and libgraphar_bundled_dependencies.a in one CMakeLists.txt [incubator-graphar]

2024-12-22 Thread via GitHub
lixueclaire merged PR #657: URL: https://github.com/apache/incubator-graphar/pull/657 -- 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..

[I] centic9 has previously contributed to [projects/apache-poi](https://github.com//google/oss-fuzz/tree/master/projects/apache-poi). The previous PR was [#11807](https://www.github.com/google/oss-fuz

2024-11-30 Thread via GitHub
Killinskillz69 opened a new issue, #37: URL: https://github.com/apache/incubator-graphar-website/issues/37 centic9 has previously contributed to [projects/apache-poi](https://github.com//google/oss-fuzz/tree/master/projects/apache-poi). The previous PR was [#11807](https://www.github.com/g

Re: [I] centic9 has previously contributed to [projects/apache-poi](https://github.com//google/oss-fuzz/tree/master/projects/apache-poi). The previous PR was [#11807](https://www.github.com/google/oss

2024-11-30 Thread via GitHub
Killinskillz69 commented on issue #37: URL: https://github.com/apache/incubator-graphar-website/issues/37#issuecomment-2509571066 > centic9 has previously contributed to [projects/apache-poi](https://github.com//google/oss-fuzz/tree/master/projects/apache-poi). The previous PR was [#11807

Re: [I] feat(c++): supprt for multi-property data [incubator-graphar]

2024-11-26 Thread via GitHub
Elssky commented on issue #660: URL: https://github.com/apache/incubator-graphar/issues/660#issuecomment-2503108027 Here is a yml example for multi-property, We introduce `is_multiple` field to distinguish a property whether it is multiple-property ``` yml # person.vertex.yaml type

  1   2   3   4   5   6   7   8   9   10   >