Hello, Community.
On this issue, I wrote a code that removes AgensGraph and everything related to
a specific company.
We are closing this vote to fix the issue and will reopen later.
Thanks.
Alex
On 2022/02/03 21:21:56 Josh Innis wrote:
> Hi Trista,
>
> [1] When AGE Viewer was donated to Apache, it already supported both
> AgensGraph and AGE as backend database connections. To completely remove
> all references to AgensGraph would mean removing AGE Viewer's ability to
> connect to AgensGraph as a backend database.
> [2] Before AGE was donated to Apache, its name was AgensGraph-Extension. In
> the proposal process, the community determined that the project had to be
> renamed. AgensGraph itself is not a part of Apache.
>
> Thanks,
> Josh
>
> On Thu, Feb 3, 2022 at 4:28 AM Juan Pan <panj...@apache.org> wrote:
>
> > Hi Josh,
> >
> >
> > Actually, our ppmc of AGE more care about AGE project (its brand,
> > community, etc.) rather than AgensGraph.
> > My suggestion is to erase the marks around them as possible as we can for
> > safety, based on
> > 1. Why do our AGE viewer need to be compatible with AgensGraph? Is
> > AgensGraph part of this incubator project?
> > 2. AgensGraph is suggested to be renamed in gene...@incubator.ml by other
> > IPMCs, I remember?
> >
> >
> > Best,
> > Trista
> >
> > ------------------------------------------------------------------------------
> > Trista Pan, Twitter: @tristaZero
> >
> >
> > On 02/3/2022 01:26，Josh Innis<joshin...@gmail.com> wrote：
> > HI Juan,
> >
> > 1. AGE Viewer connects to two databases: Apache AGE and AgensGraph. No code
> > from AgensGraph resides in AGE Viewer. When AGE Viewer is connected to
> > AgensGraph, is it an issue if AGE Viewer gives a context specific language
> > in code or in messages given to the user?
> >
> > For example:
> > backend/src/util/JsonBuilder.js:66
> > if (flavor === 'AGE') {
> > return AGECreateEdge(client, graphPathStr, createQ);
> > } else {
> > return AgensGraphCreateEdge(client, graphPathStr, createQ);
> > }
> >
> > Since AgensGraphCreateEdge is used only when AGE Viewer is connected to
> > AgensGraph, is it necessary to change it?
> >
> > 2. Some of the results are for modules that handles the connection to the
> > database. It looks like this is from when the source code was originally
> > donated to Apache from Bitnine. These modules probably need to be renamed
> > to something generic.
> >
> > 3. Since AGE Viewer originally connected to AgensGraph and extended to
> > support AGE there are some hardcoded values that need to be altered.
> >
> > For example:
> > frontend/src/components/frame/presentations/ServerStatusFrame.jsx:86
> > if (status === 'disconnected') {
> > return (
> > ...
> > <h3>Connection Status</h3>
> > <p>You are currently not connected to AgensGraph</p>
> > ...
> >
> > This code displays a message to the user when there is a connection error.
> > Personally, I think this needs a conditional expression for the type of
> > database the user is connecting to, but if all references to Agensgraph
> > need to be removed it could instead display: <p>You are currently not
> > connected</p> for all database connection types.
> >
> > 4. Many of the results come from the file
> > frontend/src/documents/tutorial/northwind.jsx. This file is a tutorial that
> > shows how to use AGE Viewer with AgensGraph. Should this file be removed?
> >
> > On Wed, Feb 2, 2022 at 3:39 AM Juan Pan <panj...@apache.org> wrote:
> >
> > I also have no idea about this vote.
> >
> >
> > I tracked the whole thread and found others raised some issues, but some
> > have had no feedback so far. Could anyone sync me this vote is still in the
> > process and double check the issue I mentioned?
> >
> >
> > It is the first time for the community to release AGE-view. It is
> > efficient to solve all the potential issues internally and definitely will
> > cost our lots of efforts and patients.
> >
> >
> > My check list,
> >
> >
> > [x] Download links are valid.
> > [x] Checksums and PGP signatures are valid.
> > [ ] LICENSE, NOTICE and DISCLAIMER files are correct. (Need double check)
> > [x] All files have license headers if necessary.
> >
> >
> > Most of the files are fine, but it still has some bits around AgensGraph
> > or bitnine.
> >
> >
> > Run
> > ```
> > grep bitnine . -R -n
> > grep AgensGraph . -R -n
> > ```
> > It will show you, like
> > ```
> > ./frontend/src/components/frame/presentations/ServerStatusFrame.jsx:92:
> > <p>You are currently not connected to AgensGraph</p>
> > ./frontend/src/components/frame/presentations/ServerDisconnectFrame.jsx:32:
> > disconnectToAgensGraph,
> > ```
> >
> >
> > Please keep in mind Apache AGE totally has nothing to do with AgensGraph
> > or bitnine business currently and in the future. (Maybe AgensGraph is in
> > consideration to rename?)
> >
> >
> >
> >
> >
> > ------------------------------------------------------------------------------
> > Juan Pan(Trista), Twitter: @tristaZero
> >
> >
> > On 02/1/2022 02:10，John Gemignani<jgemign...@apache.org> wrote：
> > I will give this a +1.
> >
> > However, I haven't seen any traction on this thread for a while. Should
> > this be closed and then reopened? Or, should we continue to try to get
> > enough votes? Or?
> >
> > Mentors, could you please give your input?
> >
> > Thanks!
> >
> > John
> >
> > On 2022/01/10 04:38:32 Alex Kwak wrote:
> > Sorry, All.
> >
> > It was a mistake to close it. We will continue voting.
> >
> > On 2022/01/10 02:37:21 Alex Kwak wrote:
> > Hello Community,
> >
> > We are closing the vote. Results will be sent shortly.
> >
> > On 2022/01/06 02:20:46 Alex Kwak wrote:
> > Dear Apache AGE Community,
> >
> > This is an official vote for the Apache AGE Viewer release v1.0.0-rc1 that
> > we have been working toward it.
> >
> > To learn more about Apache AGE, please see http://age.apache.org/
> >
> > Functionalities included and addressed in this release:
> > - Graph visualization for AGE.
> > - Extends edge and vertex point by point.
> >
> >
> > The vote is open now and until January 8th at 6:00 PM PST and passes if a
> > majority +1 votes are cast, with a minimum of 3 +1 votes.
> >
> > [ ] +1 Release this package as Apache AGE Viewer v1.0.0
> > [ ] 0 I won't get in the way
> > [ ] -1 Do not release this package because ...
> >
> >
> > The git tag to be discussed and voted upon
> > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0-rc1
> >
> > The git commit hash:
> > commit 2b7fe6018d9da5ce10cf206cc6e879577a3d8051
> >
> > The release files, including signatures, digests, etc. can be found at:
> >
> >
> > https://dist.apache.org/repos/dist/dev/incubator/age/viewer/apache-age-viewer-1.0.0-incubating-rc1/
> >
> > The SHA512 Checksum for these artifacts is:
> >
> >
> > dce65e48c222d4bd0b1be8f571495b50217fcf60d82599702f3e42b36cc5afb0045b798099f981c0db13ed76a94157f9f6dadeb5940a20756bb5c2f6fb6b8009
> >
> > Release artifacts are signed with the following key:
> > https://downloads.apache.org/incubator/age/KEYS
> >
> > The fingerprint of key to sign release artifacts:
> > 0E7F 408D 8C6A 1952 329C B379 D471 FDCE 5F5C 5B82
> >
> > For more information about the contents of this release, see:
> > https://github.com/apache/incubator-age-viewer/releases/tag/v1.0.0-rc1
> >
> >
> >
> > Best regards,
> > Alex Kwak
> >
> >
> >
> >
> >
>
