ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-708

zkpython failing due to undefined symbol deallocate_String_vector

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Blocker Blocker
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 3.3.0
        • Component/s: c client, contrib-bindings
        • Labels:
          None
        • Hadoop Flags:
          Reviewed

          Description

          ant test in zkpython is failing. I think this is due to mahadev's changes to remove unnecessary exports from the client lib.

          [exec] ImportError: /home/phunt/dev/workspace/gitzk/build/contrib/zkpython/lib.linux-x86_64-2.6/zookeeper.so: undefined symbol: deallocate_String_vector

          Mahadev can you take a look?

          1. Text File
            ZOOKEEPER-708.patch
            0.7 kB
            Mahadev konar
          2. Text File
            ZOOKEEPER-708.patch
            0.6 kB
            Mahadev konar
          3. Text File
            ZOOKEEPER-708.patch
            0.6 kB
            Mahadev konar

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Mahadev konar added a comment -

            good catch pat!!! Ill fix it....

            Show
            Mahadev konar added a comment - good catch pat!!! Ill fix it....
            Hide
            Permalink
            Mahadev konar added a comment -

            this patch should fix the issue.

            Show
            Mahadev konar added a comment - this patch should fix the issue.
            Hide
            Permalink
            Patrick Hunt added a comment -

            now I'm seeing another issue :

            [exec] ImportError: /home/phunt/dev/workspace/gitzk/build/contrib/zkpython/lib.linux-x86_64-2.6/zookeeper.so: undefined symbol: zerror

            Show
            Patrick Hunt added a comment - now I'm seeing another issue : [exec] ImportError: /home/phunt/dev/workspace/gitzk/build/contrib/zkpython/lib.linux-x86_64-2.6/zookeeper.so: undefined symbol: zerror
            Hide
            Permalink
            Mahadev konar added a comment -

            can you try this?

            Show
            Mahadev konar added a comment - can you try this?
            Hide
            Permalink
            Hadoop QA added a comment -

            -1 overall. Here are the results of testing the latest attachment
            http://issues.apache.org/jira/secure/attachment/12439120/ZOOKEEPER-708.patch
            against trunk revision 924521.

            +1 @author. The patch does not contain any @author tags.

            -1 tests included. The patch doesn't appear to include any new or modified tests.
            Please justify why no tests are needed for this patch.

            +1 javadoc. The javadoc tool did not generate any warning messages.

            +1 javac. The applied patch does not increase the total number of javac compiler warnings.

            +1 findbugs. The patch does not introduce any new Findbugs warnings.

            +1 release audit. The applied patch does not increase the total number of release audit warnings.

            +1 core tests. The patch passed core unit tests.

            +1 contrib tests. The patch passed contrib unit tests.

            Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/140/testReport/
            Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/140/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
            Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/140/console

            This message is automatically generated.

            Show
            Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12439120/ZOOKEEPER-708.patch against trunk revision 924521. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no tests are needed for this patch. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/140/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/140/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/140/console This message is automatically generated.
            Hide
            Permalink
            Patrick Hunt added a comment -

            third patch addresses all zkpython issues - zkpython tests all pass.

            Show
            Patrick Hunt added a comment - third patch addresses all zkpython issues - zkpython tests all pass.
            Hide
            Permalink
            Patrick Hunt added a comment -

            +1 looks good, committed

            Show
            Patrick Hunt added a comment - +1 looks good, committed
            Hide
            Permalink
            Hudson added a comment -

            Integrated in ZooKeeper-trunk #761 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/761/)
            . zkpython failing due to undefined symbol deallocate_String_vector

            Show
            Hudson added a comment - Integrated in ZooKeeper-trunk #761 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/761/ ) . zkpython failing due to undefined symbol deallocate_String_vector

              People

              • Assignee:
                Mahadev konar
                Reporter:
                Patrick Hunt
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development