• Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: 2.2
    • Component/s: None
    • Security Level: All
    • Labels:


      Pull-Request was automatically synchronized:

      Hi there,
      I'm wondering myself if the QueryBuilder::getQuery is a good method name.
      Actually, a getMethodObject makes me think that I can use many times the getMethodObject and work with the same object.
      e.g :

      $qb->getQuery()->useResultCache (true);
      In my code, the problem is that getQuery return a new instance of Query each time that I call getQuery.
      Furthermore, the getQuery method invoke a createQuery method.

      So, maybe the QueryBuiler::getQuery can store a reference to the created Query instance, and if the QueryBuilder::getQuery is call many times without any change return the stored instance.
      If the QueryBuilder::getQuery have to generate a new Query, throw an exception or something like that.

      Because of a change to QueryBuilder::createQuery is a major change, I don't think that It could be the solution, but it's a reflexion around this.

      What do you think about that ?


        Benjamin Eberlei created issue -
        Guilherme Blanco added a comment -

        This includes a BC break.
        Marking as won't fix.

        Guilherme Blanco added a comment - This includes a BC break. Marking as won't fix.
        Guilherme Blanco made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Benjamin Eberlei [ beberlei ] Guilherme Blanco [ guilhermeblanco ]
        Fix Version/s 2.2-DEV [ 10157 ]
        Resolution Won't Fix [ 2 ]
        Benjamin Eberlei made changes -
        Workflow jira [ 13262 ] jira-feedback [ 15147 ]
        Benjamin Eberlei made changes -
        Workflow jira-feedback [ 15147 ] jira-feedback2 [ 17011 ]
        Benjamin Eberlei made changes -
        Workflow jira-feedback2 [ 17011 ] jira-feedback3 [ 19264 ]

        This list may be incomplete, as errors occurred whilst retrieving source from linked applications:

        • Request to failed: Error in remote call to 'FishEye 0 (' ( [AbstractRestCommand{path='/rest-service-fe/search-v1/crossRepositoryQuery', params={query=DDC-1536, expand=changesets[0:20].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)


          • Assignee:
            Guilherme Blanco
            Benjamin Eberlei
          • Votes:
            0 Vote for this issue
            0 Start watching this issue


            • Created: