Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.0.0, 1.1.0
    • Component/s: build
    • Labels:
      None

      Description

      Let's bump Ignite to the latest 1.2 to take advantage of all latest improvements and updates. As an exception, I propose to get it into pending 1.0 branch as well.

      1. BIGTOP-1923.patch
        1 kB
        Konstantin Boudnik

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        Committed to branch-1.0 and master.

        Show
        cos Konstantin Boudnik added a comment - Committed to branch-1.0 and master.
        Hide
        cos Konstantin Boudnik added a comment -

        I have ran some validation tests as well: both for in-memory only and with IGFS backed by HDFS as its secondary FS. IGNITE-218 has been fixed for sure and the rest of it seems to be working fine. So I will commit to the master and branch-1.0

        On the more general topic: current Ignite tests won't work because they are using wrong path names in the classpath formation logic. That's easy to fix. And there's BIGTOP-1809 to track this.

        Show
        cos Konstantin Boudnik added a comment - I have ran some validation tests as well: both for in-memory only and with IGFS backed by HDFS as its secondary FS. IGNITE-218 has been fixed for sure and the rest of it seems to be working fine. So I will commit to the master and branch-1.0 On the more general topic: current Ignite tests won't work because they are using wrong path names in the classpath formation logic. That's easy to fix. And there's BIGTOP-1809 to track this.
        Hide
        evans_ye Evans Ye added a comment -

        OK. The pi job rans smoothly with ignite accelerator. +1. I'll left this to you to commit.

        Show
        evans_ye Evans Ye added a comment - OK. The pi job rans smoothly with ignite accelerator. +1. I'll left this to you to commit.
        Hide
        evans_ye Evans Ye added a comment -

        Thanks YoungWoo Kim, build ok on my end as well. I'm currently running deployment. Might be able to run one or two smoke test manually.

        Show
        evans_ye Evans Ye added a comment - Thanks YoungWoo Kim , build ok on my end as well. I'm currently running deployment. Might be able to run one or two smoke test manually.
        Hide
        warwithin YoungWoo Kim added a comment -

        +1, Packaging rpm and deb works on my end.

        However, verification is hard for me because ignite-smokes is a bit outdated. Konstantin Boudnik, I don't have expertise in Ignite so please do the verification and then commit this! if so, I believe we should be fine. Thanks!

        Show
        warwithin YoungWoo Kim added a comment - +1, Packaging rpm and deb works on my end. However, verification is hard for me because ignite-smokes is a bit outdated. Konstantin Boudnik , I don't have expertise in Ignite so please do the verification and then commit this! if so, I believe we should be fine. Thanks!
        Hide
        cos Konstantin Boudnik added a comment -

        Here we go.

        Show
        cos Konstantin Boudnik added a comment - Here we go.

          People

          • Assignee:
            cos Konstantin Boudnik
            Reporter:
            cos Konstantin Boudnik
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development