1 2 Previous Next 19 Replies Latest reply on Oct 27, 2008 2:55 AM by timfox Go to original post
      • 15. Re: Ordering Group -- Transaction Scenario Analysis (Typical
        gaohoward
        gaohoward Oct 24, 2008 7:36 AM (in response to gaohoward)

         

        "timfox" wrote:
        "gaohoward" wrote:
        Tim, the message group is not transaction aware.


        Does it need to be transaction aware?

        As we discussed, it may still work under tx env. So far I haven't figured out an exception where it won't work. But I think it won't make much difference to get the message group code over to 1.4, as my code is already there and can do the same.


        I would rather use code that we know works than use new code that does the same thing.




        That's what i'm not sure at the moment, if it doesn't need to be tx aware, that's great to us. I just want to hold my code for some time to allow me to think the issue over and then make a decision.


          • Actions
        • 16. Re: Ordering Group -- Transaction Scenario Analysis (Typical
          gaohoward
          gaohoward Oct 24, 2008 7:39 AM (in response to gaohoward)

           

          "timfox" wrote:
          In other words, don't reinvent the wheel! ;)


          I know, but if you just have a look at my branch, u'll find mine is not just 'reinvent the wheel". I'm reinventing a 'worse' wheel. :)





            • Actions
          • 17. Re: Ordering Group -- Transaction Scenario Analysis (Typical
            timfox
            timfox Oct 24, 2008 7:42 AM (in response to gaohoward)

            At this point, don't worry about your code.

            We just need to find out if a simple grouping approach as we discussed works.

              • Actions
            • 18. Re: Ordering Group -- Transaction Scenario Analysis (Typical
              gaohoward
              gaohoward Oct 26, 2008 10:35 PM (in response to gaohoward)

              Yes, my code is almost there and what I need next is tests.

              The grouping approach should work but I still need to change it to handle priorities, or multiple MDB cases for example. Once I put some changes in grouping code, it's polluted and still need tests.
              So I'd rather keep my code and writing tests now.

                • Actions
              • 19. Re: Ordering Group -- Transaction Scenario Analysis (Typical
                timfox
                timfox Oct 27, 2008 2:55 AM (in response to gaohoward)

                Howard - don't get too hung up on your code. :)

                First I would like us to discuss how the current grouping approach (from JBM 2.0) does or does not solve the problem, and what changes would need to be made to that code to get it to work.

                This includes the MDB case.

                Once we've worked out how to solve the problem, we can think about what actual code to use, and write tests

                BTW I'm not sure what you meant by "change it to handle priorities"

                  • Actions
                1 2 Previous Next