[Question] Why don't we need redirectMasterRole annotation in VertexApi and EdgeApi about incubator-hugegraph HOT 6 CLOSED

Hi, your understanding is generally correct. The master/slaver design is mainly used to perform asynchronous tasks. The master node distributes tasks and the slave performs tasks. For requests that need to create tasks, such as rebuilding the index, the request needs to be redirected to the master. No. CRUD requests that need to create tasks do not need to go through the master. If we only have one node, there will be a single point problem.
VertexApi/EdgeApi does not create asynchronous tasks, so it can be executed on the any both node

from incubator-hugegraph.

but why do GremlinApi need this annotation? see this https://github.com/apache/incubator-hugegraph/blob/master/hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/api/gremlin/GremlinAPI.java and following code.

    @POST
    @Timed
    @Compress
    @Consumes(APPLICATION_JSON)
    @Produces(APPLICATION_JSON_WITH_CHARSET)
    @RedirectFilter.RedirectMasterRole
    public Response post(@Context HugeConfig conf,
                         @Context HttpHeaders headers,
                         String request) {
        /* The following code is reserved for forwarding request */
        // context.getRequestDispatcher(location).forward(request, response);
        // return Response.seeOther(UriBuilder.fromUri(location).build())
        // .build();
        // Response.temporaryRedirect(UriBuilder.fromUri(location).build())
        // .build();
        String auth = headers.getHeaderString(HttpHeaders.AUTHORIZATION);
        Response response = this.client().doPostRequest(auth, request);
        GREMLIN_INPUT_HISTOGRAM.update(request.length());
        GREMLIN_OUTPUT_HISTOGRAM.update(response.getLength());
        return transformResponseIfNeeded(response);
    }

from incubator-hugegraph.

another issue is that the Apis that add and update vertex/edge/index label won't create async task as well, but these apis are also annotated with RedirectMasterRole. Some of the references can be viewed as following：

• org.apache.hugegraph.api.schema.VertexLabelAPI#create
• org.apache.hugegraph.api.schema.VertexLabelAPI#update

I cannot find any clue to create async tasks in the schema update/add code.

    protected void updateSchema(SchemaElement schema,
                                Consumer<SchemaElement> updateCallback) {
        LOG.debug("SchemaTransaction update {} with id '{}'",
                  schema.type(), schema.id());
        this.saveSchema(schema, true, updateCallback);
    }

    protected void addSchema(SchemaElement schema) {
        LOG.debug("SchemaTransaction add {} with id '{}'",
                  schema.type(), schema.id());
        setCreateTimeIfNeeded(schema);
        this.saveSchema(schema, false, null);
    }

Thx.

from incubator-hugegraph.

Thank you for your question. It seems that this annotation is not needed in these places. I will wait for other people in the community to see their opinions.

from incubator-hugegraph.

Do U have a realtime communication way, like Zoom or wechat meeting, to discuss some technique problem? so that I can report some concurrent safety issues and related concerns which are to some extent hard to describe in github.

Just ignore the above request. It's fine to use github issue and PR workflow. let's respect the method that an open-source project often operates.

from incubator-hugegraph.

Do U have a realtime communication way, like Zoom or wechat meeting, to discuss some technique problem? so that I can report some concurrent safety issues and related concerns which are to some extent hard to describe in github.

Just ignore the above request. It's fine to use github issue and PR workflow. let's respect the method that an open-source project often operates.

Thank you for your understanding and support. We also have groups to discuss issues and more documents. You can send me an email (as I'm not sure about your email address)

BTW, current master-worker mechanism will be replaced by PD in future (refer #2265)

from incubator-hugegraph.